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

Add dark-mode supported sponsor logos on Github #5492

Closed
1 task done
tyler36 opened this issue Nov 2, 2023 · 13 comments · Fixed by #5501
Closed
1 task done

Add dark-mode supported sponsor logos on Github #5492

tyler36 opened this issue Nov 2, 2023 · 13 comments · Fixed by #5501

Comments

@tyler36
Copy link
Collaborator

tyler36 commented Nov 2, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Output of ddev debug test

Currently, some sponsor logos our Github homepage are barely readable or look strange.

Expected Behavior

Highlight our wonderful sponsors.

image

(image from ddev.com)

Actual Behavior

image

Steps To Reproduce

Visit: https://github.com/ddev/ddev#wonderful-sponsors

Anything else?

There were recent updates to https://ddev.com/ that fixed the issue there. We should be able to re-use those images.

@tyler36
Copy link
Collaborator Author

tyler36 commented Nov 2, 2023

@Firesphere pointed it out the issue on Discord .

@rfay
Copy link
Member

rfay commented Nov 2, 2023

@bmartinez287 want to take a look?

@bmartinez287
Copy link
Collaborator

I will look into this. https://www.stefanjudis.com/notes/how-to-define-dark-light-mode-images-in-github-markdown/
the picture element should work nicely here.

@rpkoller
Copy link
Collaborator

rpkoller commented Nov 2, 2023

+1 for opening the issue as well as the suggestion about the use of the picture element by @bmartinez287 . but the only detail i wonder about would it make sense to ask all the sponsors if they have variants of their logo for dark backgrounds (and in case they dont have if they are ok with the suggested inversion of colors done in the expected behavior screenshot). but some of the logos there couldnt be inverted and therefore fail WCAG SC 1.4.11 like agaric, the parenthesis of tag1, the heart of undpaul, the background square for centaros logo to name a few details.

@bmartinez287
Copy link
Collaborator

bmartinez287 commented Nov 2, 2023

We could ask them after we replace them. In some cases, their reply might just be "it looks fine". Some logos prove a unique challenge since they are a single icon. But in most cases altering the SVG text seems to make it ally compatible.

We could ask them now, but this might be low in their priorities list so they might say yes but give us no timeframe for it.

@rpkoller
Copy link
Collaborator

rpkoller commented Nov 2, 2023

yes i agree but i am just worried that some of them might have strict rules in regards of their corporate identity and changing colors in logos might alter that corporate identity therefore i think give them a headsup early on might be a good idea in any way.

@rfay
Copy link
Member

rfay commented Nov 2, 2023

The real problem here is that I'm going to be unwilling to bother them :)

@bmartinez287
Copy link
Collaborator

bmartinez287 commented Nov 3, 2023

b3c1539
I'm not sure where the PR above hid those items. Searching for them does not bring them up in the repo. I can find the images folder where they are stored in the repo but not the definition for where they are placed to add the picture html tag.
Is ![DDEV featured sponsor logos] hidden behind some secure setting I need access to? @rfay

@rfay
Copy link
Member

rfay commented Nov 3, 2023

Those things come from ddev.com, 100% open source.

@bmartinez287
Copy link
Collaborator

bmartinez287 commented Nov 3, 2023

I guess, I never seen the ...... ![something] syntax on markup. What is that called?

@rfay
Copy link
Member

rfay commented Nov 3, 2023

In markdown, that's how you insert an image.

@bmartinez287
Copy link
Collaborator

Screenshot 2023-11-08 at 1 11 50 PM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants