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

Fix location modal jumping off-screen at small screen sizes #7284

Merged
merged 1 commit into from
Jan 17, 2020

Conversation

nstjean
Copy link
Contributor

@nstjean nstjean commented Jan 17, 2020

Fixes #7283 (<=== Add issue number here)

I'm not sure where the negative margin is coming from, but setting it to 0 for #blurred-location-modal .modal-content fixes the problem.

@codecov
Copy link

codecov bot commented Jan 17, 2020

Codecov Report

Merging #7284 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #7284      +/-   ##
=========================================
+ Coverage   81.59%   81.6%   +<.01%     
=========================================
  Files          97      97              
  Lines        5602    5599       -3     
=========================================
- Hits         4571    4569       -2     
+ Misses       1031    1030       -1
Impacted Files Coverage Δ
app/models/tag.rb 97.46% <ø> (+0.46%) ⬆️
app/controllers/user_sessions_controller.rb 66.45% <0%> (-1.25%) ⬇️
app/controllers/users_controller.rb 81.69% <0%> (+0.35%) ⬆️
app/models/spamaway.rb 94.87% <0%> (+2.56%) ⬆️

@jywarren jywarren merged commit a175adc into publiclab:master Jan 17, 2020
@jywarren
Copy link
Member

Great! Thanks!!

vinitshahdeo pushed a commit to vinitshahdeo/plots2 that referenced this pull request Feb 1, 2020
NitinBhasneria pushed a commit to NitinBhasneria/plots2 that referenced this pull request Feb 5, 2020
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.

Location modal jumps off-screen at small screen sizes
2 participants