Nextcloud-vue: Scrollbar on AppNavigation

Created on 20 Jan 2020 · 2Comments · Source: nextcloud/nextcloud-vue

In the Calendar, opening or closing the settings introduces the scollbar for a second so the + New event and other buttons shrinks a bit for a split second.

output

Having overflow-y: hidden; would fix the issue but it looks like it is set to auto explicitly. Was that intentional? Maybe add the overflow-y as a configurable prop in case an app wants the scrollbar?

I can make the PR.

https://github.com/nextcloud/nextcloud-vue/blob/88c062728fb6b2df6977b45aa32e4e61b109c7e5/src/components/AppNavigation/AppNavigation.vue#L46

cc @skjnldsv @georgehrke

1. to develop app-navigation

Source

gary-kim

Most helpful comment

Sorry, I fall over my keyboard. What I wanted to say: you can't know if your navigation is longer than available room. Hence overflow must be auto. The issue has to be solved using another fix.

korelstar on 20 Feb 2020

👍3

All 2 comments

Sorry, I fall over my keyboard. What I wanted to say: you can't know if your navigation is longer than available room. Hence overflow must be auto. The issue has to be solved using another fix.

korelstar on 20 Feb 2020

👍3

@gary-kim Does removing the height: 100% on the ul fix your issue? Could be it doesn't need to re-render the 100% then...?

Edit: Just recognized i can check that on my own. 🤦‍♂ It doesn't solve.