Problem/Motivation

This would improve UX. It's also a blocker to #3425489: use DraggableListBuilder, as using a textfield form element means that the weights are saved as strings instead of ints, and this crashes DraggableListBuilder.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#5 environment_indicator_weight_field.png19.73 KBtessa bakker

Issue fork environment_indicator-3425490

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

joachim created an issue. See original summary.

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

trackleft2 opened merge request !60

trackleft2’s picture

trackleft2
Primary language English
Location Tucson, AZ 🇺🇸
commented
Status: Active » Needs review
tessa bakker’s picture

tessa bakker
she/they
Primary language Dutch
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new environment_indicator_weight_field.png19.73 KB

Change works as expected:

Select-element (single) of the Weight field after the change

Update hook isn't needed, existing data remains intact.

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

isholgueras’s picture

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

In addition to the Tess's tests, when it's shown its sorted correctly. The config switcher exported also exports correctly the weight. Thanks for the work!

Status: Fixed » Closed (fixed)

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