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

Revamp the reorderable list component #10186

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

janfaracik
Copy link
Contributor

@janfaracik janfaracik commented Jan 21, 2025

There's a few issues with this still so getting it up in draft for now.

Before
image

After
image

The goal of the revamp is to cleanup and simplify the reorderable list interface, making it easier on the eyes but also more compact. There's a lot of unnecessary (and unintentional) padding at the moment which makes it a little awkward looking. Reordering the list is also a little clunky, so this PR adds an animation which makes it a lot more pleasant to drag around.

Testing done

Proposed changelog entries

  • human-readable text

Proposed upgrade guidelines

N/A

Submitter checklist

Preview Give feedback

Desired reviewers

@mention

Before the changes are marked as ready-for-merge:

Maintainer checklist

Preview Give feedback

@janfaracik janfaracik changed the title Revamp the re-orderable list component Revamp the reorderable list component Jan 21, 2025
@github-actions github-actions bot added the unresolved-merge-conflict There is a merge conflict with the target branch. label Jan 21, 2025
Copy link
Contributor

Please take a moment and address the merge conflicts of your pull request. Thanks!

@mawinter69
Copy link
Contributor

Maybe add screenshots for something that has also content like the build steps in a freestyle job

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
unresolved-merge-conflict There is a merge conflict with the target branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants