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 the undo/redo button visibility on mobile #92

Closed
wants to merge 22 commits into from

Conversation

echoriiku
Copy link
Collaborator

@echoriiku echoriiku commented Jul 30, 2024

The colors of the undo/redo buttons were similar to the background, making it hard to see, specially on mobile. This changes make it a lot more readable.

Before

Screenshot_20240730-162546-103.png

After

Screenshot_20240730-162617-804.png

@echoriiku echoriiku marked this pull request as ready for review July 30, 2024 18:24
Copy link

@thecodrr
Copy link
Contributor

thecodrr commented Aug 1, 2024

I don't think this change should be necessary. The issue is in the UI itself. The color for disabled buttons should be derived from the color of the button instead of being based on the border color.

@echoriiku
Copy link
Collaborator Author

echoriiku commented Aug 1, 2024

Thanks for the feedback. I'm going to close this PR then.

@echoriiku echoriiku closed this Aug 1, 2024
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.

2 participants