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

Update the readme #793

Closed
wants to merge 19 commits into from
Closed

Update the readme #793

wants to merge 19 commits into from

Conversation

zX3no
Copy link

@zX3no zX3no commented Sep 30, 2021

A few tweaks to the readme.

  • Simplifying the install section
  • Adding some nicer screenshots
  • Adding the troubleshooting section
  • Adding a documention section

I initially had some trouble setting up helix(installing from source). I wanted to clarify the runtime section because it's not required for most platforms.

Some of the changes won't be everyones taste, so I'd like to hear your thoughts on it. I feel like the acylic might be a little too much? Screenshots not tall enough? More purple?

Oh yeah, I was wondering if you need to setup the runtime when installing with homebrew. I'll need to add that to the readme.

If you want to see a live preview.

@zX3no zX3no changed the title update the readme Update the readme Sep 30, 2021
@kirawi
Copy link
Member

kirawi commented Sep 30, 2021

I feel that the troubleshooting section should remain on the wiki so it's easier to edit (no need to create PRs). Not sure how to give a link to it enough exposure in the readme though. Perhaps also add a link to the FAQ?


<div align="center" style="display:inline">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't particularly like the screenshots, they don't use the default theme and the shine distracts away from the editor. We should show it as it looks out of the box. (I think the current screenshot looks "nice" enough but that's because I like my bitnap fonts)

Copy link
Author

@zX3no zX3no Oct 1, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I understand not liking the acyclic, but I think using a theme that is appealing to the majority would help with first impressions(if you care). The Onedark theme is included and seems like a more popular choice.

It's seem like you want to show off features rather than looks, which is hard to do in a screenshot, maybe a few gifs would be a better choice? Like what onivim has on their website.

Up to you really, can change it back if you want.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

using a theme that is appealing to the majority would help with first impressions

I think this a very subjective thing. Editors are usually presented as they are out of the box, and given that I've written both the editor and the theme, that's the theme that's default. See Kakoune for very plain screenshots that do a great job of showcasing basic features.

Copy link
Author

@zX3no zX3no Oct 1, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this a very subjective thing.

Whether a theme is good or not is subjective, however the popularity of a theme is not. One dark for example, has almost 7 million downloads on vscode, there is atleast something quantifiable about that.

As for Kakoune, it looks at least 30 years out of date(the gifs are okay). Which is frustrating because I'm sure the features are good. There's no need to compromise like that, you can have both functionality and aesthetics. Again onivim manages to do this well.


# Features
## ✨ Features
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd drop the emoji too

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of them or just the one?

README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@pickfire
Copy link
Contributor

pickfire commented Oct 1, 2021

The screenshot is hard to read, the translucent background reduces the contrast and hurt my eyes reading it. Is it possible to have an opaque background to be clearer?

@zX3no
Copy link
Author

zX3no commented Oct 1, 2021

Yes, I doubt they will be the final screenshots. Also, here's a more moderate opacity of 85%.

@kirawi
Copy link
Member

kirawi commented Oct 3, 2021

Maybe something for docs/vision.md as well?

@pickfire
Copy link
Contributor

pickfire commented Oct 3, 2021

Yes, I doubt they will be the final screenshots. Also, here's a more moderate opacity of 85%.

Yes, that is readable. But not sure why top left part looks weird, it have a reddish rectangle.

@zX3no zX3no closed this Feb 20, 2022
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.

None yet

4 participants