Skip to content
This repository has been archived by the owner on Mar 25, 2023. It is now read-only.

Add a "delete mail" option #161

Closed
pprw opened this issue Apr 6, 2017 · 6 comments
Closed

Add a "delete mail" option #161

pprw opened this issue Apr 6, 2017 · 6 comments

Comments

@pprw
Copy link

pprw commented Apr 6, 2017

This is a request feature (for thunderbird). I would be nice to be able to delete an email directly from the notification. If you see a new email which is clearly not interesting for you, I would like to delete it straight with a button close do "open" and "mark as read".

bye

@mkiol
Copy link
Owner

mkiol commented Apr 7, 2017

Nice idea. I will add it to the next release.

@pprw
Copy link
Author

pprw commented Apr 7, 2017 via email

@mkiol
Copy link
Owner

mkiol commented Apr 15, 2017

Please check the latest gnotifier-1.9.9-preX.xpi package.

What I added:

  • In the settings, "Default click action for E-mail notifications" has new "Delete" option. If you are on Windows, this option sets "click" action.
  • On Linux, every new e-mail notification has 3 buttons: "Open", "Mark as read", "Delete".
  • Click on "Delete" marks message as read and moves it to the Trash folder

Let me know what you think.

@tooomm
Copy link
Contributor

tooomm commented Apr 18, 2017

Is it possible to have for example a right click menu option on Windows to delete messages as well right from the notification?

@mkiol
Copy link
Owner

mkiol commented Apr 23, 2017

@tooomm I'm forwarding your question to @kabili207 (He is an author of GNotifier's windows integration).

Questions:

  • Is it possible to have right click action on windows notification?
  • Is it possible to have buttons on notification (something similar to what is possible on Linux)?

kde_notification

@kabili207
Copy link
Contributor

Unfortunately, adding buttons can't be done without switching from the Windows 8 API to the Windows 10 API, which is problematic for all the reasons I've outlined in this comment on issue 65. Right-click actions aren't possible in either API.

@mkiol mkiol closed this as completed May 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants