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(core.emptyview): add name to emptyview #1569

Merged
merged 2 commits into from
Oct 22, 2024

Conversation

takase1121
Copy link
Member

This has always been a nit for me; When plugins fail to load it always shows --- for EmptyView.
This PR changes it to Get Started. Maybe there are more suitable names, but I think this one's just fine.

As a side effect, it also says Get Started when nothing is opened now. Perhaps it should return the current project name, etc. but I think this looks okay.
image

@Guldoman
Copy link
Member

Guldoman commented Jul 23, 2023

Maybe there are more suitable names, but I think this one's just fine.

If we're not to show it, I'd just call it Empty View.

When plugins fail to load it always shows --- for EmptyView.

We should probably make workspace loading more robust, and relayout if something fails to load.

@takase1121
Copy link
Member Author

takase1121 commented Jul 24, 2023

If we're not to show it, I'd just call it Empty View.

I don't really agree with this. I think the name should be end user friendly, and "Empty View" doesn't really describe the view. The View is a landing page for the editor, so "Get Started" would be more appropriate.

We should probably make workspace loading more robust, and relayout if something fails to load.

I agree, but this is still relevant since I think most views should have names.

@Guldoman
Copy link
Member

If we're not to show it, I'd just call it Empty View.

I don't really agree with this. I think the name should be end user friendly, and "Empty View" doesn't really describe the view. The View is a landing page for the editor, so "Get Started" would be more appropriate.

I mean my point was about it not being shown, so users would never see it.

@takase1121 takase1121 closed this Jul 24, 2023
@takase1121 takase1121 reopened this Jul 24, 2023
@Guldoman Guldoman merged commit b9e9041 into lite-xl:master Oct 22, 2024
Guldoman pushed a commit to Guldoman/lite-xl that referenced this pull request Oct 22, 2024
* feat(core.emptyview): add name to emptyview

* fix(core.emptyview): set filename to empty
takase1121 added a commit to takase1121/lite-xl that referenced this pull request Nov 25, 2024
* feat(core.emptyview): add name to emptyview

* fix(core.emptyview): set filename to empty
takase1121 added a commit that referenced this pull request Nov 27, 2024
* feat(core.emptyview): add name to emptyview

* fix(core.emptyview): set filename to empty
takase1121 added a commit that referenced this pull request Nov 27, 2024
* feat(core.emptyview): add name to emptyview

* fix(core.emptyview): set filename to empty
takase1121 added a commit that referenced this pull request Nov 27, 2024
* feat(core.emptyview): add name to emptyview

* fix(core.emptyview): set filename to empty
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants