Skip to content

Don't ClassColor names less than 3 characters#798

Open
eTzmNcbkrng wants to merge 4 commits intoElvUI-WotLK:masterfrom
eTzmNcbkrng:ChatClassColorFix
Open

Don't ClassColor names less than 3 characters#798
eTzmNcbkrng wants to merge 4 commits intoElvUI-WotLK:masterfrom
eTzmNcbkrng:ChatClassColorFix

Conversation

@eTzmNcbkrng
Copy link

I've noticed a growing trend of people creating characters with names only two characters long.
This commit prevents words like he, it, to, my, no, etc. being coloured as the class of the player that used them.

@eTzmNcbkrng eTzmNcbkrng force-pushed the ChatClassColorFix branch from 30c903a to 4dccfee Compare May 5, 2021 07:43
@eTzmNcbkrng eTzmNcbkrng force-pushed the ChatClassColorFix branch from 4dccfee to 779ebd4 Compare May 7, 2021 09:24
@Apollyonn
Copy link
Contributor

I don't think thats necessary because you can already exclude names that you don't want to be colored in the elvui setrtings.
WoWScrnShot_061321_133443

@eTzmNcbkrng
Copy link
Author

Yeah I saw that, I was constantly adding players to it though.

@Barsoomx
Copy link

Yeah super annoying, would love to see that merged

@Apollyonn
Copy link
Contributor

In order to accept this, you have to change it a bit and add the ability to completly disable it. For example add a slider to chat settings and modify the code to look something like this. If the slider is set to 0 it will be disabled. I think this is the most ideal way

if wordMatch and (E.global.chat.classColorMentionLetterThreshold > 0 and strlen(lowerCaseWord) >= E.global.chat.classColorMentionLetterThreshold) and not E.global.chat.classColorMentionExcludedNames[wordMatch] then local classColorTable = CUSTOM_CLASS_COLORS and CUSTOM_CLASS_COLORS[classMatch] or RAID_CLASS_COLORS[classMatch] end

@eTzmNcbkrng
Copy link
Author

image

@eTzmNcbkrng
Copy link
Author

In order to accept this, you have to change it a bit and add the ability to completly disable it.

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants