Problem/Motivation

In #3083650-5: Support of btn-outline-danger we decided to use `btn-outline-danger` CSS class which is a style that Seven theme is using.

Claro will use danger button with full background and when core switches to Claro we should also switch to CSS class `btn-danger`.

CommentFileSizeAuthor
#4 delete-link-styling-3085223-4.patch1.89 KBpivica

Comments

pivica created an issue. See original summary.

pivica’s picture

pivica commented

Check #3083650-9: Support of btn-outline-danger for an idea about button link CSS class.

Also, latest Claro screenshots show that they dropped danger button for a link with left icon, check screenshots in https://www.drupal.org/project/drupal/issues/3079738.

Let's install the latest Claro, check how it works related to this and implement a similar solution.

pivica’s picture

pivica commented

Claro will be used as default admin theme from Drupal 9.4 so we can do this for next beta3 release.

pivica’s picture

pivica commented
Status: Active » Needs review
StatusFileSize
new delete-link-styling-3085223-4.patch1.89 KB

In the mean time, Claro switched to link style so we will do the same. Here is a patch.

  • pivica committed 720837ab on 8.x-1.x 
    Issue #3085223 by pivica: When core switch to claro we should switch to...
pivica’s picture

pivica commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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