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

feat(draw): new features, improvements and bug fixes #897

Merged
merged 12 commits into from
Sep 10, 2021

Conversation

LAMM26
Copy link
Contributor

@LAMM26 LAMM26 commented Aug 17, 2021

Please check if the PR fulfills these requirements

What is the current behavior? (You can also link to an open issue here)
infra-geo-ouverte/igo2#655
infra-geo-ouverte/igo2#649
infra-geo-ouverte/igo2#646

What is the new behavior?
Issue 655:

  • L'attribut id correspond à l'étiquette lorsque présente, correspond à l'id (simplifié) sinon.

Issue 649:

  • Les sélecteurs de couleur ont été déplacés vers la gauche.
  • Un délai de 250 ms a été ajouté avant l'apparition de la boîte de dialogue permettant de saisir une étiquette.
  • Un bouton "Aucune étiquette" a été ajouté à la boîte de dialogue de saisie d'étiquette.
  • La couleur de contour par défaut a été modifiée à rouge afin de distinguer une entité sélectionnée plus facilement.
  • Les contours des entités dessinées ont été amincies afin de rendre une entité sélectionnée plus visible.
  • Il est maintenant possible d'exporter les entités de type Cercle en polygones dans l'outil d'exportation.
  • Ajout de raccourcis clavier lors du dessin (Esc ou c pour annuler le dessin, Backspace ou u pour enlever le dernier vertex, Enter ou f pour terminer le dessin).
  • Autre modifications de l'interface dans l'outil Dessin.

Issue 646:

  • Une entité peut être modifiée en déplaçant des arêtes ou des vertex.
  • L'étiquette d'une entité peut être modifiée en double cliquant sur une entité et en entrant une nouvelle étiquette.

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications:

Other information:

Copy link
Member

@pelord pelord left a comment

Choose a reason for hiding this comment

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

General comment :
1- Split your differents modification into commit. issueX = 1 or more commit. Your PR is too broad and it is hard to review.
2- Ensure the content is mobile friendly (
image
)
3- This is not a final review. A new review will be needed aftherward.
4- Ensure to let the draw layer been exported.. IGO raise an issue..

@pelord
Copy link
Member

pelord commented Aug 17, 2021

@LAMM26 Good job!!

@pelord
Copy link
Member

pelord commented Aug 17, 2021

@LAMM26 One more thing, when modifying the circle, show the result of th drag..... not only a the end of the drag

@LAMM26 LAMM26 closed this Aug 17, 2021
@LAMM26 LAMM26 reopened this Aug 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants