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

add Colorblind tree search highlight support #6866

Merged
merged 3 commits into from
Dec 1, 2023

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
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