Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug Report][2.4.7] Navigation Drawer - expand-on-hover, mini-variant and v-main width #13309

Closed
jd-solanki opened this issue Mar 23, 2021 · 3 comments · Fixed by #14523
Closed
Assignees
Labels
C: VNavigationDrawer VNavigationDrawer S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected
Milestone

Comments

@jd-solanki
Copy link
Contributor

Environment

Vuetify Version: 2.4.7
Vue Version: 2.6.12
Browsers: Chrome 89.0.4389.90
OS: Linux x86_64

Steps to reproduce

  1. Hover over navigation drawer
  2. Click on plus icon to expand navigation drawer (Working as expected)
  3. Click again on minus icon to collapse it (make it mini and enable expand on hover - Main content should update according to navigation drawer width)

Expected Behavior

When we click on minus icon. Navigation menu should collapse and v-main should get more width as navigation have less width.

Also when navigation drawer is collapsed it should have expand on hover functionality.

In short, when navigation drawer is collapsed it can be viewd as full on hover. And width of v-main should update according to width of navigation drawer except navigation drawer is mini and user hovers it for temporary view.

Actual Behavior

v-main is not adapting remaining space when we collapse the navigation drawer again.

Reproduction Link

https://codepen.io/jd-0001/pen/JjEPRqY?editors=101

Other comments

Expected behavior: https://pixinvent.com/demo/vuexy-vuejs-admin-dashboard-template/demo-1/dashboard/analytics

Check pinning the navigation drawer and content area width according to width of navigation drawer.

@ghost ghost added the S: triage label Mar 23, 2021
@ElijahKotyluk ElijahKotyluk added the C: VNavigationDrawer VNavigationDrawer label Jun 24, 2021
@johnleider johnleider changed the title [Bug Report] Navigation Drawer - expand-on-hover, mini-variant and v-main width [Bug Report][2.4.7] Navigation Drawer - expand-on-hover, mini-variant and v-main width Jul 16, 2021
@panteparak

This comment has been minimized.

@MisterIsaak

This comment has been minimized.

@changemyminds
Copy link

changemyminds commented Jul 28, 2022

Just add key value and it will work fine. see https://codepen.io/changemyminds/pen/MWVOgJV?editors=101

This bugs I think is vue redner issue, see can-you-force-vue-js-to-reload-re-render

I guess fixed this issue casued #15531 bugs.

<v-navigation-drawer
  v-model="drawer"
  :mini-variant="mini"
  :expand-on-hover="mini"
  :key="mini"                   
  app
>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: VNavigationDrawer VNavigationDrawer S: has PR The issue has a pending Pull Request T: bug Functionality that does not work as intended/expected
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants