dropbutton.pcss.css: Unexpected duplicate selector ".dropbutton-wrapper.open .dropbutton__toggle::before"

Created on 24 July 2023, over 1 year ago

Problem/Motivation

Unexpected duplicate selector ".dropbutton-wrapper.open .dropbutton__toggle::before" and double "margin-top" with different values, first used at line 208.

Line: 208
  .dropbutton-wrapper.open .dropbutton__toggle::before {
    margin-top: calc(0.5625rem / (2 * 1.41429));
    transform: translate(50%, -50%) rotate(315deg);
  }
...
Line: 225
  .dropbutton-wrapper.open .dropbutton__toggle::before {
    margin-top: calc(0.4375rem / (2 * 1.41429));
  }

Steps to reproduce

File: core/themes/claro/css/components/dropbutton.pcss.css

Proposed resolution

Line: 208
  .dropbutton-wrapper.open .dropbutton__toggle::before {
    margin-top: calc(0.4375rem / (2 * 1.41429));
    transform: translate(50%, -50%) rotate(315deg);
  }

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

🐛 Bug report

Status

Closed: duplicate

Version

11.0 🔥

Component

CSS →

Last updated 5 days ago

Maintained by
🇹🇼Taiwan @johnalbin
🇺🇸United States @mherchel

Created by

🇩🇪Germany fnalb2

Live updates comments and jobs are added and updated live.

Novice
It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Comments & Activities

Issue created by @fnalb2
Comment over 1 year ago →
cilefen
Status changed to Needs review over 1 year ago9:18am 24 July 2023
Open in Jenkins → Open on Drupal.org →
Environment: PHP 8.2 & MySQL 8
last update over 1 year ago
29,878 pass
Comment over 1 year ago →
🇮🇳India _utsavsharma
Patch fixing the issue.
Please review.
Status changed to Closed: duplicate over 1 year ago12:27pm 24 July 2023
Comment over 1 year ago →
🇺🇸United States smustgrave
Should be covered in https://www.drupal.org/project/drupal/issues/3376458 🐛 Update Stylelint to check for duplicate selectors RTBC

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024