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

Fixes in contrast issue in secondary menu in main menu #170

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

mariannuar
Copy link
Collaborator

@mariannuar mariannuar commented Dec 6, 2024

Summary

This PR fixes the issue in the second row of the following spreadsheet

I thought that we would need to update the color but by updating (in fact adding) a font-size of 18px, it meet's accessibility

How to review this pull request

Copy link

netlify bot commented Dec 6, 2024

Deploy Preview for emulsify-ui-kit ready!

Name Link
🔨 Latest commit b0b7414
🔍 Latest deploy log https://app.netlify.com/sites/emulsify-ui-kit/deploys/67535a44b7c5f40008712988
😎 Deploy Preview https://deploy-preview-170--emulsify-ui-kit.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.

@mariannuar mariannuar self-assigned this Dec 6, 2024
@mariannuar mariannuar added the 👍 Ready for Review Work is ready for review. label Dec 6, 2024
Copy link
Contributor

@callinmullaney callinmullaney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mariannuar Looks good! Feel free to merge when you have a moment.

@callinmullaney callinmullaney added 🎉 Passes Functional Review Functionality is approved by the reviewer. 🎉 Passes Code Review Code is approved by the reviewer. 🎉 Ready to Merge Functionality is approved by the reviewer. and removed 👍 Ready for Review Work is ready for review. labels Dec 19, 2024
@mariannuar mariannuar merged commit f06f662 into main Dec 19, 2024
5 of 6 checks passed
@mariannuar mariannuar deleted the a11y-fixes-in-main-menu branch December 19, 2024 17:57
Copy link

🎉 This PR is included in version 1.4.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎉 Passes Code Review Code is approved by the reviewer. 🎉 Passes Functional Review Functionality is approved by the reviewer. 🎉 Ready to Merge Functionality is approved by the reviewer. released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants