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

Utilities: Add one value for link-underline-opacity #38131

Merged
merged 1 commit into from
Mar 2, 2023

Conversation

louismaximepiton
Copy link
Member

Description

Add a value for link-underline-opacity

Motivation & Context

Add a use case to make the underline appear.
Might be done in other ways but I think this one is the lightest for the bundle size.
Not quite sure of the accessibility of the new examples.

Type of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change)
  • Breaking change (fix or feature that would change existing functionality)

Checklist

  • I have read the contributing guidelines
  • My code follows the code style of the project (using npm run lint)
  • My change introduces changes to the documentation
  • I have updated the documentation accordingly
  • (NA) I have added tests to cover my changes
  • All new and existing tests passed

Live previews

Related issues

NA

@louismaximepiton louismaximepiton requested a review from a team as a code owner February 28, 2023 10:22
@julien-deramond julien-deramond changed the title Utilities: Add one Utilities: Add one value for link-underline-opacity Feb 28, 2023
@julien-deramond julien-deramond self-requested a review March 2, 2023 06:44
@julien-deramond julien-deramond merged commit 23d20fa into main Mar 2, 2023
@julien-deramond julien-deramond deleted the main-lmp-utilities-update branch March 2, 2023 06:45
@mahilanmjd mahilanmjd mentioned this pull request Apr 16, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants