? Success
Pull Request for # 11013

User tests: Successful: Unsuccessful:

avatar C-Lodder
C-Lodder
12 Jul 2016

Pull Request for Issue #11013.

Summary of Changes

Fixes the position of the secondary dropdown if it overflows the viewport height

Testing Instructions

First, ensure the "Banners" menu item is at the bottom of the "Components" dropdown, bu openening the following file ROOT/administrator/language/en-GB/en-GB.com_banners.sys.ini

and changing:
COM_BANNERS="Banners"
to:
COM_BANNERS="Zbanners"

Then ensure you're viewport height is relatively small. Before the patch, the secondary dropdown should look like this:
06d9fc5e-41c3-11e6-9dec-f2131123ba8f

As you can see, it's overflowing the viewport height.

Once the patch is applied, the position will change so all menu items are visible.

avatar C-Lodder C-Lodder - open - 12 Jul 2016
avatar C-Lodder C-Lodder - change - 12 Jul 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Jul 2016
Labels Added: ?
avatar C-Lodder C-Lodder - change - 12 Jul 2016
The description was changed
avatar C-Lodder C-Lodder - change - 12 Jul 2016
The description was changed
avatar C-Lodder C-Lodder - change - 12 Jul 2016
The description was changed
avatar C-Lodder C-Lodder - change - 12 Jul 2016
The description was changed
avatar brianteeman brianteeman - change - 12 Jul 2016
Category Templates (admin) UI/UX
avatar brianteeman brianteeman - change - 12 Jul 2016
Rel_Number 0 11013
Relation Type Pull Request for
avatar dgt41
dgt41 - comment - 12 Jul 2016

Any suggestions for a component with pretty long second level menu to test this?

avatar brianteeman
brianteeman - comment - 12 Jul 2016

Not really - you can fake it in the DB

On 13 July 2016 at 00:32, Dimitri Grammatikogianni <notifications@github.com

wrote:

Any suggestions for a component with pretty long second level menu to test
this?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#11080 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABPH8ZTeknf5Fl0LyQ0WoDD5rIz3F__Aks5qVCQWgaJpZM4JK0HK
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar RonakParmar RonakParmar - test_item - 13 Jul 2016 - Tested successfully
avatar RonakParmar
RonakParmar - comment - 13 Jul 2016

I have tested this item successfully on b5a1206


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

avatar RonakParmar
RonakParmar - comment - 13 Jul 2016

I have applied this PR and tested with my custom component, and works fine.
screen shot 2016-07-12 at 22 51 40


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

avatar killoltailored killoltailored - test_item - 13 Jul 2016 - Tested successfully
avatar killoltailored
killoltailored - comment - 13 Jul 2016

I have tested this item successfully on b5a1206

I have tested successfully with Joomla Version: Joomla! 3.6.0 Stable [ Noether ] 12-July-2016 21:14 GMT
And system Windows 8.1, Browsers: Firefox 47.0.1


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

avatar gunjanpatel gunjanpatel - change - 13 Jul 2016
Status Pending Ready to Commit
avatar dgt41 dgt41 - test_item - 13 Jul 2016 - Tested successfully
avatar dgt41
dgt41 - comment - 13 Jul 2016

I have tested this item successfully on b5a1206


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

avatar joomla-cms-bot joomla-cms-bot - change - 13 Jul 2016
Labels Added: ?
avatar dgt41
dgt41 - comment - 13 Jul 2016

RTC

avatar brianteeman brianteeman - change - 13 Jul 2016
Milestone Added:
avatar roland-d roland-d - close - 16 Jul 2016
avatar roland-d roland-d - merge - 16 Jul 2016
avatar joomla-cms-bot joomla-cms-bot - close - 16 Jul 2016
avatar roland-d roland-d - change - 16 Jul 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-07-16 13:30:44
Closed_By roland-d
avatar joomla-cms-bot joomla-cms-bot - change - 16 Jul 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment