This component adds in the ability to mark a notification as read, or to delete a notification without leaving the page that you are on. No more having to click the various notifications to mark them as read. This utility grants the user the ability to delete their own notifications (and thus help keep your database clean). Modeled off of how Facebook allows you to just mark a notification as read, or to delete those annoying "this person liked your comment" notifications.
Suggestions for improvement, or bug fixes (or simply finding one and letting me know) are always welcome!
This is the initial release of the component, and I have tested it on a live site.
This is an example of notifications that don't get deleted "out of the box" with this component. Have not attempted to bug fix this yet, I imagine it's just because Videos/group/event haven't been tagged as an item to receive the action button. Thanks for your hard work!
Just tried 184.108.40.206, and the logic appears still strange to me. This screenshot shows two groupies of mine:
Nevertheless, I get the same 'Mark Read' link in both cases - I would expect to get it with Botox, only.
As seen in the image below, only the first notification goes through the templating process, as defined in view.php. (I copied over the original into my component so that I could modify it.)
@Hugo -- Thank you! I added it in.
@Michael -- thank you for the direction! This task, is proving to be difficult. Mainly, because I realized that I had to put the controls into the block to be removed. I surrounded it by a <div id="notif-$guid"> so that I can remove all of it (rather than having the notification gone, and the controls to delete / mark read sitting there pointing at nothing). <-- that mostly works, because some notifications don't get templated?! (don't get it).
and it doesn't on the fly refresh the all notifications page content.
jquery will be your friend ... you would use something like
Study OssnComment's delete action + accompanying script and you'll get an idea. :)
Hi Michieal, nice update, all working great.
I added the spanish locale file in the github.
Added the Delete All Reactions as a button on the bottom of the Notifications dropdown, next to the See All link.
It's not 100% exactly what I want it to be, but it's very, very close. Ie., it doesn't (yet) have the ability to on the fly refresh just the notifications drop down, and it doesn't on the fly refresh the all notifications page content.
If anyone is willing to help with this, I would be extremely happy!
I would do a pull request, as I can merge it right in.
Because I haven't figured out how to make use of the "friends" on here.
Hi Michiel, how i can send you the locale file? i added you here.
Or i fork your repository and i open a pull request?
I was going off of the comment in the code, that said that the section was to delete the notification as a failsafe, when encountering a broken notification. But, I 100% agree with you that it doesn't remove the notification and you will get directed to a blank page or a 404.
I think, that it should have some kind of automatic notification "cleaner" for when something like that happens and the system has a broken notification... But the "Delete" link in MarkNotifications will do exactly that, albeit not automatically.