? ? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
24 Sep 2018

Pull Request for Issue # .

Summary of Changes

Creates a dropdown-reverse class to align dropdowns left instead of right. Applies this class to com-menus -> menus.

Simple class which will only work for one level of child items.

Testing Instructions

Apply and navigate to Menu Manager

Before

image

After

image

Documentation Changes Required

avatar ciar4n ciar4n - open - 24 Sep 2018
avatar ciar4n ciar4n - change - 24 Sep 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Sep 2018
Category Administration com_menus Templates (admin)
avatar ciar4n ciar4n - change - 24 Sep 2018
Labels Added: ?
avatar Quy
Quy - comment - 24 Sep 2018

Now it is an issue with RTL.

rtl

avatar brianteeman
brianteeman - comment - 24 Sep 2018

I think the correct way to resolve this is using the bootstrap dropdown and associated popper js - this is what we already use in the save button https://github.com/joomla/joomla-cms/pull/22015/files

We should avoid (if possible) ever having markup that says left which is actually right because of the css

avatar ciar4n
ciar4n - comment - 24 Sep 2018

Considering the RTL issue, I was gonna suggest wording something like dropdown-reverse. But yes, if this has already been dealt with in a different way then let's go with that.

avatar ciar4n
ciar4n - comment - 24 Sep 2018

This dropdown does not appear to use popper.js ?

avatar mbabker
mbabker - comment - 24 Sep 2018

popper.js isn't used in BS2 I thought?

avatar brianteeman
brianteeman - comment - 24 Sep 2018

bubbles I didnt realise that it was new in bs4

avatar ciar4n ciar4n - change - 25 Sep 2018
Title
Create and apply 'dropdown-left' css class
Create and apply 'dropdown-reverse' css class
avatar ciar4n ciar4n - edited - 25 Sep 2018
avatar ciar4n ciar4n - change - 25 Sep 2018
The description was changed
avatar ciar4n ciar4n - edited - 25 Sep 2018
avatar Quy
Quy - comment - 25 Sep 2018

I have tested this item successfully on 5ee8ca1


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

avatar Quy Quy - test_item - 25 Sep 2018 - Tested successfully
avatar infograf768
infograf768 - comment - 26 Sep 2018

I have tested this item successfully on 5ee8ca1


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

avatar infograf768 infograf768 - test_item - 26 Sep 2018 - Tested successfully
avatar infograf768 infograf768 - change - 26 Sep 2018
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 26 Sep 2018

RTC


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

avatar mbabker mbabker - change - 2 Oct 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-10-02 16:12:54
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 2 Oct 2018
avatar mbabker mbabker - merge - 2 Oct 2018

Add a Comment

Login with GitHub to post a comment