It's a very good alternative of draggable views module,

One suggestion, its better if we can Highlight row on till sorted weights are not saved.

CommentFileSizeAuthor
#13 Screenshot 2026-06-13 at 05.57.51.png73.96 KBkieran.cott

Issue fork sortableviews-3020269

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

visabhishek created an issue. See original summary.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Issue tags: +Novice

Yeah that's a really nice idea, maybe the classes are already set by SortableJS and we just need some CSS (that should be as universal as possible). Maybe someone could create a MR?

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Title: Heighlight row on unsaved state » Highlight row on unsaved state

enomo103 made their first commit to this issue’s fork.

enomo103 opened merge request !18

enomo103’s picture

enomo103 commented
Status: Active » Needs review

kieran.cott made their first commit to this issue’s fork.

kieran.cott’s picture

kieran.cott
Location 🇬🇧 United Kingdom
commented

Nice one Takuya - MR 18 looks like it'll work, though at the moment it looks like the pipelines are failing.

Since we're modifying this part of the code anyway, it might be a good opportunity to get the pipelines to pass.

I’ve prepared a couple of small follow-up fixes to get the pipeline green: replacing the deprecated Request::get() usage while preserving the existing parameter lookup behaviour, updating the PHPUnit 11 expectations/data providers, and fixing the CSS property order warning.

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented
Status: Needs review » Needs work

The JS part looks good, but this seems to contain a lot of unrelated changes. Please only add the related parts here!

See #3584702: Fix code style (cspell, phpstan) and phpunit for other tasks.

kieran.cott’s picture

kieran.cott
Location 🇬🇧 United Kingdom
commented

Got it, I've reverted the MR to its original state in comment #5, and will address the pipeline failures in issue 3584702 instead.

kieran.cott’s picture

kieran.cott
Location 🇬🇧 United Kingdom
commented
Status: Needs work » Needs review
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Thanks, that LGTM, did you test it manually?

kieran.cott’s picture

kieran.cott
Location 🇬🇧 United Kingdom
commented
StatusFileSize
new Screenshot 2026-06-13 at 05.57.51.png73.96 KB

I have, yes - looks good!

kieran.cott’s picture

kieran.cott
Location 🇬🇧 United Kingdom
commented
Status: Needs review » Reviewed & tested by the community
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Thanks, yes. Confirming RTBC.

dieterholvoet made their first commit to this issue’s fork.

dieterholvoet’s picture

dieterholvoet commented
Status: Reviewed & tested by the community » Fixed

Works well, thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.