Skip to content

feat: allow closing the open message preview - #13912

Open
steven-mpawulo wants to merge 2 commits into
nextcloud:mainfrom
steven-mpawulo:fix/close-mail-preview
Open

steven-mpawulo wants to merge 2 commits into
nextcloud:mainfrom
steven-mpawulo:fix/close-mail-preview

Conversation

@steven-mpawulo

@steven-mpawulo steven-mpawulo commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Fixes #13841

Adds a close icon to the message preview header.
Also makes re-clicking the open envelope close the preview.

Screenshot

image

Signed-off-by: steven-mpawulo <stevenmpawulo@gmail.com>
@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

@kesselb

kesselb commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Hi @steven-mpawulo, thanks a lot for your contribution! We will take a closer look at your pull request soon.

@kesselb

kesselb commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Please remove click again to de-select #13841 (comment)

@kesselb
kesselb requested a review from kra-mo November 6, 2026 17:59

@kra-mo kra-mo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I can't exactly see how much padding there is from the screenshot, but it should be top-right aligned with 8px padding on each side.

@steven-mpawulo

Copy link
Copy Markdown
Collaborator Author

Please remove click again to de-select #13841 (comment)

Hello
Am working on it now

Signed-off-by: steven-mpawulo <stevenmpawulo@gmail.com>

This branch has not been deployed

No deployments
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.

[Feature]: Need a close preview button/option in mail view

3 participants