Files icon indicating copy to clipboard operation
Files copied to clipboard

Feature: Use different icon to indicate whether the Recycle Bin is empty or not.

Open cinqmilleans opened this issue 3 years ago • 5 comments
trafficstars

What's the Problem?

To know if the Recycle Bin is empty or not, you have to open it, or open its contextual menu.

Solution/Idea

You can add information about the item from the Recycle Bin to favorites. This can be done in many ways: different icon, bold text, number of items in parentheses when there are any, or any different style.

Requirements

  • Use different icon to indicate whether recycle bin is empty or full

cinqmilleans avatar May 16 '22 15:05 cinqmilleans

Thank you for your feedback, we're currently tracking this in #1089. Keeping this open as well.

yaira2 avatar May 16 '22 15:05 yaira2

@yaichenbaum has this been superseded by #1089 or is it still open for pull requests? To keep things clean just thinking making it bold under the favorites when theres items in the recycle bin, adding the number of items in parenthesis would look out of place

btomblinson avatar Sep 04 '22 01:09 btomblinson

#1089 is just to group these issues together, the proper solution is to use different icons to highlight the status.

yaira2 avatar Sep 04 '22 01:09 yaira2

Understood, so changing the text or text weight is not the recommended approach?

btomblinson avatar Sep 04 '22 01:09 btomblinson

Correct

yaira2 avatar Sep 04 '22 02:09 yaira2

A PR that was implementing this issue has been made, but removed for limitation issues regarding the current implementation.

A few things to keep in mind for the next person implementing the feature:

  • There are several scenarios where the icon needs to be reevaluated:

When files are deleted into the recycle bin When files are being restored in the recycle bin When files are being deleted from the recycle bin When the recycle bin is entirely restored or empty

  • The icons should be taken from Constants.ImageRes.
  • The indexes for the Constants.ImageRes are namely public const int RecycleBin = 54 and public const int EmptyRecycleBin = 55.

QuaintMako avatar Oct 27 '22 08:10 QuaintMako

Is anybody working on this issue? If not, would be glad to solve it.

thebarshablog avatar Dec 29 '22 05:12 thebarshablog

This issue if free as far as I know, feel free to look into it!

QuaintMako avatar Dec 29 '22 08:12 QuaintMako

Appreciate the update @QuaintMako

thebarshablog avatar Dec 29 '22 09:12 thebarshablog

Merging with #9946

yaira2 avatar Jan 03 '23 16:01 yaira2