Skip to content

Add togglable Places & Programs#821

Merged
dremin merged 1 commit intocairoshell:masterfrom
TheXorog:master
Jul 17, 2024
Merged

Add togglable Places & Programs#821
dremin merged 1 commit intocairoshell:masterfrom
TheXorog:master

Conversation

@TheXorog
Copy link
Contributor

I've added the ability to toggle the Places and Programs option in the menu bar. I'd like to disable it as i don't really use it anyways and it takes up space for my way too big tray.

I'm quite unfamiliar with the code base and just went off of how everything else, i think, is done.
For translations i'm just piggy backing off of the existing translations for the 2 options.

If i made a mistake or it's not up to standard then let me know.

@dremin dremin self-requested a review July 17, 2024 00:26
Copy link
Collaborator

@dremin dremin left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @TheXorog! It looks great, just a few minor changes to request before merging.

@TheXorog
Copy link
Contributor Author

I think i just messed up the branch?

@TheXorog
Copy link
Contributor Author

TheXorog commented Jul 17, 2024

I quickly re-implemented it and force pushed it. Sorry about that 😅

Since the setup places method is gone, i instead put the updating of visibility into setupChildren. I hope that's fine.

Copy link
Collaborator

@dremin dremin left a comment

Choose a reason for hiding this comment

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

Thanks! 🎉

@dremin dremin merged commit f011b43 into cairoshell:master Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants