Layer palette unreachable on mobile #1260
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Deploy Preview for labs-zola ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Member
ZoLa is configured to automatically create deploy previews on Netlify; running it locally isn't strictly necessary to test the behavior. There's apparently also a QR code so that it's easy to open on a mobile device. |
TangoYankee
approved these changes
Jun 17, 2025
pratishta
force-pushed
the
1228/layer-palette-unreachable-on-mobile
branch
2 times, most recently
from
f7ab1bc to
b946f5c
Compare
pratishta
force-pushed
the
1228/layer-palette-unreachable-on-mobile
branch
from
cd9ed58 to
36c449e
Compare
Personal local development Stop tracking vscode settings Restore lock files Remove yarn Re-adding yarn.lock Re-adding yarn.lock again
pratishta
force-pushed
the
1228/layer-palette-unreachable-on-mobile
branch
from
36c449e to
59b2328
Compare
TylerMatteo
added a commit
that referenced
this pull request
Jun 25, 2025
#1267) * Issue 1219 Updated Asset Link in ZOLA * Issue 1222 Updated Web Link * Digital ocean tax maps Update digital tax map hyperlinks to point to digital ocean spaces closes #1221 * Bugfix: incorrect zoning map path Add leading zero to zoning map path closes #1221 * Adds Zoning Map Index Sidebar * Refactored to pull data from Carto. * Moved inline styles to scss * String manipulation of header text. * Removed unneccessary file * Issue 1229 Add Zoning Index Layer to ZOLA * Issue 1242 Enabled Clickable Zoning Index Map Grid Squares - enabled select stying to be applied to clicked ZMI grids - enabled bookmarks with formatted titles * Issue 1243 Add Zoning for Accessibility Layer - Closes #1243 - Closes #1244 - Closes #1245 - Closes #1246 - Closes #1247 - Closes #1248 * Added Zoning Map Index to new features modal and features page. * Issue 1258 Update ZFA Slide Out Description text Closes #1258 * Add Zoning Map Index to Features section * Change hideMessage local storage variable name so users see modal with new Zoning Map Index Feature * Update intersecting layer template to hide ZFA layer behind feature flag * Layer palette unreachable on mobile (#1260) * Remove overflow property to enable scroll * Add vscode settings to gitignore Personal local development Stop tracking vscode settings Restore lock files Remove yarn Re-adding yarn.lock Re-adding yarn.lock again * Issue 1253 Add ZFA messaging to ZOLA - Add demo gif of ZFA functionality - Add ZFA callout messaging to popup and features tab - change local storage key to trigger pop-up with new ZFA messaging * Update netlify.toml to set ZFA feature flag to ON (#1266) --------- Co-authored-by: horatio <[email protected]> Co-authored-by: tangoyankee <[email protected]> Co-authored-by: David Hochbaum <[email protected]> Co-authored-by: py <[email protected]> Co-authored-by: horatio <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #1228
To replicate the bug / test out the PR:
Now you should not be able to scroll back up or down. This PR should fix that.*
overflowproperty I removed was initially in place to fix a mobile scrolling issue. So it may be that there are some knock-on effects