Skip to content

Conversation

@jancborchardt
Copy link
Member

This will eventually be fixed when we use the AppContentListItem component, cc @ma12-co nextcloud-libraries/nextcloud-vue#616 (This is also essentially a mockup of how this should look like in Mail)

Before:

  • Avatar small and not vertically centered
  • Second row is too low and falls off
  • Action icon hover/focus feedback is outside of element

Mail messages before

Now:

  • Nice vertical alignment
  • Basically same layout as list in Contacts
  • Menu hover/focus feedback fits into entry
  • The only thing a bit off is that timestamp text and icon don’t right-align, but that’s something for another time, or possibly not fixable since we need the clickable area.
    mail list layout new

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@ChristophWurst ChristophWurst merged commit 5657caf into master Oct 17, 2019
@ChristophWurst ChristophWurst deleted the design/message-list branch October 17, 2019 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants