In user.admin.css file, we have following code. Which can be optimised.
.permissions .permission {
padding-left: 1.5em; /* LTR */
}
[dir="rtl"] .permissions .permission {
padding-right: 1.5em;
padding-left: 0;
}
Instead of having padding-left and padding-right, we can have padding-inline-start. So we will not need
[dir="rtl"] .permissions .permission {
padding-right: 1.5em;
padding-left: 0;
}| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 3338823-2.patch | 464 bytes | gauravvvv |
Comments
Comment #2
gauravvvv commentedI have attached a patch for same, Please review
Comment #3
smustgrave commentedFrom what I've seen in the claro refactoring this is a valid cleanup.
Comment #4
longwaveI think this is valid cleanup, but we try to scope problems so they get fixed everywhere at once, instead of just in a single module. Please see https://www.drupal.org/docs/develop/issues/issue-procedures-and-etiquett... for more information on this.
Changing this across all of core already has its own issue: #3312966: Enforce the use of CSS Logical Properties in core - therefore, closing this as duplicate.
Comment #5
longwave