Giter VIP home page Giter VIP logo

Comments (15)

tlsaeger avatar tlsaeger commented on May 26, 2024 1

Working on those 3 points you mentioned @bomanimc

from ml5-website.

shiffman avatar shiffman commented on May 26, 2024 1

Fully support this plan!

from ml5-website.

wenqili avatar wenqili commented on May 26, 2024

I actually have run chrome dev tools(lighthouse audit) to test the accessibility, we are all good for now.
Except for the fake search bar.. will fix it during the weekend

from ml5-website.

joeyklee avatar joeyklee commented on May 26, 2024

Thanks @wenqili !

from ml5-website.

bomanimc avatar bomanimc commented on May 26, 2024

Really love this! I reran the Google Lighthouse report on our site. Here's our position overall:
Screen Shot 2021-03-07 at 11 46 59 AM

The last 2% on the Accessibility section here relates to the color contrast on a few of elements:

  • Contrast between our "Get started with ml5.js" button and background (likely the issue is contrast on hover)
  • Contrast between comments in the code snippet and the background
  • Contrast between elements in the footer and the background (likely the issue is contrast on hover)

I'm going to retitle this issue to address the color contrast issues here! May be a great option for a new contributor as well!

from ml5-website.

tlsaeger avatar tlsaeger commented on May 26, 2024

I pushed a pull request (#177) It improves on a lot of things you mentioned above. Unfortunately the purple you are using, #a256ff, which is mostly paired with white does not give such a great contrast. I left it as it is and am suggesting to think about making it a little darker side wide.
For the button on the start page I changed the hover into a color invert and for hover in the footer I used a text-decoration: underline. Changing the color on hover resulted in A) The change was really subtle and barley noticeable B) The change cause a really bad contrast. I will add screenshots in the pull-request #177

from ml5-website.

bomanimc avatar bomanimc commented on May 26, 2024

Thanks @tlsaeger for catching this issue with the purple (#a256f) on white aspect of our design and branding. I agree that, if we can find a more accessible option (i.e. a slightly darker purple), it could be sensible to change the purple site-wide.

@c-dacanay: Do you have a suggestion on what the best course of action here might be regarding these colors? Asking since I know you have a lot of design experience!

from ml5-website.

c-dacanay avatar c-dacanay commented on May 26, 2024

Thanks @tlsaeger, @bomanimc! Totally agree.

Switching our brand purple to #722ACF would pass WCAG tests at all sizes, and it's still a rather bright purple: https://webaim.org/resources/contrastchecker/?fcolor=722ACF&bcolor=FFFFFF.

If needed I can also re-create the ml5 logo with this darker purple as well!

from ml5-website.

tlsaeger avatar tlsaeger commented on May 26, 2024

Screenshot 2021-04-02 at 19 19 34

Screenshot 2021-04-02 at 19 20 21

changed the $purple in the _variables.scss to check it out. Looks quite different, but I think its something I need to get used to, having seen the old purple so much in the past. The contrast is way better!

from ml5-website.

shiffman avatar shiffman commented on May 26, 2024

This is great, thanks all!

from ml5-website.

bomanimc avatar bomanimc commented on May 26, 2024

Agreed! Great calls @c-dacanay and @tlsaeger!

I agree that it'd be great to update the logo color to match the new purple. I think I know how to do that in Sketch, but @c-dacanay if you have time to make the logo update, that'd be great (no worries if you don't though!).

@tlsaeger wanna open a PR for these changes?

from ml5-website.

bomanimc avatar bomanimc commented on May 26, 2024

That said our logo should be an SVG that we can change with a CSS fill property. I'm not sure what we're using now, but we might not actually need to make a new asset for this specific change.

from ml5-website.

tlsaeger avatar tlsaeger commented on May 26, 2024

SVG would be cool.
Maybe one more thought an the color we could also keep the original purple and make the background black. This would create a good contrast, that only fails the WCAG AAA Normal Text. A minor adjustment would get rid of that as well. I really like the old purple, if you know what I mean. @c-dacanay @bomanimc

from ml5-website.

bomanimc avatar bomanimc commented on May 26, 2024

Good point @tlsaeger! Thanks for exploring that pathway!

It seems like we have two options:

Simple Workaround: Keep our current color and change the background to black, and then fix the accessibility issues with WCAG AAA Normal Text that @tlsaeger mentioned (by the way @tlsaeger, what corrections for this did you imagine?)

More Detailed Fix: Changing our brand color to the darker color @c-dacanay. This would require some pretty quick changes to the CSS-defined colors in the code, but would also require some reworking and replacement of our logo usage across the project and the internet. We may also want to announce the update towards a more accessible logo/color scheme online, which I think would be a cool touchpoint. As there are more places we'd need to change, I think this would require a bit more work (likely on the design and community side).

Personally, I'm open to both options! The simple fix gives us time to work on other things, while the more complex one could be a way to create some excitement and some code changes that new contributors can take on.

@c-dacanay, if it's okay, I'd like to ask that you make the final decision on this one! I think you'd have the best purview as community maintainer and as a designer!

from ml5-website.

tlsaeger avatar tlsaeger commented on May 26, 2024

@bomanimc @c-dacanay Regarding the fixing of WCAG AAA Normal Text I thought we could darken the color a little. Now that I think about it, that would mean we need to change the logo etc. anyway.

Right now the contrast is only too low in two places, the Getting Started Button, which by the way does not really lead anywhere. And the text block titled »Meet Our team!« I would suggest that we might leave at as it is right now and plan for a more holistic redesign approach. Working together with some of the people of discord, like Miluanda.

What do you think about that?

from ml5-website.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.