napari / napari-sphinx-theme

https://napari.org/napari-sphinx-theme
BSD 3-Clause "New" or "Revised" License
4 stars 10 forks source link

Calendar details view improvement suggestions #138

Closed jni closed 7 months ago

jni commented 7 months ago

CC @melissawm @psobolewskiPhD

Just implementing the easiest one of the above would be enough for a first pass. Right now, there's a lot of mousing back and forth between the calendar and the X to close the event!

psobolewskiPhD commented 7 months ago

The first 2 are priorities, IMO. For me the 3rd is take it or leave it. I actually rather like the current less in the way implementation on a larger monitor--I can see the calendar and the event. On a small screen it's basically centered anyways.

Super minor thing: the dates (e.g. 17) and days of the week "light up" on hover, like they should do something when you click, but they don't.

BTW: for me it shows all times America New York is that for everyone? I think if it could be aware of time zones it would be amazing.

melissawm commented 7 months ago

It is - that should be your time zone. Is it not? Maybe that's a browser thing again?

Screenshot_2024-01-18-17-45-00-594_org.mozilla.firefox-edit.jpg

psobolewskiPhD commented 7 months ago

Nono, sorry I wasn't clear -- it was showing me my timezone! I wasn't sure if it wasn't if it was showing the same times for everyone (with the message re: timezone) or actually adapting. Looks like it adapts, which is killer! ❤️

melissawm commented 7 months ago

The event popover looks awful on mobile because I hardcoded it to be 20% of the screen - needs to be made responsive 😅

Screenshot_2024-01-18-19-15-28-263_org.mozilla.firefox.jpg

jni commented 7 months ago

Amazing work as always @melissawm! 😍 Man I am so pumped to see this live on the stable site!