NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
21 Sep 2019

Summary of Changes

When editing template files in RTL, the text content should be aligned left.
Adding rtl overrides in .../build/media_source/com_templates/css/admin-templates-default.css both for Codemirror and no editor

Testing Instructions

Edit a template file through administration.
No need to test with Persian language. Simpler in this case to set en-GB.xml to rtl
<rtl>1</rtl>

Before patch

Screen Shot 2019-09-21 at 18 57 00

After patch

Screen Shot 2019-09-21 at 18 55 19

@Harmageddon
@richard67

avatar infograf768 infograf768 - open - 21 Sep 2019
avatar infograf768 infograf768 - change - 21 Sep 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Sep 2019
Category Repository NPM Change
avatar infograf768 infograf768 - change - 21 Sep 2019
The description was changed
avatar infograf768 infograf768 - edited - 21 Sep 2019
avatar richard67 richard67 - test_item - 21 Sep 2019 - Tested successfully
avatar richard67
richard67 - comment - 21 Sep 2019

I have tested this item successfully on 4abca42


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

avatar infograf768 infograf768 - change - 22 Sep 2019
Labels Added: NPM Resource Changed ?
avatar joomla-cms-bot joomla-cms-bot - change - 22 Sep 2019
Category Repository NPM Change Administration com_templates Repository NPM Change
avatar infograf768
infograf768 - comment - 22 Sep 2019

Space corrected.
While I was at it and to avoid another very small PR, I also corrected the display of the path of the edited file.

Before

The slash is misplaced at the end of the path.

Screen Shot 2019-09-22 at 08 11 37

After

The position of the slash is corrected

Screen Shot 2019-09-22 at 08 10 59

avatar Harmageddon Harmageddon - test_item - 22 Sep 2019 - Tested successfully
avatar Harmageddon
Harmageddon - comment - 22 Sep 2019

I have tested this item successfully on 64732b0


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

avatar richard67 richard67 - test_item - 22 Sep 2019 - Tested successfully
avatar richard67
richard67 - comment - 22 Sep 2019

I have tested this item successfully on 64732b0


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

avatar infograf768 infograf768 - change - 22 Sep 2019
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 22 Sep 2019

rtc
thanks for testing


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

avatar infograf768 infograf768 - change - 24 Sep 2019
Labels Added: ?
avatar wilsonge
wilsonge - comment - 30 Sep 2019

I think it's probably better to do this in javascript than CSS. But right now I don't have the time to find how to do this properly in all places. So merging. If anyone has the time to try and put this into JS please do!

avatar wilsonge wilsonge - change - 30 Sep 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-09-30 12:50:03
Closed_By wilsonge
avatar wilsonge wilsonge - close - 30 Sep 2019
avatar wilsonge wilsonge - merge - 30 Sep 2019
avatar wilsonge
wilsonge - comment - 30 Sep 2019

Thanks!

Add a Comment

Login with GitHub to post a comment