Skip to content

Fix Sharing page icons for a11y #6347

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

cekk
Copy link
Member

@cekk cekk commented Sep 27, 2024

This solves #5129

I should update translations too because i moved the text from under the table to page description (because it's better for the user to read instructions before the table).

Copy link

netlify bot commented Sep 27, 2024

Deploy Preview for plone-components ready!

Name Link
🔨 Latest commit 1f0dc21
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/670cbf61f5598e0008f9d93e
😎 Deploy Preview https://deploy-preview-6347--plone-components.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@cekk cekk marked this pull request as ready for review September 27, 2024 13:05
@cekk cekk requested review from a team, pnicolli and Wagner3UB September 27, 2024 13:05
@ichim-david
Copy link
Member

Please add the review suggestions and fix the branch conflicts and then we can better check this work from the @plone/volto-accessibility team

@Wagner3UB Wagner3UB requested a review from pnicolli October 14, 2024 06:53
@Wagner3UB
Copy link
Contributor

Wagner3UB commented Oct 14, 2024

@cekk I did the modifications suggested by @pnicolli, but there are some fail unit tests to re-do. Can you please run those tests and fix it if needed?

Copy link

Hi There! 👋

We haven't seen any activity on this pull request in a while 😴, and we want to make sure that it's still relevant. Please let us know by:

  • adding a comment about what needs to be done next 💬
  • updating its status and other labels 🏷️

Otherwise close this pull request. 🧹

@github-actions github-actions bot added the 27 status: stale An item that has not received updates in a long time label Apr 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
27 status: stale An item that has not received updates in a long time
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants