Skip to content

PB-1561: revert CSS changes to generic feature pop up windows#1299

Merged
pakb merged 2 commits intodevelopfrom
feat-PB-1561-drawing-tooltip-descriptions-readability
Apr 16, 2025
Merged

PB-1561: revert CSS changes to generic feature pop up windows#1299
pakb merged 2 commits intodevelopfrom
feat-PB-1561-drawing-tooltip-descriptions-readability

Conversation

@ltkum
Copy link
Contributor

@ltkum ltkum commented Apr 15, 2025

Issue : we adapted the CSS for the generic pop up which holds the 3d features info for now , but the styling doesn't please our users who are using the drawings.

Fix : We revert the changes and the 3d pop up will be done when we can get it from the backend :)

Test link

@ltkum ltkum requested a review from pakb April 15, 2025 07:24
@cypress
Copy link

cypress bot commented Apr 15, 2025

web-mapviewer    Run #5046

Run Properties:  status check passed Passed #5046  •  git commit 4d78642825: Merge pull request #1299 from geoadmin/feat-PB-1561-drawing-tooltip-descriptions...
Project web-mapviewer
Branch Review develop
Run status status check passed Passed #5046
Run duration 05m 42s
Commit git commit 4d78642825: Merge pull request #1299 from geoadmin/feat-PB-1561-drawing-tooltip-descriptions...
Committer Pascal Barth
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 22
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 246
View all changes introduced in this branch ↗︎

@ltkum ltkum force-pushed the feat-PB-1561-drawing-tooltip-descriptions-readability branch from 16e937a to 9a224f2 Compare April 15, 2025 11:57
Copy link
Contributor

@pakb pakb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there's no more floating edit window when drawing, please fix this

Issue : we adapted the CSS for the generic pop up which holds the 3d features info for now
, but the styling doesn't please our users who are using the drawings.
Fix : We revert the changes and the 3d pop up will be done when we can get it
from the backend :)
@pakb pakb force-pushed the feat-PB-1561-drawing-tooltip-descriptions-readability branch 2 times, most recently from a82a004 to 68ec52b Compare April 16, 2025 09:23
still incorporating the new SMS REGEX and the fact that we need to translate the content of the feature so that the 3D building tooltip works as expected
@pakb pakb force-pushed the feat-PB-1561-drawing-tooltip-descriptions-readability branch from 68ec52b to 83ee819 Compare April 16, 2025 09:51
@pakb pakb merged commit 4d78642 into develop Apr 16, 2025
6 checks passed
@pakb pakb deleted the feat-PB-1561-drawing-tooltip-descriptions-readability branch April 16, 2025 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants