feat: add reply action to envelope list hover actions#13310
feat: add reply action to envelope list hover actions#13310steven-mpawulo wants to merge 2 commits into
Conversation
Signed-off-by: steven-mpawulo <stevenmpawulo@gmail.com>
|
Thank you Please post before/after screenshots for this visual change |
|
Alright |
|
Needs input from @nimishavijay. unsure if Reply should be one of the big primary actions or a regular one below. |
|
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 |
Signed-off-by: steven-mpawulo <stevenmpawulo@gmail.com>
|
Exactly, looks good now :) |
|
Awesome |
| <ActionButton | ||
| v-if="withReply" | ||
| class="action--primary" | ||
| :close-after-click="true" | ||
| @click.prevent="onReply()"> | ||
| <template #icon> | ||
| <Reply :size="24" /> | ||
| </template> | ||
| {{ t('mail', 'Reply') }} | ||
| </ActionButton> |
There was a problem hiding this comment.
To make it correct and consistent, please mirror the logic from
mail/src/components/ThreadEnvelope.vue
Lines 148 to 173 in 07ffc03
We should also handle the multiple recipients case properly in this actions menu
There was a problem hiding this comment.
Let me emulate



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