nextcloud-vue icon indicating copy to clipboard operation
nextcloud-vue copied to clipboard

Mobile: AppNavigation not closing on clicking an entry which changes the content view

Open jancborchardt opened this issue 3 years ago • 5 comments

This is a regression present in all apps with the Vue nav toggle: They do not close the navigation bar when clicking on an entry which changes the content view, as would be expected and like mobile apps do too:

Mail app not closing nav Photos app not closing nav
Mail menu not closing on clicking folder Photos nav not closing

This happens also in lots of other apps, like Talk, Contacts, etc.

Compare this with the behavior of Files and Settings, which is as expected:

Files app closing nav Settings closing nav
Files app nav closing Menu closing in settings

jancborchardt avatar Feb 24 '22 12:02 jancborchardt

Duplicate of #433. But since you have images, let's keep this issue here instead of the older one.

raimund-schluessler avatar Feb 25 '22 09:02 raimund-schluessler

@marcoambrosini @szaimen is this something you’d like to pick up? Possibly small fix for quite the annoyance on mobile. :)

jancborchardt avatar Jan 03 '24 13:01 jancborchardt

This happens also in lots of other apps, like Talk, Contacts, etc.

Note, some apps like Talk uses NcListItem, not NcAppNavigationItem for links. Theoretically, an app may have any action from NcAppnavigation, that changes navigation (e.g. creating a new conversation in Talk).

ShGKme avatar Jan 03 '24 15:01 ShGKme

I had a quick look and it looks like the solution is more complicated than a simple css change. So moving to the files team.

szaimen avatar Jan 09 '24 11:01 szaimen

I had a quick look and it looks like the solution is more complicated than a simple css change. So moving to the files team.

As an option for the first time - leave it for each individual app.

ShGKme avatar Jan 09 '24 13:01 ShGKme