NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
31 Mar 2019

Summary of Changes

Adding specific rtl overrides in media-browser.scss

Testing Instructions

To test, patch and use NPM
Install Persian Language.
Display the Media Manager

Before patch

The edit buttons display in RTL the same way as in LTR
Screen Shot 2019-03-31 at 10 18 51

After patch

The buttons are aligned correctly for RTL

Screen Shot 2019-03-31 at 10 19 32

Note: compatible with #24423

avatar infograf768 infograf768 - open - 31 Mar 2019
avatar infograf768 infograf768 - change - 31 Mar 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 Mar 2019
Category Administration com_media NPM Change
avatar brianteeman
brianteeman - comment - 31 Mar 2019

It is not needed to always mirror the ui

avatar ciar4n
ciar4n - comment - 31 Mar 2019

I tend to agree with @brianteeman here. One initial thought for actions in the top right, is it makes them easily accessed by thumb (right hand been most common) on smaller touch screens. Something not relevant to RTL/LTR.

avatar infograf768 infograf768 - change - 31 Mar 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-03-31 18:30:07
Closed_By infograf768
Labels Added: NPM Resource Changed ?
avatar infograf768 infograf768 - close - 31 Mar 2019

Add a Comment

Login with GitHub to post a comment