Joomla! Issue Tracker | Joomla! CMS #11216 - Fix last menu items not accessible on small viewport height (Isis) & improve menu scrolling

Fixed in Code Base
21 Jul 2016
Medium
Build: staging
# 11216
Diff
JoomliC:patch-33

Success

Success continuous-integration/travis-ci/pr The Travis CI build passed Details
Success JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed. Details

User tests: Successful: Unsuccessful:

JoomliC
20 Jul 2016

Summary of Changes

Fix accessibility of last menu items of each dropdown if window viewport height is too small to display the full menu and viewport height is less than 570px
Set dynamically max-height for the dropdown menu to adapt window vierport height.
Set dynamically the padding-top for the submenus (replace 8 value for offset top by getting parent menu link and dropdown padding-top)

Testing Instructions

Apply Patch
Clear all caches (admin and browser)
Resize your window in order to hide bottom of one or more dropdown menu when opened
Mouse over the dropdown, with patch applied, the scrolling will be automatically added if needed (without patch, not possible to access last items of the dropdown)
Check if submenu position top is still align with the menu dropdown
Play a bit with different window heights to check if behavior is working as expected : last menu items always accessible when hovering the menu)

Before Patch on latest staging :
(to take into account last changes for submenu dynamic position when outside the viewport: #11080 )

After Patch :

cc/ @C-Lodder

5868b9e 19 Jul 2016

Improve menu scrolling and Fix on small screen

JoomliC - open - 20 Jul 2016

JoomliC - change - 20 Jul 2016

Status

New

⇒

Pending

joomla-cms-bot - change - 20 Jul 2016

Labels

Added: ?

JoomliC - change - 20 Jul 2016

The description was changed

JoomliC - change - 20 Jul 2016

The description was changed

JoomliC - change - 20 Jul 2016

The description was changed

C-Lodder - test_item - 20 Jul 2016 - Tested successfully

C-Lodder - comment - 20 Jul 2016

I have tested this item ✅ successfully on 5868b9e

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11216.}

Thanks @JoomliC

brianteeman - change - 20 Jul 2016

Category

⇒

Administration UI/UX

killoltailored - test_item - 21 Jul 2016 - Tested successfully

killoltailored - comment - 21 Jul 2016

I have tested this item ✅ successfully on 5868b9e

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11216.}

killoltailored - comment - 21 Jul 2016

I have tested this issue successfully with Joomla Version: Joomla! 3.6.1-dev Development [ Noether ] 13-July-2016 10:08 GMT
OS Windows 8.1 and browsers Firefox 47.0.1, Chrome Version 51.0.2704.106 m and Safari 5.1.7

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11216.}

brianteeman - change - 21 Jul 2016

Status

Pending

⇒

Ready to Commit

brianteeman - comment - 21 Jul 2016

RTC

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11216.}

joomla-cms-bot - change - 21 Jul 2016

Labels

Added: ?

brianteeman - change - 21 Jul 2016

Milestone

Added:

joomla-cms-bot - change - 21 Jul 2016

Add a Comment

Older
Newer

Joomla! Issue Tracker - CMS

[#11216] - Fix last menu items not accessible on small viewport height (Isis) & improve menu scrolling

Summary of Changes

Testing Instructions

Add a Comment

Status	Ready to Commit	⇒	Fixed in Code Base
Closed_Date	0000-00-00 00:00:00	⇒	2016-07-21 11:17:10
Closed_By		⇒	wilsonge