We have css in tabledrag.css for menu-item__link, are we 100% sure it's not u...

Contribution Date

Wed, 04/12/2023 - 12:00

Technology

Contribution Project

Contribution Details

We have css in tabledrag.css for menu-item__link, are we 100% sure it's not used.

.touchevents .draggable .menu-item__link {
  padding-top: var(--space-xs);
  padding-bottom: var(--space-xs);
}

This CSS is applied to mobile devices, as the `.touchevents` class is available their only. But it doesn't have any effects on the element. As the parent element have property `vertical-align: middle;`. There is no need to align element by padding. I have added a screen recording for reference.

Contribution Author

gauravmahlawat

Contribution Issue Link

Remove completely irrelevant addition of "menu-item__link" class in tabledrag.js

Contribution Link

https://www.drupal.org/project/drupal/issues/2277795#comment-15006585

Files count

2

Patches count

0