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

Upstreaming budgie-trash-applet #355

Merged
merged 12 commits into from
Jul 6, 2023
Merged

Upstreaming budgie-trash-applet #355

merged 12 commits into from
Jul 6, 2023

Conversation

EbonJaeger
Copy link
Member

Description

This moves my Budgie Trash Applet into the Budgie Desktop source tree.

As a part of this, the applet has been redesigned and rearchitected to better fit within Budgie Desktop:

  • The header now looks similar to the in-progress Bluetooth indicator and network applet
  • Buttons to restore items or empty the trash bin have been added under the header
  • The concept of separate trash bins has been removed; there is just one global trash bin
  • The timestamp when a file was trashed is now shown under the file's name
  • Settings button has been moved to the header

In addition to the visual changes, some of the UX have been changed. The restore button no longer just restores all items in the trash; it only restores selected items. This makes it easier to restore just one file, or to quickly restore several files at once.

Also, applet settings have been changed to be per-instance, so different instances of the applet on your panels will all have their own settings, just in case you actually want that.

Lastly, upstreaming the applet more easily allows the applet to be translated into all the wonderful languages spoken by our community. Thank you, translators, for your valuable contributions! :)

Ref #28

Submitter Checklist

  • Squashed commits with git rebase -i (if needed)
  • Built budgie-desktop and verified that the patch worked (if needed)

@EbonJaeger EbonJaeger added the enhancement New feature or request label Apr 19, 2023
@EbonJaeger
Copy link
Member Author

Speaking of translations, I entirely forgot to write that code before making the PR. Derp.

@serebit
Copy link
Member

serebit commented Apr 19, 2023

CI failed—you're linking against a system header for popover.h instead of the one in the source tree.

@EbonJaeger
Copy link
Member Author

CI failed—you're linking against a system header for popover.h instead of the one in the source tree.

Derp, another thing I overlooked.

@serebit serebit self-requested a review April 22, 2023 14:55
Copy link
Member

@serebit serebit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

EbonJaeger and others added 12 commits July 7, 2023 02:38
Also renames some files to avoid conflicts.

Signed-off-by: Evan Maddock <[email protected]>
… drives

I went to all the trouble to store a consistent and correct URI for each trash item just to not use it. Derp.

Signed-off-by: Evan Maddock <[email protected]>
@JoshStrobl JoshStrobl self-requested a review July 6, 2023 23:39
Copy link
Member

@JoshStrobl JoshStrobl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@JoshStrobl JoshStrobl merged commit e11307d into main Jul 6, 2023
1 check passed
@JoshStrobl JoshStrobl deleted the upstreaming-trash-applet branch July 6, 2023 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants