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

JS: Enhancing digitizing component and module #4594

Merged
merged 5 commits into from
Jul 15, 2024

Conversation

rldhont
Copy link
Collaborator

@rldhont rldhont commented Jul 10, 2024

  • Add erase all button and method
  • Removed from localstorage when saved is toggled
  • Add a measure attribute to enable or not the measure button
  • Fix locale key confirm erase by replacing confirme by confirm

@rldhont rldhont added user interface javascript Pull requests that update Javascript code labels Jul 10, 2024
@github-actions github-actions bot added this to the 3.9.0 milestone Jul 10, 2024
Copy link
Member

@Gustry Gustry left a comment

Choose a reason for hiding this comment

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

If all buttons have a nice UI (tooltip, CSS, position, icon), LGTM

assets/src/components/Digitizing.js Outdated Show resolved Hide resolved
@rldhont rldhont force-pushed the fix-digitizing branch 2 times, most recently from 6e2837c to de4b653 Compare July 15, 2024 13:51
@rldhont rldhont marked this pull request as ready for review July 15, 2024 13:53
@rldhont rldhont added the run end2end If the PR must run end2end tests or not label Jul 15, 2024
@rldhont
Copy link
Collaborator Author

rldhont commented Jul 15, 2024

@Gustry @nboisteault it is ready for review

rldhont added 5 commits July 15, 2024 16:39
To enable measure with digitizing component, the attribute measure has to be added.
Add a new button to erase all digitizing features in the digitizing component, and the method in the digitizing module.
Replace `digitizing.confirme.erase` by `digitizing.confirm.erase`
Now when the saved is toggle the browser localstorage is clear.
When the user click on digitizing saved button, a message is displayed to explain that the drawing changes wil be saved in the browser, and the digitizing save title changed to explain that the drawing will be removed from the browser.
* Test erase all
* Check toggle save
@rldhont rldhont merged commit 8dcb68a into 3liz:master Jul 15, 2024
13 checks passed
@rldhont rldhont deleted the fix-digitizing branch July 15, 2024 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport release_3_8 javascript Pull requests that update Javascript code run end2end If the PR must run end2end tests or not user interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants