Problem/Motivation

Paragraphs within the modal are not sorted based on their Paragraph Selection weight

Steps to reproduce

Proposed resolution

Use the weight within the field configuration to sort the Paragraphs within the modal

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 3381981-sort-by-weight-4.patch1.14 KBhlopez
#4 interdiff-3381981-sort-by-weight-3_4.txt912 byteshlopez
#3 interdiff-3381981-sort-by-weight-2_3.txt898 byteshlopez
#3 3381981-sort-by-weight-3.patch1.14 KBhlopez
#2 3381981-sort-by-weight-2.patch1.14 KBhlopez

Issue fork paragraphs_browser-3381981

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

hlopez created an issue. See original summary.

hlopez’s picture

hlopez commented
Status: Active » Needs review
StatusFileSize
new 3381981-sort-by-weight-2.patch1.14 KB
hlopez’s picture

hlopez commented
StatusFileSize
new 3381981-sort-by-weight-3.patch1.14 KB
new interdiff-3381981-sort-by-weight-2_3.txt898 bytes

Fix conditional

hlopez’s picture

hlopez commented
StatusFileSize
new interdiff-3381981-sort-by-weight-3_4.txt912 bytes
new 3381981-sort-by-weight-4.patch1.14 KB

Switch to ascending order

bander2’s picture

bander2 commented
Status: Needs review » Reviewed & tested by the community

#4 works. Drupal 10.2.6 Paragraphs Browser 1.1.0.

mlncn’s picture

mlncn commented
Version: 8.x-1.1 » 8.x-1.x-dev

Confirming this fix; would be excellent to get it in a release. Thanks!

lifelifeone’s picture

lifelifeone commented

Can confirm patch #4 fixes the issue. Tested on Drupal 10.4.6 and Paragraphs Browser 1.3.0 👍

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented

Can we convert it to a merge request instead of just a patch please?

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

damienmckenna opened merge request !27

bvoynick’s picture

bvoynick
he/him
Primary language English
commented
Related issues: +#3448164: Sort the paragraphs alphabetically by label, +#2883185: Sort the paragraphs by label.
bvoynick’s picture

bvoynick
he/him
Primary language English
commented
Category: Feature request » Bug report

bvoynick’s picture

bvoynick
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Thanks y'all! Committed

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.