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.

Sign up for GitHub

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

Jump to bottom

#218 - Template - Composant libre #333

Merged
merged 3 commits into from
Nov 28, 2024
Merged

#218 - Template - Composant libre #333

merged 3 commits into from
Nov 28, 2024

Conversation

Gaetanbrl
Copy link
Contributor

@Gaetanbrl Gaetanbrl commented Nov 28, 2024 •
edited
Loading

Ref. #218

Cette Pull request ajoute un composant libre permettant de saisir le contenu du HTML sans assistance :

image

NE PAS MERGER !

RAF :

  • Dépend de la correction de l'affichage du template via l'URL du mst (issue indépendante)
  • charger la valeur saisie lors de la modification du template existant
  • Afficher un composant libre
  • Prévisualisation du composant libre
  • Test dans mviewer
  • prettier
  • documentation

@Gaetanbrl Gaetanbrl mentioned this pull request Nov 28, 2024
Closed
@Gaetanbrl
fix get active layer by dom query
07833e5 
roll back id retrieve method

pretty code

add free component and adapt

Display free component

pretty code

add free component doc ref
@Gaetanbrl
fix mviewer#334 with default featurecount number
82b0f6e
@Gaetanbrl
Free component - read and write
da333f5 
remove useless code
@lecault
Copy link
Contributor

lecault commented Nov 28, 2024

OK pour la limite des 5.
Ok pour template libre.

J'ai relevé un souci, voici le scénario :

  • Je charge une couche et je met un template personnalisé
  • Je prévisualise
  • Quand j'interroge, ça ne fonctionne pas (même erreur qu'avant)

@lecault lecault added this to the 4.2 milestone Nov 28, 2024
@Gaetanbrl
Copy link
Contributor Author

  • Quand j'interroge, ça ne fonctionne pas (même erreur qu'avant)

A tester avec une MAJ mviewer (git pull sur branche develop)

@lecault
Copy link
Contributor

lecault commented Nov 28, 2024

Fonctionne avec le git pull

@lecault
Copy link
Contributor

lecault commented Nov 28, 2024

Ok pour merge

@Gaetanbrl Gaetanbrl merged commit a97dbbe into mviewer:develop Nov 28, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
4.2
Development

Successfully merging this pull request may close these issues.
2 participants
@Gaetanbrl @lecault