Skip to content

[stable8] fix(NcAppNavigationItem): close mobile navigation on router-link click - #8760

Open
backportbot[bot] wants to merge 1 commit into
stable8from
backport/8732/stable8
Open

backportbot[bot] wants to merge 1 commit into
stable8from
backport/8732/stable8

Conversation

@backportbot

@backportbot backportbot Bot commented Jul 20, 2026

Copy link
Copy Markdown

Backport of PR #8732

@backportbot backportbot Bot added bug Something isn't working 3. to review Waiting for reviews feature: app-navigation Related to the app-navigation component papercut Annoying recurring issue with possibly simple fix. design Design, UX, interface and interaction design AI assisted This PR contains AI-assisted commits labels Jul 20, 2026
@backportbot backportbot Bot added this to the 8.40.1 milestone Jul 20, 2026
fix #2523

Assisted-by: ClaudeCode:claude-opus-4-8
Signed-off-by: Jan C. Borchardt <925062+jancborchardt@users.noreply.github.com>
@susnux
susnux force-pushed the backport/8732/stable8 branch from 20421a5 to 6ba90c4 Compare August 24, 2026 13:25

import { subscribe, unsubscribe } from '@nextcloud/event-bus'
import { mount } from '@vue/test-utils'
import { describe, expect, it, vi } from 'vitest'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you need to adapt this for jest on this branch @jancborchardt

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ping @jancborchardt

could your adjust this PR so we can align features on both branches?

@susnux susnux modified the milestones: 8.41.0, 8.42.0 Sep 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews AI assisted This PR contains AI-assisted commits bug Something isn't working design Design, UX, interface and interaction design feature: app-navigation Related to the app-navigation component papercut Annoying recurring issue with possibly simple fix.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants