Skip to content

feat: add reply action to envelope list hover actions#13310

Open
steven-mpawulo wants to merge 2 commits into
nextcloud:mainfrom
steven-mpawulo:feature/12210-allow-reply-from-email-listing
Open

feat: add reply action to envelope list hover actions#13310
steven-mpawulo wants to merge 2 commits into
nextcloud:mainfrom
steven-mpawulo:feature/12210-allow-reply-from-email-listing

Conversation

@steven-mpawulo

Copy link
Copy Markdown
Collaborator

Adds a Reply quick-action to the hover menu on the envelope list, next to Favorite/Read/Important.
Fixes #12210

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

Copy link
Copy Markdown
Member

Thank you

Please post before/after screenshots for this visual change

@steven-mpawulo

Copy link
Copy Markdown
Collaborator Author

Alright
Will do.

@steven-mpawulo

Copy link
Copy Markdown
Collaborator Author

Thank you

Please post before/after screenshots for this visual change

Screenshot 2026-07-17 at 3 49 36 PM Screenshot 2026-07-17 at 3 52 53 PM

@ChristophWurst

Copy link
Copy Markdown
Member

Needs input from @nimishavijay. unsure if Reply should be one of the big primary actions or a regular one below.

@steven-mpawulo steven-mpawulo self-assigned this Jul 17, 2026
@nimishavijay

Copy link
Copy Markdown
Member

Yes, I would steer clear of adding a fourth item to the top row. Reply can be the first item in the list of actions below the date.

@steven-mpawulo

Copy link
Copy Markdown
Collaborator Author

Yes, I would steer clear of adding a fourth item to the top row. Reply can be the first item in the list of actions below the date.

Something like this @nimishavijay
Screenshot 2026-07-23 at 9 27 30 AM

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

Copy link
Copy Markdown
Member

Exactly, looks good now :)

@steven-mpawulo

Copy link
Copy Markdown
Collaborator Author

Awesome

Comment on lines +251 to +260
<ActionButton
v-if="withReply"
class="action--primary"
:close-after-click="true"
@click.prevent="onReply()">
<template #icon>
<Reply :size="24" />
</template>
{{ t('mail', 'Reply') }}
</ActionButton>

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.

To make it correct and consistent, please mirror the logic from

<NcActionButton
:close-after-click="true"
@click="onReply('', false)">
<template #icon>
<ReplyAllIcon
v-if="hasMultipleRecipients"
:title="t('mail', 'Reply all')"
:size="20" />
<ReplyIcon
v-else
:title="t('mail', 'Reply')"
:size="20" />
</template>
{{ t('mail', 'Reply') }}
</NcActionButton>
<NcActionButton
v-if="hasMultipleRecipients"
:close-after-click="true"
@click="onReply('', false, true)">
<template #icon>
<ReplyIcon
:title="t('mail', 'Reply to sender only')"
:size="20" />
</template>
{{ t('mail', 'Reply to sender only') }}
</NcActionButton>
.

We should also handle the multiple recipients case properly in this actions menu

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Let me emulate

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.

Allow/add reply from email listing

3 participants