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.

Sign up for GitHub

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

Jump to bottom

The default layout is not loaded. #1431

Closed
a-kenji opened this issue May 23, 2022 · 3 comments · Fixed by #1432
Closed

The default layout is not loaded. #1431

a-kenji opened this issue May 23, 2022 · 3 comments · Fixed by #1432
Labels
config Improvements to the configuration system layout issues related to the layout system suspected bug

Comments

@a-kenji
Copy link
Contributor

a-kenji commented May 23, 2022

The default layout is not loaded on the current main branch.

zellij --version: zellij 0.30.0

To reproduce:

  • create a default.yaml inside the ZELLIJ_LAYOUT directory.
  • run zellij
@a-kenji a-kenji added suspected bug config Improvements to the configuration system layout issues related to the layout system labels May 23, 2022
@imsnif
Copy link
Member

imsnif commented May 23, 2022

How does it work then? Can you pinpoint where this issue comes from?

@a-kenji
Copy link
Contributor Author

a-kenji commented May 23, 2022

It uses the internal default layout as opposed to the configured one.

Can you pinpoint where this issue comes from?

Not yet, I wanted to document it first.
I hope I'll get some time this evening to look into it.

@jaeheonji
Copy link
Member

jaeheonji commented May 23, 2022 •
edited
Loading

@a-kenji

I think it seems to be related to #1413. I'll check it out too.

EDIT: In the latest update, I think I missed one condition. I found the problem and resovled it. If you have the time, I would be grateful if you could simply look it through.

@jaeheonji jaeheonji mentioned this issue May 23, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
config Improvements to the configuration system layout issues related to the layout system suspected bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: add checking for missing extensions
3 participants
@imsnif @jaeheonji @a-kenji