search

nextcloud / mail

💌 Mail app for Nextcloud
https://apps.nextcloud.com/apps/mail
GNU Affero General Public License v3.0
833 stars 258 forks source link

Faded thread scrolling vs hard message list scrolling #7186

Open ChristophWurst opened 2 years ago

ChristophWurst commented 2 years ago

Steps to reproduce

  1. Open the app
  2. Configure an account
  3. Open a (long) message/thread
  4. Scroll the message list
  5. Scroll the message

Expected behavior

The designers are happy

Actual behavior

Message list scrolling Thread/message scrolling
Bildschirmfoto vom 2022-09-06 15-37-31 Bildschirmfoto vom 2022-09-06 15-37-19

Mail app version

2.0.0 beta 1

Mailserver or service

No response

Operating system

No response

PHP engine version

No response

Web server

No response

Database

No response

Additional info

No response

ChristophWurst commented 2 years ago

^ @jancborchardt @nimishavijay should the New message header also have some kind of fade for the list underneath?

jancborchardt commented 2 years ago

^ @jancborchardt @nimishavijay should the New message header also have some kind of fade for the list underneath?

Yeah, that would indeed be nice! Then it seems less boxy. Would probably be good to do this in the components too? We have a similar issue in the left navigation of Talk with the search bar on top, and in all apps with the settings button in the bottom left. cc @marcoambrosini as well.

marcoambrosini commented 2 years ago

I think we should drop the gradient altogether and use a small grey border for the top of scroll containers. The main reason Is that for a fade that looks good, the fade itself needs to take a considerable amount of space. And even then, this is not a widely adopted pattern and might be confusing.

jancborchardt commented 2 years ago

I’m fine either way. But it should be consistent, and needs to look good in the newly styled sidebar as well as on color-main-background, both light and dark theme.