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: redesign link sharing panel in right sidebar #11606

Merged
merged 4 commits into from
Sep 24, 2024

Conversation

JammingBen
Copy link
Collaborator

@JammingBen JammingBen commented Sep 19, 2024

Description

Aligns the design of the link sharing section with the people sharing section. Also refactors the code so both sections use the same structure (LinkShares > ListItem > EditDropdown).

Additionally, moves the permanent link to the top right of the sharing panel.

image image image

Related Issue

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests
  • Documentation
  • Maintenance (e.g. dependency updates or tooling)

@JammingBen JammingBen self-assigned this Sep 19, 2024
Copy link

update-docs bot commented Sep 19, 2024

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@JammingBen JammingBen force-pushed the feat/redesign-link-share-section branch 10 times, most recently from 7e5ad7a to b2c9a1e Compare September 20, 2024 07:30
@JammingBen JammingBen marked this pull request as ready for review September 20, 2024 07:53
@tbsbdr
Copy link
Contributor

tbsbdr commented Sep 20, 2024

nice! could you add more spacing to the icons? already fulfilled https://www.w3.org/WAI/WCAG22/Understanding/target-size-minimum.html but its more convenient to have more space.
screenshot_003150

Aligns the design of the link sharing section with the people sharing section. Also refactors the code so both sections use the same structure (`LinkShares` > `ListItem` > `EditDropdown`).
@JammingBen JammingBen force-pushed the feat/redesign-link-share-section branch from 68eebe9 to ca199ad Compare September 24, 2024 11:00
Copy link

sonarcloud bot commented Sep 24, 2024

Copy link
Member

@kulmann kulmann left a comment

Choose a reason for hiding this comment

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

love it 😍

@JammingBen JammingBen dismissed tbsbdr’s stale review September 24, 2024 11:50

Clarified this in the comments.

@JammingBen JammingBen merged commit 80a7b1e into master Sep 24, 2024
3 checks passed
@JammingBen JammingBen deleted the feat/redesign-link-share-section branch September 24, 2024 11:50
@micbar micbar mentioned this pull request Oct 1, 2024
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Redesign link share sidebar section
3 participants