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

Workspaces module rewrite #748

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Workspaces module rewrite #748

wants to merge 2 commits into from

Conversation

JakeStanger
Copy link
Owner

@JakeStanger JakeStanger commented Oct 18, 2024

fix(workspaces): rewrite module to fix several small issues

Rewrites the module code to be better structured, in a similar pattern to the launcher. The code is now more robust and more maintainable, yay!

Fixes #705

Fixes an issue with moving favourite workspaces.

Fixes an issue with workspace visible state being incorrect.

Fixes an issue where the inactive class looked at hidden instead of closed favourites.

@JakeStanger JakeStanger force-pushed the fix/workspaces branch 3 times, most recently from 0d6a36f to 7b53deb Compare October 19, 2024 22:39
@JakeStanger
Copy link
Owner Author

No idea why the build is failing, seems GH have messed up their task runner

Rewrites the module code to be better structured, in a similar pattern to the launcher. The code is now more robust and more maintainable, yay!

Fixes #705

Fixes an issue with moving favourite workspaces.

Fixes an issue with workspace visible state being incorrect.

Fixes an issue where the `inactive` class looked at hidden instead of closed favourites.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant