PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding

Configuration toggles

オープン
#3,989 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

enhancement
主要言語
Lua
スター
5.4k
フォーク
2.4k
平均マージ
1日 12時間
マージ済み PR(30日)
26

説明

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

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

Configurationページから開始し、issueで説明されているSlayerの「Bane of Legends」シナリオを再現します。まず「Have you killed recently」トグルを有効にし、その後ascendancyを切り替えて、非表示のトグルとDPS表示を確認します。非表示の設定トグルを引き続き見つけられるようにするか、そのDPSへの影響を明確に表示し、提案されている「show all」案を評価できれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
lua
領域
desktop
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。