desktopfolder icon indicating copy to clipboard operation
desktopfolder copied to clipboard

Locked panels could be deleted!

Open saidbakr opened this issue 5 years ago • 5 comments

Description

When a panel is set to be Locked, the user able to delete it using the trash icon appeared on its title bar.

Steps to reproduce

  1. Create a panel on the desktop.
  2. Go to the panel properties.
  3. Activate the Lock panel property under Behavior .
  4. Press Close button.
  5. Mouse over the panel title. The trash icon of delete the panel is appeared.
  6. Move the cursor towards the trash icon and left click over it.

Expected result

In step 5, the trash icon should not be shown, because the panel has been locked and Lock property should protect it from delete beside move.

Actual result

The panel is deleted.

System Information

  • OS: elementary OS 5.1 Hera
  • Desktop Folder version: 1.1.1

saidbakr avatar Dec 10 '19 16:12 saidbakr

@saidbakr Locking a panel Prevents it from being moved not deleted and so the program is producing expected behaviour in your case. If you want the ability to prevent deleting then you can file an feature request.

Djcoper avatar Dec 12 '19 13:12 Djcoper

@saidbakr Locking a panel Prevents it from being moved not deleted and so the program is producing expected behaviour in your case. If you want the ability to prevent deleting then you can file an feature request.

Lock means lock! Like we found in KDE desktop widgets for example. Logically, move to trash is a kind of moving!

saidbakr avatar Dec 13 '19 12:12 saidbakr

Thanks @saidbakr for reporting. As @Djcoper comments, locking in desktopfolders just means you can't move the panel. I will tag this as a feature request, As this seem to be a new feature added to the current lock functionality. I would like to collect some users feedback on this proposal behavioral feature associated with the lock funciontality. If they feel it is interesting or not.

spheras avatar Jan 02 '20 09:01 spheras

Lock icons and Lock panels I guess could be slightly confusing when coming from other environments. I would be interested to know if KDE and others use the phrase "Pin" i.e. pin to the desktop to make the 'widget' not moveable.

fossfreedom avatar Jan 03 '20 00:01 fossfreedom

Thanks @saidbakr for reporting. As @Djcoper comments, locking in desktopfolders just means you can't move the panel. I will tag this as a feature request, As this seem to be a new feature added to the current lock functionality. I would like to collect some users feedback on this proposal behavioral feature associated with the lock funciontality. If they feel it is interesting or not.

Here, I'm about a logical concept. Lock means protection. Moving to trash is a kind of move too. Additionally, if possible too, the deleted panel should be found in the trash and the user should be able to restore it if he/she mistakenly deleted it.

saidbakr avatar Jan 03 '20 12:01 saidbakr