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.

Sign up for GitHub

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

Jump to bottom

button tinting is broken on status theme / custom tinting does not apply on first load #7525

Closed
ara4n opened this issue Oct 20, 2018 · 5 comments
Closed

button tinting is broken on status theme / custom tinting does not apply on first load #7525

ara4n opened this issue Oct 20, 2018 · 5 comments
Labels
A-Theming P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround S-Tolerable Low/no impact on users T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented Oct 20, 2018

No description provided.

@MTRNord
Copy link
Contributor

MTRNord commented Oct 20, 2018

Switching to light and back fixes it again

@MTRNord
Copy link
Contributor

MTRNord commented Oct 20, 2018

grafik

This was the result

@lampholder
Copy link
Member

Ahboo, the fix for #6424 had a side-effect maybe?

@lampholder lampholder added T-Defect P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround S-Tolerable Low/no impact on users A-Theming labels Oct 23, 2018
@turt2live turt2live changed the title button tinting is broken on status theme on 0.17.2, at least after you hit the upgrade button button tinting is broken on status theme / custom tinting does not apply on first load Oct 29, 2018
@turt2live
Copy link
Member

Have updated the title. This applies to account-wide tinting too if you land on the homepage rather than a room. Looks like it's an edge case that wasn't fixed by matrix-org/matrix-react-sdk#2214

@turt2live turt2live self-assigned this Nov 8, 2018
@turt2live turt2live removed their assignment Nov 8, 2018
@turt2live
Copy link
Member

This is going to be implicitly fixed by the redesign.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Theming P1 S-Major Severely degrades major functionality or product features, with no satisfactory workaround S-Tolerable Low/no impact on users T-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@turt2live @ara4n @MTRNord @lampholder