Skip to content

Conversation

@Tankonyako
Copy link

SKIN.text_dark does not hook with custom skins.

There was a bug in which this did not work, the entire text was black by default, on different topics. I have a black derma theme, in which this variable is replaced, but it does not work.
Screenshot (fixed): https://i.imgur.com/hNnujQ8.png

…ext_dark.

There was a bug in which this did not work, the entire text was black by default, on different topics. I have a black derma theme, in which this variable is replaced, but it does not work.
Screenshot (fixed):  https://i.imgur.com/hNnujQ8.png
pnl.TextArea.selectAll = t.selectall
pnl.Label:SizeToContents()
pnl:SetZPos( t.zpos or 0 )
if t.skin then pnl.Label:SetSkin( t.skin ) end
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
if t.skin then pnl.Label:SetSkin( t.skin ) end
if t.skin then pnl:SetSkin( t.skin ) end

Applying a skin should take effect on the whole slider. This should also set the skin of the label, right? (it's been a while for me lol)

@SticklyMan
Copy link
Member

Would you mind linking the SKIN you're using to test this? The ones I have for testing don't seem to handle text_dark correctly anyways, and I can't find any real great documentation for derma color conventions either.

@Tankonyako
Copy link
Author

Tankonyako commented Jul 25, 2021

Would you mind linking the SKIN you're using to test this? The ones I have for testing don't seem to handle text_dark correctly anyways, and I can't find any real great documentation for derma color conventions either.

https://steamcommunity.com/sharedfiles/filedetails/?id=1597409940

@zpetty33
Copy link
Contributor

zpetty33 commented Aug 5, 2022

@SticklyMan - When you get the chance, can you follow up?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants