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

New Release v1.28.0 - #minor #931

Merged
merged 29 commits into from
Jun 18, 2024
Merged

New Release v1.28.0 - #minor #931

merged 29 commits into from
Jun 18, 2024

Conversation

ltshb
Copy link
Contributor

@ltshb ltshb commented Jun 18, 2024

ltshb and others added 29 commits June 17, 2024 13:15
Since this PR is merged on develop those test are often not passing because
it seems that when trying to open the menu settings a second time it opened
the give feedback view somehow.
Try solve test flakiness introduced by #922
The roadmap was outdated and we don't want to manage the roadmap here anymore.

Same for ADR we will put our ADR in confluence in the future.
When resizing the window from small to big the elevation profile was not re-render
properly.

See https://www.chartjs.org/docs/master/configuration/responsive.html#important-note
Some attribution are not link and cannot be clicked. In this case they should
be selectable. Previously all attributions were not selectable and they all
had the pointer cursor.
The key press is only effective when the map is in focus or the time slider
button or the timeslisder play button. This ensure that the key press doesn't
interfere with other UI component like the search input field for example.
PB-643: Implement left/right key press to move time slider
@ltshb ltshb requested a review from pakb June 18, 2024 10:15
@github-actions github-actions bot changed the title release New Release v1.28.0 - #minor Jun 18, 2024
Copy link

cypress bot commented Jun 18, 2024

Passing run #2610 ↗︎

0 208 21 0 Flakiness 0

Details:

Merge pull request #928 from geoadmin/bug-PB-643-timeslider-keyboard
Project: web-mapviewer Commit: 771869ef71
Status: Passed Duration: 04:21 💡
Started: Jun 18, 2024 10:18 AM Ended: Jun 18, 2024 10:23 AM

Review all test suite changes for PR #931 ↗︎

@ltshb ltshb merged commit 15d7e45 into master Jun 18, 2024
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.

3 participants