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

[Cosmetic] Star icon and date can go on the second line if width is small #737

Closed
Massedil opened this issue Dec 2, 2022 · 5 comments
Closed

Comments

@Massedil
Copy link

Massedil commented Dec 2, 2022

A picture is worth a thousand words :

  • First email is as expected
  • Second has the date and star in a new line
  • Third has the star in a new line
  • Fourth is as expected

2

  • snappymail v. 2.22.5 (bug was also here in v.2.22.4, at least)
@the-djmaze
Copy link
Owner

Looks like the problem is the date format.
For narrow screens making the format just the day/month/year could be an idea.
Or use 3 lines as now, but better aligned.

Will figure it out as most apps on phones don't show this amount of details.

@Massedil
Copy link
Author

Massedil commented Dec 3, 2022

For narrow screens making the format just the day/month/year could be an idea.

Maybe like k9-mail do?

image5

Or use 3 lines as now, but better aligned.

Yep, also.

Will figure it out as most apps on phones don't show this amount of details.

My first screenshot is from the desktop view. I just reduced the mail list.

Maybe we can also use more of the available space we can see at the left of the date?

the-djmaze pushed a commit that referenced this issue Dec 4, 2022
@the-djmaze
Copy link
Owner

I've made the flex a bit smaller to solve it.

However, i also have this idea (but it currently breaks the selector functionality).
afbeelding

@the-djmaze the-djmaze reopened this Dec 4, 2022
the-djmaze pushed a commit that referenced this issue Dec 5, 2022
@the-djmaze
Copy link
Owner

v2.24 now has the split 😄

@Massedil
Copy link
Author

v2.24 now has the split

I just updated to 2.24 and will test it ! Thanks !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants