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

app-navigation-toggle is annoying #1414

Closed
dartcafe opened this issue Sep 24, 2020 · 2 comments
Closed

app-navigation-toggle is annoying #1414

dartcafe opened this issue Sep 24, 2020 · 2 comments
Labels
design Design, UX, interface and interaction design feature: app-navigation Related to the app-navigation component

Comments

@dartcafe
Copy link
Contributor

The toggle is placed right near the navigation and forces app developers find their own way to avoid visual issues.
Examples:
Polls:
grafik
Calendar:
grafik
Talk:
grafik
Deck:
grafik

Proposal:
Give the navigation more space, move the toggle inside the navigation and leave 44px of the closed navigation visible:
Open state
grafik
Closed state
grafik

In the result, It costs just 44px of the app container, but the app-container is again a fully usable square and the toggle is still usable.

CSS changes for mockup:

.app-navigation {
	width: 344px !important;
	padding-right: 44px !important;
}

.app-navigation-toggle {
	margin-right: initial !important;
}

.app-navigation--close {
	margin-left: -300px  !important;
	transform: translateX(-300px)  !important;

	.app-navigation-entry {
		width: initial !important;
		margin-right: 44px;
	}
}

Comments? Opinions?

@dartcafe dartcafe added feature: app-navigation Related to the app-navigation component design Design, UX, interface and interaction design labels Sep 24, 2020
@jotoeri
Copy link
Contributor

jotoeri commented Nov 22, 2020

@jancborchardt That's something that needs your opinion. ;)

I also thought about this annoying toggle sometimes...

@skjnldsv
Copy link
Contributor

We have had this discussion already.
See the full disclosure here https://github.com/nextcloud/server/issues/16934

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design Design, UX, interface and interaction design feature: app-navigation Related to the app-navigation component
Projects
None yet
Development

No branches or pull requests

3 participants