Add a draggable list builder for content entities

Created on 6 August 2018, almost 6 years ago
Updated 9 January 2024, 6 months ago

Problem/Motivation

I often have content entities with a weight, and it would be nice to have a draggable list builder for them, just like we have for config entities.

Proposed resolution

I opened πŸ“Œ Extract DraggableListBuilderTrait from DraggableListBuilder for use with content entities Needs work in core, this builds on that to actually implement a draggable list builder for content entities.

✨ Feature request
Status

RTBC

Version

1.0

Component

Entity CRUD admin UI

Created by

πŸ‡©πŸ‡ͺGermany tstoeckler Essen, Germany

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Open in Jenkins β†’ Open on Drupal.org β†’
    Core: 9.5.5 + Environment: PHP 7.3 & MySQL 5.7
    last update 8 months ago
    PHPLint Failed
  • πŸ‡©πŸ‡ͺGermany tstoeckler Essen, Germany

    Updated for the latest core change. Leaving at RTBC because the integration parts (almost*) didn't change.

    * I just had to add the access check to the entity query.

    I couldn't get #13 to apply (just like the bot, I guess), so I started from #11, and just fixed up the info file change manually.

  • Open in Jenkins β†’ Open on Drupal.org β†’
    Core: 9.5.5 + Environment: PHP 7.3 & MySQL 5.7
    last update 8 months ago
    PHPLint Failed
  • @tstoeckler opened merge request.
  • πŸ‡©πŸ‡ͺGermany tstoeckler Essen, Germany

    Hmm... the DrupalCI setup still uses a too old PHP version for the int|float typehint, so I thought I'd try an MR instead, but I guess Gitlab CI is not set up yet either. So will need some maintainer action at least if we want to see this "officially" pass tests. Leaving at RTBC, though, because the patch itself is fine, as far as I can tell. (And I did try it out locally, at least...)

  • πŸ‡©πŸ‡ͺGermany tstoeckler Essen, Germany

    Note that the core issue has been committed now. πŸŽ‰οΈ

    Still makes sense for the patch/MR to be committed as is, as the new trait wil only be in 10.3 but that should hopefully alleviate any concerns of the long-term maintenance burden.

Production build 0.69.0 2024