Conversation
|
A few more syntax highlighting colors that fail right now:
Any suggestions welcome. |
|
Darkening the colors should be ok, I just did that in 200b0c0e80c70d7da4b686f08cc6d048608381ae |
|
@MartijnCuppens: not quite there yet :P The ratio seems to be 4.49 for I personally use the https://squizlabs.github.io/HTML_CodeSniffer/ bookmarklet. |
200b0c0 to
b812e01
Compare
|
I used Chrome's built-in colorpicker which can behave a little buggy it seems. Changed the values now |
|
I'll go through the pages tomorrow and see what else is left. I'm still not sure how to proceed with the search form. |
|
@patrickhlauke: friendly ping :) |
afc91e6 to
7f00b6e
Compare
|
Alright, here's the list of the remaining syntax highlighting related issues:
They seem a little tricky since they are dark already. Maybe the suggestions are wrong, so we need to be careful. |
ad424c3 to
1f827f6
Compare
|
I think this can be reviewed. We haven't heard from @patrickhlauke, so I guess we'll tackle the remaining stuff later. @MartijnCuppens I changed the colors you changed, let me know if you agree with the new ones |
MartijnCuppens
left a comment
There was a problem hiding this comment.
We better define the contrast color in theme-colors.yml instead of checking if the name is light or warning. Not sure if that should be tackled in this PR. If not, this LGTM
|
How do you mean exactly? I think it would be better if we tackled it in a separate PR later, just don't forget about it. For the time being, I just want another approval from @mdo or @ysds or @patrickhlauke |
80c687c to
a56d6ba
Compare
|
Ping @mdo for review |
866c2ec to
cc99303
Compare
|
This looks good from my tests so far. We need to double check these though: |
|
few more bits of low contrast i stumbled across while looking over https://deploy-preview-29208--twbs-bootstrap.netlify.com/ (some of these part of our core BS styles though, so not specifically documentation i guess) |
|
the search form is probably tricky, as it's not really a form as such that you submit, but rather that you type in and get your results right away/supposed to select from the dropdown...bit of a quirky one. i'd note though that there as well though, contrast starts to get quite low (around 4.3:1) for gray text on pink background. and the pink highlight color itself is barely distinguishable from the non-selected background |
|
Yeah this doesn't fix all of the issues. I'll try to tackle the rest in
separate PRs.
…On Mon, Sep 16, 2019, 22:27 Patrick H. Lauke ***@***.***> wrote:
few more bits of low contrast i stumbled across while looking over
https://deploy-preview-29208--twbs-bootstrap.netlify.com/ (some of these
part of our core BS styles though, so not specifically documentation i
guess)
[image: non-text-content-low-contrast]
<https://user-images.githubusercontent.com/895831/64986759-a596fe00-d8bf-11e9-9a66-5a486a2efbad.png>
[image: 2to1]
<https://user-images.githubusercontent.com/895831/64986760-a596fe00-d8bf-11e9-9edc-74676d483be3.png>
[image: 3 4to1-4to1]
<https://user-images.githubusercontent.com/895831/64986761-a62f9480-d8bf-11e9-9de3-c0a694adaefd.png>
[image: 4 4to1]
<https://user-images.githubusercontent.com/895831/64986762-a62f9480-d8bf-11e9-9bf1-bc7cf529584e.png>
[image: 4 4to1-2]
<https://user-images.githubusercontent.com/895831/64986763-a62f9480-d8bf-11e9-81ba-eb91fee4b364.png>
[image: 4to1]
<https://user-images.githubusercontent.com/895831/64986764-a62f9480-d8bf-11e9-8fc9-6db292b71434.png>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#29208?email_source=notifications&email_token=AACVLNKIQNJLI6ESWTD2RCLQJ7M3RA5CNFSM4IJLDGOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD62HO6Q#issuecomment-531920762>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AACVLNPYHBQOJ6TEF2K2LF3QJ7M3RANCNFSM4IJLDGOA>
.
|







IMO we should backport these.
@patrickhlauke: there are probably more color and general accessibility issues, but I try to solve the issues in batches. There's one thing that bugs me:
For the search form.
Preview: https://deploy-preview-29208--twbs-bootstrap.netlify.com/