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 blockly comment text color #2179

Merged
merged 1 commit into from Apr 28, 2020
Merged

Conversation

BeksOmega
Copy link
Collaborator

Description

Currently blockly comments have the colors background: #FFC and font: #555. Technically these colors pass accessibility tests, but it can still be hard to read.

By default blockly comments are supposed to have a font color of #000. AI's comments did not because the designer css was overflowing into the Blockly workspace.

This PR changes the comment text color back to the default #000.

Testing

BetterCSS
(open the image in a new tab for a better reference).

I also inspected the textarea element to ensure the font color was being applied.

Additional Information

This was originally discussed in the comments of this forum post.

Copy link
Member

@ewpatton ewpatton left a comment

Choose a reason for hiding this comment

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

LGTM

@ewpatton ewpatton merged commit 8b6e181 into mit-cml:master Apr 28, 2020
@BeksOmega BeksOmega deleted the fix/comment-color branch June 10, 2020 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants