? ? Conflicting Files ? ? Pending

User tests: Successful: Unsuccessful:

avatar pinak3748
pinak3748
14 Aug 2021

Pull Request for Issue #35068

Summary of Changes

Currently, the left navigation expands and scroll down when you scroll the page. For example, if you have 20 components, you expand the components menu and click on the last link. The page loads with the last link down at the bottom of the page. you need to scroll down to see the bottom links.

Actual result BEFORE applying this Pull Request

Modules (Administrator) - pinak - Administration - Brave 14-08-2021 11_25_14

Expected result AFTER applying this Pull Request

Modules (Administrator) - pinak - Administration - Brave 14-08-2021 11_25_29

Documentation Changes Required

avatar pinak3748 pinak3748 - open - 14 Aug 2021
avatar pinak3748 pinak3748 - change - 14 Aug 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Aug 2021
Category Administration Templates (admin) NPM Change
avatar boddunan
boddunan - comment - 14 Aug 2021

I don't see any difference in functionality between RC6 and this. In both cases, the last menu link appears down at the bottom and need to scroll the page to see it.

avatar brianteeman
brianteeman - comment - 14 Aug 2021

Are you sure you committed all the files that you wanted to change? there is nothing in the changes currently that could possibly make the navigation sticky or scrollable.

need more coffee

avatar pinak3748
pinak3748 - comment - 14 Aug 2021
Untitled.mp4

It works in my system I think you forgot to npm ci or else try clearing cache of localhost
Mine also did not show any changes after clearing my localhost cache it workes

avatar brianteeman brianteeman - test_item - 14 Aug 2021 - Tested successfully
avatar brianteeman
brianteeman - comment - 14 Aug 2021

I have tested this item successfully on 104faf5


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

avatar boddunan
boddunan - comment - 14 Aug 2021

The main content is now scrollable now without the sidebar, but it is more painful than earlier. The main issue is the sidebar was not scrolled to the last non-visible item, this fix made the content scrollable with stick sidebar. Now if I want to see the last menu item, I have to scroll the entire content before the sidebar scrolls. I would rather stick to an old design than this. A better approach would be scroll left sidebar to selected menu item when the page is loaded.

avatar jwaisner jwaisner - test_item - 24 Aug 2021 - Tested successfully
avatar jwaisner
jwaisner - comment - 24 Aug 2021

I have tested this item successfully on 104faf5


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

avatar jwaisner jwaisner - change - 24 Aug 2021
Status Pending Ready to Commit
avatar jwaisner
jwaisner - comment - 24 Aug 2021

RTC


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

avatar boddunan
boddunan - comment - 24 Aug 2021

Request you to please relook at the issue that I raised before (the issue for which this PR was raised), this fix just aggravates the problem than fixing it.

avatar pritam825
pritam825 - comment - 2 Dec 2021

I have tested this item successfully on 104faf5


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

avatar pritam825 pritam825 - test_item - 2 Dec 2021 - Tested successfully
avatar bembelimen bembelimen - change - 22 Jan 2022
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2022-01-22 11:47:24
Closed_By bembelimen
Labels Added: ? ? Conflicting Files ? ?
avatar bembelimen
bembelimen - comment - 22 Jan 2022

I see it the same, we need a better solution for longer menus.

Although I close this PR, thank you for your contribution @pinak3748

avatar bembelimen bembelimen - close - 22 Jan 2022

Add a Comment

Login with GitHub to post a comment