Bug 357763 - clipboard widget delete history button
Summary: clipboard widget delete history button
Status: RESOLVED INTENTIONAL
Alias: None
Product: klipper
Classification: Unmaintained
Component: plasma-widget (other bugs)
Version First Reported In: unspecified
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Martin Flöser
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-01-09 22:12 UTC by andreas
Modified: 2021-04-01 19:19 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:
kainz.a: Usability+


Attachments
main ui (40.28 KB, image/png)
2016-01-09 22:14 UTC, andreas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description andreas 2016-01-09 22:12:08 UTC
in the main UI there is right to the search an delete button but in my point of view delete the history could be e.g. in the preferences of the plasmoid but should not be part of the MAIN ui. the standard setting has 7 clipboard entries and you can delete every single entry so you have to click 7times the delete button and the main UI is cleaner.

Reproducible: Always
Comment 1 andreas 2016-01-09 22:14:04 UTC
Created attachment 96549 [details]
main ui
Comment 2 Heiko Tietze 2016-01-10 09:58:02 UTC
I think you agree with the need for a function to delete all items. And this feature might be frequently used. If you care about the position next to the input field, which is usually taken by the clear-input-button, I agree. But take into consideration that the input field and the icon next to it are overloaded with the QR functionality (terrible solution). And if your intention is safety, a confirmation dialog could help. 
However, if you believe that a delete-all function is not that frequently used your suggestion makes sense. Perhaps another menu entry could offer the function. But actually the context menus are not contextual because it comes either from the panel with system tray and panel options or the desktop/folder settings and 'remove this clipboard' (who came up with the idea to delete a clipboard?!). All context menus need a context section for the plasmoid itself and another for the parent, clearly separated of course. 
It's conceptual broken on many places.
Comment 3 Kai Uwe Broulik 2016-01-10 10:40:42 UTC
> But take into consideration that the input field and the icon next to it are overloaded with the QR functionality

What?

> 'remove this clipboard' (who came up with the idea to delete a clipboard?!)

Remove this clipboard widget.
Comment 4 Heiko Tietze 2016-01-10 11:04:09 UTC
(In reply to Kai Uwe Broulik from comment #3)
> > But take into consideration that the input field and the icon next to it are overloaded with the QR functionality
> What?
Click the barcode icon of an entry: the input becomes a button to go back and the clear all icon is replaced by a menu with QR types.

> > 'remove this clipboard' (who came up with the idea to delete a clipboard?!)
> Remove this clipboard widget.
Sure, that's how it should be called.
Comment 5 Kai Uwe Broulik 2016-01-10 11:44:21 UTC
> the input becomes a button to go back and the clear all icon is replaced by a menu with QR types.

So? If you go to a different category in System Settings you also get a different set of controls in the toolbar. You get a back button and the search field disappears.
Comment 6 andreas 2016-01-10 12:01:18 UTC
as I don't use the QR stuf and the UX should be simple by default, powerfull when needed I would prefere to have an option for show QR icon and show delete all history icon. So if the user want to use the QR stuff it can be turn on.
For me the QR stuff is an gimix like look how cool I'm what's ok but should not overload the UX by default.
Comment 7 Kai Uwe Broulik 2016-01-10 16:19:06 UTC
Overload? There's one additional button for QR... we shouldn't introduce options for every corner case
Comment 8 andreas 2016-01-10 17:10:08 UTC
OK and what about the delete history icon. I only know what happens when I click in the icon.
Comment 9 andreas 2016-06-14 06:51:24 UTC
Should I close this bug cause in general I only have a problem with the delete all button cause this button in very popular in the UI and dangerous. e.g. for web browsers there is an delete history for the last 24 hours, ... by default this applet had 7 items for 7 items I don't need an delete all button and when you have e.g. > 10 items do you really want to delete all items at once?
Comment 10 Thomas Pfeiffer 2016-06-18 18:36:45 UTC
I agree that the position of the button does not make much sense semantically because it visually associates it with the search field, which it isn't.
That said, I'm not really sure where it should be placed instead.
It could be positioned before or after the list, but then of course one less entry would be visible. This might be an acceptable price to pay for the improved clarity, however (and we could then use a text label instead of the icon).

And I'm against introducing config options just for showing or hiding UI elements.

What would make sense to me at some point is to offer an alternative "Simple clipboard" Plasmoid which does not show any of the actions (because they are all quite advanced usecases).
Comment 11 Nate Graham 2021-04-01 19:19:54 UTC
Not sure this makes sense really, sorry. If you do want to clear the history, having to do it for every item is not fun, and having to go to the settings window is annoying and unintuitive (clearing history isn't a setting to be adjusted. In Plasma 5.21, the clear history item is now in the header with other relevant actions so it is no longer visually associated with the search field. So I think this can be considered no longer an issue. :)