CeON / dataverse

Open source research data repository software. ICM dataverse version.
http://dataverse.org
Other
8 stars 4 forks source link

Enable mass notification deletion with GUI #2509

Closed tlewand42 closed 1 month ago

tlewand42 commented 4 months ago

We need some additional elements on Notifications tab in user dashboard. The goal is to enable users to delete multiple notifications at once rather than deleting them one by one (which can be dramatically time-consuming after a prolonged period of suboptimal notification hygiene).

wfenrich commented 4 months ago

Ad. Lazy Option: we should have an intermediary confirmation (possibly a modal window), just to avoid mass destruction caused by an accidental click.

Ad. Greedy Option: if deletion is the only available action, it does not improve things much. If user selects only some (but not all) notifications, the only difference is that s/he has to click a checkbox instead of an "X" on the right side (the one that we have now). We should add an intermediary confirmation for this solution as well.

tlewand42 commented 3 months ago

one majr difference between mass checking and mass deleting is that checking in a checkbox does not trigger page reload (and the deletion does). Page trying to reload upon every deletion generates most of the lag, especially when there are many notification present on the page and the page load time is very long to begin with. Nevertheless, the rest of the argumentation is valid and proper.

wfenrich commented 3 months ago

We go with: