? ? NPM Resource Changed Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
29 Nov 2021

The RTL specific css was wrong as can be seen in the before screenshot

This PR removes the RTL specific css (should have been scss) and updates the scss to use logical properties instead.

This PR also removes an unused class breadcrumb-item.

To test RTL you will need to install arabic or persian.

and dont forget to npm ci

There is no visible change in LTR and the RTL changes are as shown below

Before LTR

image

Before RTL

image

After LTR

image

After RTL

image

avatar brianteeman brianteeman - open - 29 Nov 2021
avatar brianteeman brianteeman - change - 29 Nov 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Nov 2021
Category Repository NPM Change
avatar ceford
ceford - comment - 29 Nov 2021

I have tested this item successfully on 0f09661


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36154.

avatar ceford ceford - test_item - 29 Nov 2021 - Tested successfully
avatar pritam825
pritam825 - comment - 2 Dec 2021

I have tested this item successfully on 0f09661


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36154.

avatar pritam825 pritam825 - test_item - 2 Dec 2021 - Tested successfully
avatar richard67 richard67 - change - 2 Dec 2021
Status Pending Ready to Commit
Labels Added: NPM Resource Changed ?
avatar richard67
richard67 - comment - 2 Dec 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36154.

avatar khu5h1
khu5h1 - comment - 4 Dec 2021

I have tested this item successfully on 0f09661


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36154.

avatar khu5h1 khu5h1 - test_item - 4 Dec 2021 - Tested successfully
avatar Quy Quy - change - 2 Jan 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-01-02 18:44:31
Closed_By Quy
Labels Added: ?
avatar Quy Quy - close - 2 Jan 2022
avatar Quy Quy - merge - 2 Jan 2022
avatar Quy
Quy - comment - 2 Jan 2022

Thanks

Add a Comment

Login with GitHub to post a comment