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
[Dark Mode] Fix text color in Aztec - HTML text view #12462
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.
Going off of the included screenshot, these colors look fine to me. 👍
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.
Works great! 🎉
Tested via wordpress-mobile/AztecEditor-iOS#1209
Podfile.lock
Outdated
@@ -3,7 +3,7 @@ PODS: | |||
- Alamofire (4.7.3) | |||
- AlamofireNetworkActivityIndicator (2.3.0): | |||
- Alamofire (~> 4.7) | |||
- Automattic-Tracks-iOS (0.4.0): | |||
- Automattic-Tracks-iOS (0.4.1): |
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.
I don't understand these changes on Podfile.lock 🤔
Maybe they are outdated on the release branch?
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.
Podfile.lock should be ok now!
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! 🎉
Refs. #12320
This PR set the color to the HTML textView in the Aztec editor.
I'll update the Podfile as soon Aztec #1209 will be merged
Adding also @mattmiklic to set the right colors used for the comments and the tags.
To test:
• Run the branch with Xcode 11 and iOS 13
• Edit a post and switch between visual and HTML mode
• Check the html text color is not black in Dark Mode
• Run it on iOS 12 to check everything works fine
Update release notes:
RELEASE-NOTES.txt
if necessary.