Problem/Motivation

When switching the media library core bulk actions field to VBO + views_bulk_edit, which provides bulk editing and a lot more functionality, the styling of the checkbox is broken in the media library.

We should instead handle the VBO views checkbox 1:1 like the core one so both work. We definitely need VBO in large projects media library to be able to bulk edit media entity field values.

An example for the bug is "GS" ;)

Steps to reproduce

Proposed resolution

Style VBO checkboxes like core bulk actions
Try to replace core bulk actions form by VBO for the following benefits:

  • Select all - functionality
  • Modify values functionality

Remaining tasks

User interface changes

API changes

Data model changes

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:

Comments

Anybody created an issue. See original summary.

anybody’s picture

@thomas.frobieter please look at this in the context of "GS" (but after all other changes and a new stable release!)

anybody’s picture

@thomas.frobieter: In 3.1.0 (and our VL) is prepared the switch to VBO in general. Currently the field is still hidden, but was already added. We should have a look at this. You can already prepare the styling of course.

But then we should check, if there are any drawbacks from this replacement.

anybody’s picture

@thomas.frobieter I just tried VBO at MD, but if I check checkboxes, even by "select all" it always says "No entries selected". No idea why, but for that reason I'll remove VBO until we can have a look.

  • Anybody committed 9a5a261 on 3.x
    Issue #3274230 by Anybody: Media Library: Display VBO checkbox like core...
anybody’s picture

Issue summary: View changes
anybody’s picture

@thomas.frobieter after testing it carefully, we might also completely replace the core bulk actions by VBO. Call me when you're ready, as this is important for larger pages from time to time so higher priority, but not critical.

We should solve this early once you have the time.

anybody’s picture

Okay, indeed core AND VBO can't Co-Exist (not even hidden) in the same view, VBO has to replace the core select, otherwise both say "Nothing selected".

Styling should be compatible with both for special cases and backwards compatibility. When styling is in here, I'll create an update hook for the views.media_library.yml changes.

Old core style can be tested in VL, new VBO checkbox at GS.

anybody’s picture

Status: Active » Needs work
thomas.frobieter’s picture

Hopefully VBO is not that buggy as it seems in the GS project. I will try to add the VBO styles as addition to the regular media library styles, so we can disable VBO at all time.

Update: Just saw that the regular media library checkboxes are still present in the media selection widgets, so we MUST support both at the same time.

thomas.frobieter’s picture

Assigned: thomas.frobieter » anybody
Status: Needs work » Needs review
anybody’s picture

Assigned: anybody » thomas.frobieter
Status: Needs review » Needs work

Review done, just some final adjustments, then this is ready to go! (resolve the reviews)

Do you see any risk of breaking pages where VBO is NOT used? I think not... but please finally think about that.

thomas.frobieter’s picture

No, I wrote it explicitly to support both. No changes to the core bulk styles, only additions for VBO.

anybody’s picture

Assigned: thomas.frobieter » Unassigned
Status: Needs work » Reviewed & tested by the community

All fine!

anybody’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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