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

Fix #5088: Revert ""Fix #4710 the button scaling problem in largest text, and change button name"" #5090

Merged
merged 1 commit into from
Jul 14, 2023

Conversation

adhiamboperes
Copy link
Collaborator

@adhiamboperes adhiamboperes commented Jul 14, 2023

Fixes #5088.
Reverts #4987 which removed styles and added changes that broke the dark mode support previously correctly implmented in #4796.

This is a clean revert so it can be merged directly.

Before - Light Mode Before - Dark Mode
After - Light Mode After - Dark Mode
Before - Dark Mode Landscape After - Dark Mode Landscape
Before - Dark Mode Tab After - Dark Mode Tab

@adhiamboperes adhiamboperes requested a review from a team as a code owner July 14, 2023 16:38
Copy link
Member

@seanlip seanlip left a comment

Choose a reason for hiding this comment

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

LGTM, thanks. Since this is a clean revert, I am merging it directly.

@seanlip seanlip merged commit 12ded96 into develop Jul 14, 2023
@seanlip seanlip deleted the revert-4987-#4710 branch July 14, 2023 18:07
@MohitGupta121
Copy link
Member

@adhiamboperes so can I close this PR #5089 as now this issue is fixed by reverting PR ?

@adhiamboperes
Copy link
Collaborator Author

@MohitGupta121 yes please. Thank you.

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.

[BUG]: Buttons in Resume lesson dark mode
3 participants