? NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
9 Apr 2021

Summary of Changes

Adding shadow to dropdown. Normalizing RTL and LTR. Using em instead of px.
Taking off useless Reduce Line Height parameter (as we now use bootstrap).
Correcting arrow distance from dropdown.

Testing Instructions

Install multilingual site with sample data, including Persian language.
I used فارس (Iran) for Persian native content language)
Edit mod_languages settings to:

Screen Shot 2021-04-09 at 18 37 11

Display the switcher in frontend
Patch and test.
(Reduce Line Height will not display any more in module params)

Actual result BEFORE applying this Pull Request

LTR

before_ltr

RTL

before_rtl

Expected result AFTER applying this Pull Request

LTR

after_ltr

RTL

after_rtl

Documentation Changes Required

avatar infograf768 infograf768 - open - 9 Apr 2021
avatar infograf768 infograf768 - change - 9 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Apr 2021
Category Repository NPM Change Language & Strings Modules Front End
avatar infograf768 infograf768 - change - 9 Apr 2021
The description was changed
avatar infograf768 infograf768 - edited - 9 Apr 2021
avatar infograf768 infograf768 - change - 9 Apr 2021
Labels Added: ? NPM Resource Changed ?
avatar sandramay0905 sandramay0905 - test_item - 10 Apr 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 10 Apr 2021

I have tested this item successfully on 667e695


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

avatar pabloarias pabloarias - test_item - 23 Apr 2021 - Tested unsuccessfully
avatar pabloarias
pabloarias - comment - 23 Apr 2021

I have tested this item ? unsuccessfully on 667e695

All is working fine but the alignment of languages in the dropdown in RTL language. They are aligned to left.


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

avatar infograf768
infograf768 - comment - 28 Apr 2021

@pabloarias
Maybe you did not use the correct clean install with patch as I get here OK
Screen Shot 2021-04-28 at 10 17 04

I suggest you make a clean install with te pre-built packages:
https://ci.joomla.org/artifacts/joomla/joomla-cms/4.0-dev/33082/downloads/41791/

avatar pabloarias pabloarias - test_item - 30 Apr 2021 - Tested successfully
avatar pabloarias
pabloarias - comment - 30 Apr 2021

I have tested this item successfully on 667e695

You are right, after installing the package with Joomla and the PR it works. Thank you!


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

avatar richard67 richard67 - change - 30 Apr 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 30 Apr 2021

RTC


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

avatar HLeithner HLeithner - change - 30 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-30 22:01:13
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 30 Apr 2021
avatar HLeithner HLeithner - merge - 30 Apr 2021

Add a Comment

Login with GitHub to post a comment