Skip to content

New Release v1.55.0 - #minor#1335

Merged
pakb merged 109 commits intomasterfrom
develop
May 12, 2025
Merged

New Release v1.55.0 - #minor#1335
pakb merged 109 commits intomasterfrom
develop

Conversation

@pakb
Copy link
Contributor

@pakb pakb commented May 9, 2025

ismailsunni and others added 30 commits April 17, 2025 11:50
Issue : An overflow was causing two scroll bars to appear on Chromium browsers
when using the 3D view.

Fix : removed the fluid css condition and set a consistent max-height of 100%
…-tooltip

PB-1202 : Fix overflow issue in 3D feature tooltip view on Chrome
Running pnpm run build:dev:watch permits the user to modify the geodamin package and updating the build so that the changes have direct effect on the running mapviewer.

Note: I observed that the mapviewer updates the open page too fast, which makes it not load correctly and a reload of the page is sometimes needed.Add a build dev watch option to geoadmin packages
Updating all i18n translation JSON through the script (some values have been formatted or slightly changed because of that)

Tweaking a bit the DrawingTooltip component so that it now properly shows the "Click to add a point" text when hover over a selected line/measure feature.
Fixing the issue:
If the user shares the link from the 2D view, you can easily switch between 2D and 3D views after opening the link. However, if it shares the link from the 3D view, for example with a predefined view, and then switches to the 2D view, the WMS layers are not displayed. You first have to reload the map.
Issues :
- We were sending 3 pairs of Coordinates to the format function while in 3D, and the function could only handle two
- The is3DReady flag was sometimes ready before the viewer could be accessed, leading to a situation where the handler could not be set up

Fixes :
- We're splitting the coordinates and adding the rounded height at the end of it. (the height is optional IMO)
- We set a timeOut function which call itself while the viewer is not ready to ensure the viewer is ready before we set the handler.

meh

PB-1595: ensuring reactivity works for 3D

Issue: When going from 3d to 2d, then back to 3d, the `isViewerReady` flag stayed at true the whole time, stopping other components from watching its state. This could mean some components would see the flag as true when the viewer was not ready.
Fix: We now set the flag to false when destroying the 3d viewer. We also removed the timeouts in the mouse position plugin.

PB-1595: show webMercator labels

Issue : the webMercator labels were not displayed
Fix : we ensure the function display all labels
PB-1606: Tooltip doesn't translate and needs some padding
We will move the generation of margin in the viewer, instead of dealing with that on the backend. This is because it's way easier to keep the "true-to-scale" when shrinking the map viewport (and let it recalculate everything) instead of resizing the flatten map and thus changing the scale of the image.

Using CSS padding instead of margin, so that we respect the A4 size we previously calculated (if we use margin, it will extend the "page" size and break the A4 layout)
Fixed issue where under certain circumstances, a duplicate of a time layer with different years have trouble being compared.
- Issue : We want to remove the UI from the embed view in some cases, so when we are in käze, we can use the sites native UI.
- Fix : We add an URL parameter whose role it is to remove these buttons.
Issue : Whenever we loaded external layers with parameters in the base URL, we sometimes had issues where the base Url of the get Capabilities and the base URL of the layer were not the same, thus blocking the equality check that allowed the layer to update. This caused the layer to never show that it was ready, even when it was.
Fix : When we update the external layer through its capabilities, if there was a baseUrl before, we set the baseUrl of the `updated` layer to the original one, ensuring the layer will be updated.
… ensure WMS supporting such a syntax are correctly supported in our viewer.
pakb and others added 17 commits May 7, 2025 17:22
highlight current segment multi line profile
and not on the whole profile feature, when there's more than one segment
…roviders

PB-1671: removing invalid URLs from external providers whitelist
…nimation

PB-1493: add a backdrop and an animation to "Zoom with CTRL" message
Looks like some recent changes (tailwind?) have broken the now old Vue2 component that was handling the collapse animation for the menu.

There’s now alternative made in Vue3, we should migrate
The current setup wasn't working, so I added a new "mode" to Vite to differentiate when we run Cypress or when we develop.

This is due to some test failing if the devtools are present (menuTray.cy.js is one example)
As pointed out by some of our users, the dynamically generated coordinates on the bottom right part of the map with the mouse are not precise when dealing with LV03.
They are generated without taking into consideration the real deformation between LV95 and LV03, and are calculated based on a simple matrix reprojection to be more dynamic (we are not using the REFRAME service after each mouse move, that would be overkill).
…use-track

PB-1670: clarify lack of precision of mouse tracked LV03 coordinates
The geolocation button has a "linking background" with the compass button (which isn't visible when pointing north).

Since modularization, this background is swelling under the button

also fixing an issue with the "share link" input, that was not expending as intended since the tooltip modularization.

I also took the opportunity to only color the border of the input when a warning is shown, and moved the tooltip with the warning text to the copy button
PB-1674: Fix tooltip creating too much height with map buttons
@github-actions github-actions bot changed the title new version New Release v1.55.0 - #minor May 9, 2025
@cypress
Copy link

cypress bot commented May 9, 2025

web-mapviewer    Run #5292

Run Properties:  status check passed Passed #5292  •  git commit 4df94c1544: Merge pull request #1334 from geoadmin/feat-PB-1579-local-bigtiff-support
Project web-mapviewer
Branch Review develop
Run status status check passed Passed #5292
Run duration 06m 23s
Commit git commit 4df94c1544: Merge pull request #1334 from geoadmin/feat-PB-1579-local-bigtiff-support
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 20
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 251
View all changes introduced in this branch ↗︎

pakb added 4 commits May 12, 2025 08:49
adding some retry-ability to the tests so that it behaves better on the CI.
we were missing the little endian signature for BigTIFF (which is different than normal TIFFs) to be able to parse a local BigTIFF file.

It was working fine with URLs already because we do not test against little endian signature in this case, and let GeoTIFF.js library do its magic, so nothing to change there.
@pakb pakb requested review from ismailsunni, schtibe and sommerfe May 12, 2025 09:39
@pakb pakb merged commit 1dedf4d into master May 12, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants