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

479-user-pics-misaligned-in-issues-assignment-view #2242

Merged

Conversation

JoshBowdenConcepts
Copy link
Contributor

@JoshBowdenConcepts JoshBowdenConcepts commented Sep 6, 2022

What are you trying to accomplish?

Looking to fix the user pics misalignment issue found in the assignments view.
Issue found here.

image

Now it should look like this

image

What approach did you choose and why?

I am following the approach provided in the issue before testing the new canary in dotcom

What should reviewers focus on?

The reviewer should focus on the spacing of the user profile images in the assignment view.

Can these changes ship as is?

  • Yes, this PR does not depend on additional changes. 🚢

@JoshBowdenConcepts JoshBowdenConcepts requested a review from a team as a code owner September 6, 2022 17:57
@changeset-bot
Copy link

changeset-bot bot commented Sep 6, 2022

🦋 Changeset detected

Latest commit: 8718373

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/css Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@JoshBowdenConcepts JoshBowdenConcepts temporarily deployed to github-pages September 6, 2022 18:03 Inactive
@github-actions github-actions bot temporarily deployed to Storybook Preview September 6, 2022 18:03 Inactive
- Not last child means that each stacking avatar would remove the border-right which is necessary to stack correctly. The presentational order is right to left not left to right.
@github-actions github-actions bot temporarily deployed to Storybook Preview September 7, 2022 15:09 Inactive
@JoshBowdenConcepts JoshBowdenConcepts temporarily deployed to github-pages September 7, 2022 15:13 Inactive
@github-actions github-actions bot temporarily deployed to Storybook Preview September 7, 2022 15:13 Inactive
- Fixed pointer to the anchor element
.changeset/smooth-crabs-protect.md Outdated Show resolved Hide resolved
src/avatars/avatar-stack.scss Show resolved Hide resolved
@JoshBowdenConcepts JoshBowdenConcepts temporarily deployed to github-pages September 7, 2022 17:28 Inactive
@github-actions github-actions bot temporarily deployed to Storybook Preview September 7, 2022 17:28 Inactive
- removed branch name from changeset

Co-authored-by: Jon Rohan <yes@jonrohan.codes>
@github-actions github-actions bot temporarily deployed to Storybook Preview September 7, 2022 17:38 Inactive
@JoshBowdenConcepts JoshBowdenConcepts merged commit ef31fd9 into main Sep 7, 2022
@JoshBowdenConcepts JoshBowdenConcepts deleted the 479-user-pics-misaligned-in-issues-assignment-view branch September 7, 2022 18:07
@primer-css primer-css mentioned this pull request Sep 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants