Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Map floats over the footer while scrolling! - UI Bug #10327

Closed
wants to merge 7 commits into from

Conversation

wajeehamushtaq
Copy link
Contributor

@wajeehamushtaq wajeehamushtaq commented Oct 13, 2021

Fixes #10306
I try to fix the Map floats over the footer while scrolling! - UI Bug (1/3). The problem arise due to leaflet map styling. The position is set to absolute so this will create the problem.

WhatsApp Image 2021-10-13 at 9 16 05 AM

WhatsApp Image 2021-10-13 at 9 16 26 AM

Sorry for the unclear pictures.

@wajeehamushtaq wajeehamushtaq requested a review from a team as a code owner October 13, 2021 04:28
@gitpod-io
Copy link

gitpod-io bot commented Oct 13, 2021

@wajeehamushtaq wajeehamushtaq changed the title Fixes #10306 Map floats over the footer while scrolling! - UI Bug Oct 17, 2021
@jywarren
Copy link
Member

Hi! I believe this is an error unrelated to your code - due to #10341 and my fault! I'm restarting the test to try to get past it.

Warning: NING] Unrecognized arguments: google_project, google_json_key_location
Progress: |================================================ FAIL TagTest#test_contributors_with_timeframe (7.57s)
        Expected: [1, 2]
          Actual: [1, 2, 5, 6, 12]

@jywarren
Copy link
Member

Yay looks like this worked well!

@@ -2,7 +2,15 @@
<div class="card-view questions-page row">
<% questions.each_with_index do |node,i| %>
<div class="<% if @widget %>col-xs-4 col-sm-4 col-md-4 <% end %>col-lg-4" style="margin-top:20px;">
<%= render partial: 'notes/card', locals: { node: node, i: i, tagname: @tagname } %>
<div class="card bg-white rounded<% if node.status == 4 %> moderated<% end %>">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh hi @wajeehamushtaq it looks like changes from #10336 made it into this PR too. If both check out, we can just ignore and merge both. But in the future this may come up in relation to working from your main branch, so all your changes get mixed together. Instead, each time you work on a new PR, try branching off a new branch name from main so that they are all separate divergences from the main branch. Then you can keep updating your local main branch from publiclab/plots2 main branch, and keep working off a recent copy of that. Does that make sense?

For future PRs you may need to do 'git pull -f https://github.com/publiclab/plots2.git mainto overwrite your local main branch so that you have an updated copy to work from. Just be sure you don't overwrite changes you want to be using! You can first save them by runninggit branch myFeatureBranchName` (using your own custom name) and they'll be saved in their own branch. Happy to help with more Git stuff too! There's a nice guide on this workflow here: https://guides.github.com/introduction/flow/

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @jywarren , I got my mistake. I will fix it

@codeclimate
Copy link

codeclimate bot commented Oct 19, 2021

Code Climate has analyzed commit 7081b00 and detected 0 issues on this pull request.

View more on Code Climate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Map floats over footer while scrolling! - UI Bug (1/3)
2 participants