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;
}
CommentFileSizeAuthor
#2 3338823-2.patch464 bytesgauravvvv

Comments

Gauravvv created an issue. See original summary.

gauravvvv’s picture

Status: Active » Needs review
StatusFileSize
new464 bytes

I have attached a patch for same, Please review

smustgrave’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: +Needs Review Queue Initiative

From what I've seen in the claro refactoring this is a valid cleanup.

longwave’s picture

Status: Reviewed & tested by the community » Closed (duplicate)

I 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.

longwave’s picture