PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Configuration toggles

Ouverte
#3,989 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

enhancement
Langage dominant
Lua
Étoiles
5.4k
Forks
2.4k
Merge moyen
1 j 12 h
PR mergées (30 j)
26

Description

Check for duplicates
  • I've checked for duplicate issues by using the search function of the issue tracker
Is your feature request related to a problem?

This is related to the new ascendancy related jewels so i don't know if any developer is already solving the problem.

The first problem is being unable to pick nodes from another ascendancy which is understandable.

The second problem is related to Configuration toggles:
eg. I pick the node on the Slayer "Bane of Legends" which adds a "Have you killed recently" toggle, when i remove the node the toggle disappears, i have no way to see that and similar toggles if the nodes are disabled.
However if i go to Slayer, select the nodes, enable the toggles, and then go to another ascendancy the the now hidden toggle remains toggled because it shows the dps increase.

Describe the solution you'd like

Showing along the dps (if any) the dps when related configurations are toggled, see example (on which the node is toggled otherwise i wouldn't see the green number)
toggle

Describe alternatives you've considered

Having a show all button on the configuration page, not sure how feasible.

Additional context

No response

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez sur la page Configuration et reproduisez le scénario de Slayer « Bane of Legends » décrit dans l’issue : activez son toggle « Have you killed recently », puis changez d’ascendancy et observez le toggle masqué ainsi que l’affichage des DPS. C’est terminé lorsque les toggles de configuration masqués restent accessibles ou que leur impact sur les DPS est clairement affiché, et que l’alternative proposée « show all » a été évaluée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
lua
Domaine
desktop
Type d'issue
Fonctionnalité
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.