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

Fix misaligned list borders #3781

Merged
merged 7 commits into from
Jun 27, 2023
Merged

Fix misaligned list borders #3781

merged 7 commits into from
Jun 27, 2023

Conversation

bearfriend
Copy link
Contributor

Currently, the borders between list items are not evenly spaced between the contents of those list items and also do not line up with the borders around a list item when it is hovered, causing a jitter/jumping effect.

Before and after:
bad-borders good-borders

Once visual diff finishes I'll run this by Jeff before proceding.

@github-actions
Copy link
Contributor

Thanks for the PR! 🎉

We've deployed an automatic preview for this PR - you can see your changes here:

URL https://pr-3781-brightspace-ui-core.d2l.dev/

Note
The build needs to finish before your changes are deployed.
Changes to the PR will automatically update the instance.

@github-actions
Copy link
Contributor

Visual diff tests failed - pull request #3782 has been opened with the updated goldens.

@bearfriend bearfriend marked this pull request as ready for review June 26, 2023 17:06
@bearfriend bearfriend requested a review from a team as a code owner June 26, 2023 17:06
@github-actions
Copy link
Contributor

Visual diff tests failed - pull request #3785 has been opened with the updated goldens.

@bearfriend
Copy link
Contributor Author

#3782 is the vdiff PR to review the report. I split it into two because the second one just changes the height of everything, producing a useless report.

@github-actions
Copy link
Contributor

Visual diff tests failed - pull request #3786 has been opened with the updated goldens.

@bearfriend bearfriend merged commit 62175b4 into main Jun 27, 2023
@bearfriend bearfriend deleted the list-borders branch June 27, 2023 18:30
@ghost
Copy link

ghost commented Jun 27, 2023

🎉 This PR is included in version 2.131.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ghost ghost added the released label Jun 27, 2023
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.

2 participants