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

Modifier le switch de sélection des globales dans la liste des propriétés #327

Closed
thomaslhostis opened this issue Oct 10, 2019 · 4 comments

Comments

@thomaslhostis
Copy link
Collaborator

Ajouter un switch pour les cacher

@Lucas-C
Copy link
Member

Lucas-C commented Oct 10, 2019

J'ai besoin d'un peu de clarté:

  • est-ce que tu parles de toute variable globale ?
  • des variables globales de même nom qu'une propriété de module ?
  • des variables de module valorisées par des variables globales ?

@Lucas-C
Copy link
Member

Lucas-C commented Oct 10, 2019

@JordanKergoat ton avis ? :)

@JordanKergoat
Copy link
Contributor

L'idée est de rendre la cohérence de l'interface qu'elle a perdu, à savoir :

  • Les switch ne permettent pas de faire apparaitre des propriétés, elles sont déjà présente de base, ils jouent juste le role de filtres
  • Il faut donc que le filtre de globales actuel soit inversé, par défaut toutes les propriétés sont listée, et en l'activant on ne retrouve plus que les globales
  • Il faut ajouter un switch qui permet de masquer les globales

Pourquoi cet ajout de switch ? Il faut savoir qu'historiquement ce switch de globale pas pratique à été ajouté pour permettre au plateformes utilisants énormément de globale de pouvoir les cacher pour facilité a lecture des propriété "locale du module".

A titre d'exemple certaines plateformes avaient par exemple 5 propriété local de module vs 250 propriété globale. Il était donc compliqué pour les utilisateurs de trouver les locales

La remise en cohérence des switchs de filtres leur supprimerai cette possibilité, il faut donc ajouter un switch d'un nouveau type, un switch de masquage.

Le comportement par défaut devrai toujours être d'afficher toutes les propriétés du modèle, et pouvoir en filtrer/masquer, mais devoir activé un switch pour en afficher certaine est tout simplement une horreur à mon sens, voila :)

@Lucas-C
Copy link
Member

Lucas-C commented Oct 11, 2019

Merci pour l'explication et le contexte historique !
C'est super utile, et je comprends mieux ta pertinente suggestion :)

Donc on parle bien uniquement des variables globales de même nom qu'une propriété de module ?

@thomaslhostis thomaslhostis changed the title Afficher les propriétés globales dans la liste des valorisation au niveau du module Modifier le switch de sélection des globales dans la liste des propriétés Nov 7, 2019
@Bhoye05 Bhoye05 self-assigned this Feb 13, 2020
Bhoye05 added a commit that referenced this issue Feb 14, 2020
Bhoye05 added a commit that referenced this issue Feb 14, 2020
Bhoye05 added a commit that referenced this issue Feb 14, 2020
Bhoye05 added a commit that referenced this issue Feb 18, 2020
Bhoye05 added a commit that referenced this issue Feb 18, 2020
Bhoye05 added a commit that referenced this issue Feb 19, 2020
Bhoye05 added a commit that referenced this issue Feb 19, 2020
Bhoye05 added a commit that referenced this issue Feb 19, 2020
Bhoye05 added a commit that referenced this issue Feb 19, 2020
Bhoye05 added a commit that referenced this issue Feb 19, 2020
Bhoye05 added a commit that referenced this issue Feb 19, 2020
Bhoye05 added a commit that referenced this issue Feb 19, 2020
Bhoye05 added a commit that referenced this issue Feb 19, 2020
Bhoye05 added a commit that referenced this issue Feb 25, 2020
…es propriétés

issue 341 Ok

finalisation issue_341

wip

work in progress

wip

work in wip

fix sur properties.treeFilter supprimé

Fix #327 Modifier le switch de sélection des globales dans la liste des propriétés correctons après revue

Fix #327 Modifier le switch de sélection des globales dans la liste des propriétés correctons après revue

Fix #327 Modifier le switch de sélection des globales dans la liste des propriétés correctons après revue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants