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

Pb 859 info box positionpbox together - #patch #1033

Merged
merged 2 commits into from
Aug 22, 2024

Conversation

ismailsunni
Copy link
Contributor

@ismailsunni ismailsunni commented Aug 20, 2024

image

Introduce a new store key handle the location popup independently. Previously it uses clickInfo which can be modified and hide locationPopup unintentionally.

Test link

@github-actions github-actions bot changed the title Pb 859 info box positionpbox together Pb 859 info box positionpbox together - #minor Aug 20, 2024
Copy link

cypress bot commented Aug 20, 2024



Test summary

210 0 21 0


Run details

Project web-mapviewer
Status Passed
Commit 818e900
Started Aug 22, 2024 6:26 AM
Ended Aug 22, 2024 6:31 AM
Duration 04:31 💡
OS Linux Ubuntu -
Browser Electron 118

View run in Cypress Cloud ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Cloud

@ismailsunni ismailsunni force-pushed the pb-859-info-box-positionpbox-together branch from 9a43a53 to 0a3d27f Compare August 20, 2024 11:54
@ismailsunni ismailsunni requested review from pakb and ltshb August 20, 2024 12:09
@pakb pakb changed the title Pb 859 info box positionpbox together - #minor Pb 859 info box positionpbox together - #patch Aug 22, 2024
@ismailsunni ismailsunni force-pushed the pb-859-info-box-positionpbox-together branch from 0a3d27f to 818e900 Compare August 22, 2024 06:23
@ismailsunni ismailsunni merged commit 54a765c into master Aug 22, 2024
6 checks passed
@ismailsunni ismailsunni deleted the pb-859-info-box-positionpbox-together branch August 22, 2024 06:37
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.

2 participants