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

feat: Add aria-label to CreateLink button #492

Merged
merged 3 commits into from
Jun 14, 2024
Merged

Conversation

dharanish-v
Copy link
Contributor

@dharanish-v dharanish-v commented Jun 12, 2024

Added aria-label to improve accessibility

closes #493

@dharanish-v dharanish-v marked this pull request as draft June 13, 2024 05:30
@petyosi
Copy link
Contributor

petyosi commented Jun 13, 2024

This should use the translation I believe?

@dharanish-v
Copy link
Contributor Author

This should use the translation I believe?

I have few more changes to this PR, I will push those in an hour

@dharanish-v dharanish-v marked this pull request as ready for review June 13, 2024 08:04
@dharanish-v
Copy link
Contributor Author

This should use the translation I believe?

I have few more changes to this PR, I will push those in an hour

done

@dharanish-v
Copy link
Contributor Author

This should use the translation I believe?

can we merge this by today ?

@petyosi
Copy link
Contributor

petyosi commented Jun 14, 2024

No. It fails the prettier check. I will add the change as a part of a larger commit.

@dharanish-v
Copy link
Contributor Author

No. It fails the prettier check. I will add the change as a part of a larger commit.

I have made the changes, now it won't fail, kindly check because i need this for a task as soon as possible.

@petyosi petyosi merged commit 52d45ea into mdx-editor:main Jun 14, 2024
Copy link

🎉 This PR is included in version 3.4.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

petyosi added a commit that referenced this pull request Jun 22, 2024
resolves #495

Squashed commit of the following:

commit 4cac17b0d31fe88731cc6f5275c83e049bec9fef
Author: Petyo Ivanov <underlog@gmail.com>
Date:   Sat Jun 22 07:21:14 2024 +0300

    chore: eslint

commit 54bee8223588ac082a2cf58c3a4d342e13745a07
Author: Dharanish V <30527938+dharanish-v@users.noreply.github.com>
Date:   Fri Jun 14 22:38:47 2024 +0530

    fix: add aria-label to CreateLink button (#492)

    * feat: Add aria-label to CreateLink button

    * feat: Update CreateLink button aria-label for localization

    * feat: Update CreateLink button aria-label for localization

commit e427be8
Author: Adriano Scazzola <adriano.scazzola@gmail.com>
Date:   Thu Jun 13 14:28:55 2024 -0300

    Add property to display / hidde buttons on BoldItalicUnderlineToggles plugin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve Accessibility of CreateLink and LinkPlugin
2 participants