Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
cinqmilleans opened this issue May 16, 2022 · 10 comments
Labels
good first issue Good issue for new contributors to get started with

Comments

@cinqmilleans
Copy link
Contributor

cinqmilleans commented May 16, 2022

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
@yaira2
Copy link
Member

yaira2 commented May 16, 2022

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

@yaira2 yaira2 added triage approved good first issue Good issue for new contributors to get started with labels May 16, 2022
@btomblinson
Copy link
Contributor

@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

@yaira2
Copy link
Member

yaira2 commented Sep 4, 2022

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

@btomblinson
Copy link
Contributor

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

@yaira2
Copy link
Member

yaira2 commented Sep 4, 2022

Correct

@yaira2 yaira2 changed the title Highlight whether the Recycle Bin is empty or not. Use different icon to indicate whether the Recycle Bin is empty or not. Sep 8, 2022
@Josh65-2201 Josh65-2201 changed the title Use different icon to indicate whether the Recycle Bin is empty or not. Feature: Use different icon to indicate whether the Recycle Bin is empty or not. Sep 8, 2022
@QuaintMako
Copy link
Contributor

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.

@thebarshablog
Copy link

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

@QuaintMako
Copy link
Contributor

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

@thebarshablog
Copy link

Appreciate the update @QuaintMako

@yaira2
Copy link
Member

yaira2 commented Jan 3, 2023

Merging with #9946

@yaira2 yaira2 closed this as completed Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good issue for new contributors to get started with
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants