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

Mail envelope and titles design is off #10392

Closed
thstyl2000 opened this issue Nov 22, 2024 · 7 comments · Fixed by #10410
Closed

Mail envelope and titles design is off #10392

thstyl2000 opened this issue Nov 22, 2024 · 7 comments · Fixed by #10410

Comments

@thstyl2000
Copy link

Steps to reproduce

  1. Click on a message
  2. Look at the email

Expected behavior

Mail body, Subject and Reply button should be nicely aligned.

Actual behavior

Things currently look like that:

Image

Image

Mail app version

4.0.3

Nextcloud version

30.0.2

Mailserver or service

imap (dovecot)

Operating system

freebsd

PHP engine version

PHP 8.2

Nextcloud memory caching

No response

Web server

Nginx

Database

MariaDB

Additional info

No response

@ChristophWurst
Copy link
Member

Thanks for this report too @thstyl2000! I noticed it on my phone yesterday. This is a regression and not a deliberate design change.

@thstyl2000
Copy link
Author

Most welcome! Glad to be of any help.

@ChristophWurst
Copy link
Member

This was caused by #10268. Some css selectors no longer apply.

@ChristophWurst
Copy link
Member

Fix is at #10410

@github-project-automation github-project-automation bot moved this from 🏗️ In progress to ☑️ Done in 💌 📅 👥 Groupware team Nov 29, 2024
@thstyl2000
Copy link
Author

thstyl2000 commented Dec 6, 2024

Sorry to bump, the Reply button is still (4.0.7) off it now moves up and down depending on the length of the message.

Image

Image

Also still thing that the subject font is way too big. The screenshot in github looks miles better in my opinion...

@thstyl2000
Copy link
Author

Just noticed that the subject used to be aligned with the message text, and now not (that's why it looked so off to me). Should I open a new issue/feature for this and the reply button?

@thstyl2000
Copy link
Author

Still (4.0.9.) an issue when phishing warning is shown:

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ☑️ Done
Development

Successfully merging a pull request may close this issue.

2 participants