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

front: editor: changing layers requires confirmation #5593

Closed
eckter opened this issue Nov 3, 2023 · 0 comments · Fixed by #5607
Closed

front: editor: changing layers requires confirmation #5593

eckter opened this issue Nov 3, 2023 · 0 comments · Fixed by #5607
Assignees
Labels
area:front Work on Standard OSRD Interface modules kind:bug Something isn't working module:infra-editor Infra Edition severity:minor Minor severity bug

Comments

@eckter
Copy link
Contributor

eckter commented Nov 3, 2023

What happened?

In the infra edition, toggling layers requires confirmation before exiting the window. This is inconsistent with the rest of the app, as well as the map layer in the same tab (which doesn't require confirmation).

What did you expect to happen?

Toggling a layer should be effective when exiting the window (or immediately) with no extra click

How can we reproduce it (as minimally and precisely as possible)?

Toggle a layer in the infra edition module and exit the window

What operating system, browser and environment are you using?

  • Browser: Firefox
  • OS: Ubuntu
  • Env: dev

OSRD version (top right corner Account button > Informations)

c39e699

@eckter eckter added kind:bug Something isn't working area:front Work on Standard OSRD Interface modules severity:minor Minor severity bug module:infra-editor Infra Edition labels Nov 3, 2023
@SarahBellaha SarahBellaha self-assigned this Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:front Work on Standard OSRD Interface modules kind:bug Something isn't working module:infra-editor Infra Edition severity:minor Minor severity bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants