Idées en vue d'une refactorisation #275
elias75015
started this conversation in
Ideas
Replies: 3 comments
-
|
Beta Was this translation helpful? Give feedback.
0 replies
-
Link : #39 |
Beta Was this translation helpful? Give feedback.
0 replies
-
Centraliser un maximum de code commun à tous les widgets héritant de ControlExtended dans cette dernière : |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Dans ce ticket chapeau, l'idée dans un premier temps est de lister ce qui nous parait améliorable dans le code pour permettre une homogénéisation de la manière de créer un widget.
Dans un second temps, on essayera de décliner les différentes remontées listées ici en sous-tickets au périmètre mieux défini.
A terme, l'objectif est de proposer un protocole de contribution basé sur un workflow bien documenté et éventuellement des templates.
Idées Générales
Créer un "Core" pour la création du DOM : une bibliothèque de fonctions dans laquelle un contributeur pourrait piocher pour créer son widget.
Faire la distinction entre plusieurs types de widgets : widget avec formulaire (ex.import de donnéees), widget avec interaction (ex.calcul de distance)
Homogénéiser les widgets fullCustom et les widgets natifs OpenLayers qu'on surcharge
Incohérences entre widgets
les options du widgets sont passées à des endroits différents
des options similaires entre deux widgets ont des nom différents
la manière d'ajouter des classCSS aux éléments (parfois classList.add(), parfois classList = ".....")
Factorisations possibles
le constructeur des widgets est souvent surchargé avec les mêmes éléments
les setters/getter du DOM (ex. setCollapsed, getContainer)
pour les widgets de calcul/dessin, les méthodes pour récupérer la données (set/get data, set/getgeometry)
la création du DOM et des listeners sur les boutons (ex. _createShowIsoExclusionsPictoElement = _createShowElevationPathPictoElement = ....)
Beta Was this translation helpful? Give feedback.
All reactions