-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Use proper width for Cancel/Decline Button Text #4275
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, and just like the proposal!
Can you please complete the rest of the platform testing before I merge this?
@tgolen , the issue isn't happening on Android. And Sorry, I'm yet to get a macbook for other two platforms. |
OK, you still need to get an Android screenshot to show there are no regressions. I can test it out on iOS and the desktop app for you (I won't get to it until tomorrow). |
All right, I tested it on desktop, and ran into an issue. It looks like this corrupted the width of the dropdowns somehow. Here is the preferences screen on mainthis branch |
That must be the case as the removed style is not being used anywhere else. |
Added a screenshot from android. |
Well, I was able to do it today after all! iOS looks good. I verified that when I merge |
once the tests are done |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Deployed to staging in version: 1.0.81-5🚀
|
🚀 Deployed to production in version: 1.0.82-7🚀
|
Details
(Comment)
Fixed Issues
$ #4234
Tests || ### QA Steps
Tested On
Screenshots
Web
Mobile Web
Desktop
iOS
Android