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

Colorize and group defensive calc sections #5753

Merged
merged 2 commits into from
Mar 31, 2023

Conversation

deathbeam
Copy link
Contributor

@deathbeam deathbeam commented Mar 4, 2023

There is quite a lot of calc sections now and colours make them easy to quickly recognize from each other.

Also group them properly together as "primary", "secondary" and "misc" defenses.

Description of the problem being solved:

Some of the secionts are split to weird places, for example block and dodge/supp and other damage avoidance is nowhere eachother. And there is a lot of columns currently and sometimes its hard to visually distinguish one from another.

Link to a build that showcases this PR:

Just completely empty pob already showcases it.

Before screenshot:

image

After screenshot:

image

@deathbeam deathbeam force-pushed the colors branch 2 times, most recently from f6e949d to 85fa4ef Compare March 4, 2023 23:00
There is quite a lot of calc sections now and colours make them easy to
quickly recognize from each other.

Also group them properly together as "primary", "secondary" and "misc"
defenses.

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
@QuickStick123 QuickStick123 added the user-interface Changes that only affect the UI label Mar 4, 2023
@LocalIdentity LocalIdentity merged commit f49c806 into PathOfBuildingCommunity:dev Mar 31, 2023
Dullson pushed a commit to Dullson/PathOfBuilding that referenced this pull request Dec 6, 2023
)

There is quite a lot of calc sections now and colours make them easy to
quickly recognize from each other.

Also group them properly together as "primary", "secondary" and "misc"
defenses.

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
Co-authored-by: LocalIdentity <31035929+LocalIdentity@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
user-interface Changes that only affect the UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants