-
-
Notifications
You must be signed in to change notification settings - Fork 93
Add option to select the default layout #117
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A bit unfortunate, that we have to duplicate the list of layouts, but this is the limitation of the current config system (will be improved with the in the future with the separate KCM (see #94)).
JFYI, you need to fix pipeline errors to proceed :) In particular:
|
Since 2.0 is released, you will have to adopt your merge request accordingly. In particular, the configuration is now done in a separate KCM, so you'll have to write some UI code. |
can i take over? |
Yep. |
I will work on it tomorrow and make a separate PR. |
I'd hope that now that the KCM is there, we'd also be able to have a default layout per screen (as an example I have an ultrawide screen where I use three columns, and a normal 16:9 where I want to use normal tiling) |
Is there any progress on this? This is a really important feature to me. |
Really happy to know about this fork, I Adapted the changes from my pr in the original project.
Summary
Add option to select the default layout.
Test Plan
Preview
Related Issues
Closes #74