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 #10358

Closed
wants to merge 12 commits into from

Conversation

wajeehamushtaq
Copy link
Contributor

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.

Screenshot from 2021-10-12 20-34-37

Screenshot from 2021-10-12 20-34-59

@gitpod-io
Copy link

gitpod-io bot commented Oct 19, 2021

@codecov
Copy link

codecov bot commented Oct 19, 2021

Codecov Report

Merging #10358 (8a4bb4d) into main (994d0b4) will decrease coverage by 0.11%.
The diff coverage is n/a.

❗ Current head 8a4bb4d differs from pull request most recent head 1323436. Consider uploading reports for the commit 1323436 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main   #10358      +/-   ##
==========================================
- Coverage   82.11%   81.99%   -0.12%     
==========================================
  Files          98       98              
  Lines        5982     5982              
==========================================
- Hits         4912     4905       -7     
- Misses       1070     1077       +7     
Impacted Files Coverage Δ
app/services/execute_search.rb 88.88% <0.00%> (-5.56%) ⬇️
app/api/srch/search.rb 66.87% <0.00%> (-3.83%) ⬇️

@wajeehamushtaq
Copy link
Contributor Author

Hi @jywarren, I have followed your steps. Is it fine now?

@codeclimate
Copy link

codeclimate bot commented Oct 20, 2021

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

View more on Code Climate.

@jywarren
Copy link
Member

jywarren commented Nov 9, 2021

/rebase

@jywarren
Copy link
Member

jywarren commented Nov 9, 2021

This looks really good, my apologies for missing it in my notifications! Just rebasing to get past the test issues, which i think are unrelated. Let's see! 🤞

@jywarren
Copy link
Member

jywarren commented Nov 9, 2021

Hi, would you mind rebasing this over the latest main branch? i can't seem to do it easily from my end. That will resolve the test! Apologies and thanks!

@wajeehamushtaq
Copy link
Contributor Author

Hi @jywarren , Can you please guide me about rebasing it to the main branch? Thank you!

@jywarren
Copy link
Member

Hi @wajeehamushtaq of course, sorry for the slow response!


It looks like you need to rebase your work on top of the latest main. You can do that by following the process outlined here, but you may be able to skip step 2 if your main is still in sync with publiclab's main: https://publiclab.org/wiki/contributing-to-public-lab-software#Rewinding+the+master+branch

In general there are more tips and guides on rebasing here: https://publiclab.org/wiki/developers#Resources

Thanks!

@Tlazypanda
Copy link
Collaborator

Hey @wajeehamushtaq feel free to ping in case you are running into any issues. We are here to help ✌️

@cesswairimu
Copy link
Collaborator

Hi @wajeehamushtaq, we are unable to move forward without a rebase here, we understand you might be busy and engaged on other things. The solution was great and I will use the solution to convert the issue to a first-timer issue - thanks for working on this. Please feel free to come back anytime and ask for another issue to work on. Closing this. Thanks all

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)
4 participants