Skip to content

Conversation

@admSla
Copy link
Contributor

@admSla admSla commented Nov 13, 2023

Fixes # .

Description of the problem being solved:

This PR solve the isssue "Colorblind tree search highlight support". The functionality was done according to PR #6070.

Steps taken to verify a working solution:

-create edit and tooltip labels for user input
-create function inside of labels which check hex code and then execute function updateColor
-update function updateColor from which I trigger own created function which convert Hex color code to RGB
-save RGB code to array and change parameters for drawing color function in PassiveTreeView.lua

Link to a build that showcases this PR:

Before screenshot:

image

image

After screenshot:

image

image

@LocalIdentity LocalIdentity added the enhancement New feature, calculation, or mod label Nov 13, 2023
Copy link
Contributor

@LocalIdentity LocalIdentity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Include a picture of what the config looks like in the client

@admSla
Copy link
Contributor Author

admSla commented Nov 14, 2023

Hi, I add screenshots of config tab in options and remove VS config from Launch.lua

@admSla admSla requested a review from LocalIdentity November 14, 2023 22:53
Copy link
Contributor

@LocalIdentity LocalIdentity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

everything looks good and conforms to the previous standard we use for the other colour change controls

end

function hexToRGB(hex)
hex = hex:gsub("0x", "") -- Remove "0x" prefix
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding comments on what the regex sub is meant to do

@LocalIdentity LocalIdentity merged commit 51a28eb into PathOfBuildingCommunity:dev Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature, calculation, or mod

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants