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

Feat/restyle multiview layout #32

Merged
merged 3 commits into from
Oct 10, 2024
Merged

Conversation

malmen237
Copy link

What does this do?

More universal

This layout-config is seated at the top of the modal and does not connect to any specific multiview-setting.
Screenshot 2024-10-08 at 13 55 24

The Layout-modal

You begin by choosing a layout to change or just to start fresh from a preset.

Also fixed a bug that did not allow the same layout-names to exist for different productions and if you wrote a layout-name that belonged to a different production it would be found and it's layout added into view.
Screenshot 2024-10-08 at 13 55 41

Some extras

Also added the review-updates requested by Ateliere in the PR. Which mainly is removal of commented-out code and the additional of some empty lines.

@malmen237 malmen237 force-pushed the feat/restyle-multiview-layout branch from c540e10 to 1c80347 Compare October 10, 2024 08:41
@malmen237 malmen237 merged commit b002579 into main Oct 10, 2024
4 checks passed
@malmen237 malmen237 deleted the feat/restyle-multiview-layout branch October 10, 2024 08:43
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.

3 participants