Problem/Motivation

Just found this...

          $widget_actions['actions']['edit_button'] = $this->expandButton([

            '#attributes' => ['class' => ['paragraphs-button']],

            '#attributes' => [
              'class' => ['paragraphs-icon-button', 'paragraphs-icon-button-edit'],
              'title' => $this->t('Edit'),
            ],

So i guess the first class isn't present at all? :-)

Proposed resolution

Drop the duplicate definition.

CommentFileSizeAuthor
#4 paragraphs-duplicatebuttons-2940989-D8.patch683 bytesioana apetri
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

miro_dietiker created an issue. See original summary.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented

It seems the class paragraphs-button is completely unused, so we just drop it as it wasn't output anyways.

ioana apetri’s picture

ioana apetri CreditAttribution: ioana apetri at OPTASY commented
Assigned: Unassigned » ioana apetri

I'll work. Thanks

ioana apetri’s picture

ioana apetri CreditAttribution: ioana apetri at OPTASY commented
Status: Active » Needs review
FileSize
paragraphs-duplicatebuttons-2940989-D8.patch683 bytes
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Reviewed & tested by the community

Makes sense.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Reviewed & tested by the community » Fixed

Committing, thanks. :-)

Status: Fixed » Closed (fixed)

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