Problem/Motivation

The node preview button RTL :active styling is applied to the LTR version as illustrated by this gif:

Proposed resolution

Fix the RTL styling to only apply to LTR

Remaining tasks

  1. Write a patch
  2. Review

User interface changes

The node preview back button has the correct :active styling

API changes

None

CommentFileSizeAuthor
#1 2424733-1.patch708 bytesidebr
#1 node-preview-after.gif23.24 KBidebr
#1 node-preview-rtl-after.gif28.29 KBidebr
node-preview-before.gif18.55 KBidebr

Comments

idebr’s picture

idebr commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new node-preview-rtl-after.gif28.29 KB
new node-preview-after.gif23.24 KB
new 2424733-1.patch708 bytes

Attached fixes the RTL css so it only applies to RTL.

Gif after:

Gif after (RTL):

idebr’s picture

idebr commented
Issue tags: +frontend, +CSS
lewisnyman’s picture

lewisnyman
he/him
commented
Status: Needs review » Reviewed & tested by the community

Confirmed. Thanks for the fix.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

CSS is not frozen in beta. Committed 558acd6 and pushed to 8.0.x. Thanks!

  • alexpott committed 558acd6 on 8.0.x 
    Issue #2424733 by idebr: Node preview backlink RTL button :active...

Status: Fixed » Closed (fixed)

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