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

V15/sort by selected icons #18158

Merged
merged 8 commits into from
Jan 31, 2025
Merged

V15/sort by selected icons #18158

merged 8 commits into from
Jan 31, 2025

Conversation

Welander1994
Copy link

Prerequisites

  • I have added steps to test this contribution in the description below

If there's an existing issue for this PR then this fixes
Sort children displays default icon instead of selected icon #18153

Description

adds back the custom icon for the content nodes and give a hover effect to indicate that it is grabable, and applys the choosen color

Steps to reproduce

  • Create a couple of document types and set their icon
  • Create some content
  • Use the sort children action

…ct to indicate that it is grabable, and applys the choosen color
@Welander1994 Welander1994 changed the base branch from contrib to v15/dev January 29, 2025 12:48
@nul800sebastiaan
Copy link
Member

According to the related issue this has already been fixed in #17904 ?

@Welander1994 Welander1994 marked this pull request as ready for review January 30, 2025 09:18
@Welander1994
Copy link
Author

Hi bjarnef thanks for your input.
I have changed the uui-table-cell to use umb-icon instead of uui-icon

@madsrasmussen madsrasmussen added the category/ux User experience label Jan 31, 2025
@madsrasmussen
Copy link
Contributor

I have realized that we do not know where to find the icon in this generic modal. I have reverted the icon in this PR to the one that we currently use.

This PR then changes padding in the table so each item takes up less space.

@madsrasmussen madsrasmussen enabled auto-merge (squash) January 31, 2025 12:33
@madsrasmussen madsrasmussen merged commit 3829b09 into v15/dev Jan 31, 2025
29 checks passed
@madsrasmussen madsrasmussen deleted the v15/SortBySelectedIcons branch January 31, 2025 13:09
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.

5 participants