landing-page icon indicating copy to clipboard operation
landing-page copied to clipboard

Feature: Green squares is not accesible

Open bdougie opened this issue 3 years ago • 21 comments

Type of feature

🍕 Feature

Current behavior

reported here.

The “We are more than green squares” graphic is visually cool, but is marked up by 4 distinct block elements. A screen reader would likely read that as 4 paragraphs. The graphic could be marked up by a single

containing spans to chop it up visually via CSS.

Screen Shot 2022-12-14 at 9 35 50 AM

Suggested solution

This image should convert to a custom CSS.

Additional context

No response

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

Contributing Docs

  • [X] I agree to follow this project's Contribution Docs

bdougie avatar Dec 14 '22 17:12 bdougie

.take

SimranjitKaurMaan avatar Jan 11 '23 08:01 SimranjitKaurMaan

can we modify the svg image to fit our purpose instead of custom css?

SimranjitKaurMaan avatar Jan 14 '23 15:01 SimranjitKaurMaan

can we modify the svg image to fit our purpose instead of custom CSS?

@SimranjitKaurMaan Why do you think it's necessary to modify the green-square SVG?

shamimbinnur avatar Jan 14 '23 15:01 shamimbinnur

can we modify the svg image to fit our purpose instead of custom css?

No, the scope of the issue is to replace it with css as described in the suggested solution: Screenshot 2023-01-14 at 18 00 24

If you don't know where to start with that, we can help if you ask questions!

0-vortex avatar Jan 14 '23 16:01 0-vortex

Should this be closed as not planned due to having a different design right now?

a0m0rajab avatar Jul 26 '23 12:07 a0m0rajab

Still exist here https://opensauced.pizza/teams

Screen Shot 2023-07-26 at 6 59 48 AM

bdougie avatar Jul 26 '23 14:07 bdougie

can we modify the svg image to fit our purpose instead of custom css?

No, the scope of the issue is to replace it with css as described in the suggested solution: Screenshot 2023-01-14 at 18 00 24

If you don't know where to start with that, we can help if you ask questions!

where do i start with this?

dev-phantom avatar Aug 12 '23 05:08 dev-phantom

I did not have a look at the code, but based on the suggested solution, you might start by googling SVG to CSS and see how things go for you.

a0m0rajab avatar Aug 13 '23 13:08 a0m0rajab

.take

CBID2 avatar Sep 17 '23 05:09 CBID2

Thanks for taking this on! If you have not already, join the conversation in our Discord

github-actions[bot] avatar Sep 17 '23 05:09 github-actions[bot]

.take

Sorry to ask Don't mean to sound rude

Please how do you intend to fix this?

dev-phantom avatar Sep 17 '23 06:09 dev-phantom

Thanks for being interested in this issue. It looks like this ticket is already assigned to a contributor. Please communicate with the assigned contributor to confirm the status of the issue.

github-actions[bot] avatar Sep 17 '23 06:09 github-actions[bot]

.take

Sorry to ask

Don't mean to sound rude

Please how do you intend to fix this?

Hi @dev-phantom! :) I'm going to convert the svg into CSS with a converter. Then, I'll lookup the colors' accessibie counterparts.

CBID2 avatar Sep 17 '23 14:09 CBID2

Thanks for being interested in this issue. It looks like this ticket is already assigned to a contributor. Please communicate with the assigned contributor to confirm the status of the issue.

github-actions[bot] avatar Sep 17 '23 14:09 github-actions[bot]

.take

Sorry to ask Don't mean to sound rude Please how do you intend to fix this?

Hi @dev-phantom! :) I'm going to convert the svg into CSS with a converter. Then, I'll lookup the colors' accessibie counterparts.

Hey @dev-phantom. I created the PR for this issue. I'd love to have your feedback on it. It's #164

CBID2 avatar Sep 17 '23 21:09 CBID2

Thanks for being interested in this issue. It looks like this ticket is already assigned to a contributor. Please communicate with the assigned contributor to confirm the status of the issue.

github-actions[bot] avatar Sep 17 '23 21:09 github-actions[bot]

Hey @CBID2, any updates on this issue? I'm unsure why we're converting this image into CSS if screen readers can't interpret CSS elements. My suggestion is to leave the 'Green Square' as it is and simply add the alt text 'Green Square' for accessibility.

CC @bdougie

shamimbinnur avatar Nov 14 '23 05:11 shamimbinnur

Hey @CBID2, any updates on this issue?

I'm unsure why we're converting this image into CSS if screen readers can't interpret CSS elements. My suggestion is to leave the 'Green Square' as it is and simply add the alt text 'Green Square' for accessibility.

CC @bdougie

Still working on it @shamimbinnur.

CBID2 avatar Nov 14 '23 05:11 CBID2

Unassigning you for the moment @CBID2 as the PR has been open since September 2023. If you decide to pick this up again, feel free to assign yourself.

nickytonline avatar Jan 18 '24 17:01 nickytonline

Still exist here https://opensauced.pizza/teams

Screen Shot 2023-07-26 at 6 59 48 AM

@bdougie @a0m0rajab can you confirm that this change is only expected for the teams page and not home page? GitHubMock.tsx is not being used for the home page and making changes to it, for some reason, makes the build fail, as I think useMediaQuery returns null and causing type error. Or changes need to be done to useMediaQuery as well.

Arol15 avatar Jan 21 '24 21:01 Arol15

Correr. That asset got moved to /teams but still exists live on the page.

bdougie avatar Jan 21 '24 21:01 bdougie