NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
6 Jun 2020

Pull Request for Issue #29438

Summary of Changes

Correcting rtl css
Needs npm ci

Testing Instructions

Install Persian language and switch to Persian in backend
Edit a site module =>Menu Assignments
Display 3rd choice in dropdown
Screen Shot 2020-06-06 at 11 57 22

Before patch

Screen Shot 2020-06-06 at 10 57 33

After patch

Screen Shot 2020-06-06 at 11 44 32

avatar infograf768 infograf768 - open - 6 Jun 2020
avatar infograf768 infograf768 - change - 6 Jun 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Jun 2020
Category Administration Templates (admin) NPM Change
avatar adj9
adj9 - comment - 6 Jun 2020

I have set the entire site in multi language with the default language in question.
I went to module =>Menu Assignments of a generic module and I don't see the multi language flag with PR enabled.

I've enabled all the multi language plugins.

Are there other settings to set?

avatar infograf768
infograf768 - comment - 6 Jun 2020

The flags are epiphenomenic. If they don’t show, it just means you may not have set the menu items correctly.
This PR does not concerns the flags but the menu items tree.

the pr can be tested on a monolingual site where Persian is installed and chosen as backend language.

avatar richard67 richard67 - test_item - 6 Jun 2020 - Tested successfully
avatar richard67
richard67 - comment - 6 Jun 2020

I have tested this item successfully on d0ef247

Without PR, elements in the tree for menu assingment are not indented according to their level in the tree when RTL.
With the PR (and npm ci), elements in the tree for menu assingment are indented correctly according to their level in the tree when RTL.


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

avatar infograf768 infograf768 - change - 7 Jun 2020
Labels Added: NPM Resource Changed ?
avatar infograf768
infograf768 - comment - 7 Jun 2020

@ciar4n
done.
@richard67

please test again.

Note: I have a PR ready concerning the chevron for rtl (in js). I guess it is better to first merge this.

avatar richard67 richard67 - test_item - 7 Jun 2020 - Tested successfully
avatar richard67
richard67 - comment - 7 Jun 2020

I have tested this item successfully on c3b1cc0


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

avatar infograf768
infograf768 - comment - 8 Jun 2020

@wilsonge
Please test and let's get this one in.

avatar ciar4n ciar4n - test_item - 8 Jun 2020 - Tested successfully
avatar ciar4n
ciar4n - comment - 8 Jun 2020

I have tested this item successfully on c3b1cc0


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

avatar infograf768 infograf768 - change - 8 Jun 2020
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 8 Jun 2020

rtc


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

avatar wilsonge wilsonge - change - 8 Jun 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-06-08 12:02:43
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 8 Jun 2020
avatar wilsonge wilsonge - merge - 8 Jun 2020
avatar wilsonge
wilsonge - comment - 8 Jun 2020

Thanks!

Add a Comment

Login with GitHub to post a comment