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

Some issues in right sidebar #680

Closed
1 of 3 tasks
jancborchardt opened this issue Oct 22, 2019 · 2 comments
Closed
1 of 3 tasks

Some issues in right sidebar #680

jancborchardt opened this issue Oct 22, 2019 · 2 comments
Labels
1. to develop Accepted and waiting to be taken care of design Design, UX, interface and interaction design enhancement New feature or request

Comments

@jancborchardt
Copy link
Contributor

jancborchardt commented Oct 22, 2019

Initially opened these in Calendar as part of Design polishing of right sidebar nextcloud/calendar#1576 but @georgehrke asked me to open it here:

  • Event title seems rather small and un-emphasized. In the event popover it’s 20 px, here only 16. We also played around with bolding it @ma12-co @skjnldsv, which improves this a lot?
  • The menu button by default has a grey circle, which is too much emphasis. This should only happen on hover or focus.
  • The "x days ago" subline is not a heading so should not be h4 but simply a p

What do you think?

@jancborchardt jancborchardt added enhancement New feature or request 1. to develop Accepted and waiting to be taken care of design Design, UX, interface and interaction design labels Oct 22, 2019
@skjnldsv
Copy link
Contributor

skjnldsv commented Oct 23, 2019

  • The menu button by default has a grey circle, which is too much emphasis. This should only happen on hover or focus.

You asked me to do it ^^
#340 (comment)
I think it's good for now, was very poorly noticeable before and it's quite an important part of the sidebar :)

  • Event title seems rather small and un-emphasized. In the event popover it’s 20 px, here only 16. We also played around with bolding it @ma12-co @skjnldsv, which improves this a lot?
  • The "x days ago" subline is not a heading so should not be h4 but simply a p

Sure! :)
Have a look your self and post some screenshots ;)

@raimund-schluessler
Copy link
Contributor

I think most of this is fixed / outdated. If you find some glitches still persist, please open a separate issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of design Design, UX, interface and interaction design enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants