Problem/Motivation

Core dropdown link/button support should be improved. Here are some examples:

  • Admin content type page

  • Node form

Proposed resolution

Consider using https://v4-alpha.getbootstrap.com/components/dropdowns/ for this, not sure will this work fine when dropdown items are button - this is the case for node form.

Remaining tasks

Do it.

Comments

pivica created an issue. See original summary.

njagojevic’s picture

Assigned: Unassigned » njagojevic
njagojevic’s picture

Status: Active » Needs review
StatusFileSize
new18.17 KB
new42.71 KB

1. Improved core dropdown link/button support for a dropdown like in admin content type page,
2. Added support for Bootstrap button group https://v4-alpha.getbootstrap.com/components/button-group/ - for 1. case this is better solution than pure dropdown, as you can directly click on first dropdown item that is displayed by default; support added also in starterkit.
3. Also improved local action button (button Add content type on image below) with this patch,
4. For 1. I overwrited template links--dropbutton--operations.html.twig and left links--dropbutton.html.twig for dropdown link/button in node form. It's difficult to override this template as it always prints btn-primary submit button in a dropdown item. Any suggestions how to do this?

This is how form looks like after this patch:

njagojevic’s picture

Assigned: njagojevic » Unassigned
pivica’s picture

Category: Bug report » Task
Status: Needs review » Needs work

Commit 439896dbe from #2897504-3: Node edit form metadata form items could go to the right improved visual look of drop-down button/link component, we will need to check this issue now again and figure do we want to continue pushing it or close it.

One possible way for continuing here is to convert Drupal drop down component with Bootstrap drop down component - this still has various advantages, especially because on drop-down open layout is not affected.

mbovan’s picture

pivica’s picture

Version: 8.x-1.x-dev » 2.x-dev
Priority: Normal » Minor

Not sure about this now, let's move it to 2.x version for now.