-
Notifications
You must be signed in to change notification settings - Fork 25
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
Pop-ups open off screen on laptop and mobile devices #35
Comments
@MikeMillerGIS Is this a core Popup issue or is it tied to Information Lookup specifically? If it is the later, can we get this resolved soon? @previnWong |
We center the map and show pop up, map does not move itself. maybe there is a setting or something? Can they use the side panel? |
maybe something we can leverage in this thread |
@karmstro77 @NikkiGolding Made a fix to center the popup to support mobile platform. By doing so, though, the point clicked is offset from center in the display. Give it a shot and let me know. Get the version in the dev branch. |
@previnWong We should make this an option. Will this change will affect and provide a different UX from an app today? |
more worried about desktop browser based, will it only move the map if the pop up is cut off? |
The existing code in there calls for the map to center and zoom. So the map in the desktop browser will move as of today. I'm just changing it slightly where it centers. It should do the same for desktop browser, push it away from top or right from getting cut off. Speaking of it now, I haven't done the case where you click and select the underlying feature. I will have to handle that case as well. |
The text was updated successfully, but these errors were encountered: