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(issue-views): Improve drag handle and safari interactions #87119

Merged
merged 2 commits into from
Mar 14, 2025

Conversation

MichaelSun48
Copy link
Member

@MichaelSun48 MichaelSun48 commented Mar 14, 2025

This PR makes a couple of minor improvements to issue views, specifically related to the newly added grab handle, and safari improvements:

  1. Hover states in Safari are no longer super janky. This was caused by some CSS selectors not being supported by safari.
  2. The drag handle now has an interaction state layer to make it more clear that it is draggable
  3. The background opacity of each issue view has been slightly decreased so that an item being dragged will slightly show the items beneath it.

Note that there is still an issue where triggering a reorder while dragging in safari will cause the hover state to be lost. This seems to be an intentional decision by Safari, but I could not figure out a reasonable fix in the timebox I gave myself.

@MichaelSun48 MichaelSun48 requested a review from a team March 14, 2025 19:41
@MichaelSun48 MichaelSun48 requested a review from a team as a code owner March 14, 2025 19:41
@MichaelSun48 MichaelSun48 requested review from malwilley and removed request for a team March 14, 2025 19:41
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 14, 2025
@@ -45,7 +44,6 @@ const IconWrap = styled('div')`
flex-direction: column;
align-items: center;
justify-content: center;
margin-right: ${space(0.75)};
Copy link
Member Author

Choose a reason for hiding this comment

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

The project icon should not have an opinion on margin

@MichaelSun48 MichaelSun48 merged commit b1a59e9 into master Mar 14, 2025
41 checks passed
@MichaelSun48 MichaelSun48 deleted the msun/issueViews/grabHandleHoverImprovements branch March 14, 2025 21:06
@github-actions github-actions bot locked and limited conversation to collaborators Mar 30, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants