-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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
added "lean" prompt theme #1472
Conversation
@belak or @indrajitr could you take a look? |
I'm not sure what our plan is for themes. I'd like to avoid adding too many submodules... however, I like the theme and I agree that there's a certain lack of minimal themes included with prezto (I was considering merging the theme I keep in my dotfiles at some point which is roughly based on the default gentoo prompt). I'm not sure what we'll decide on, but if this doesn't get merged in here, it seems like a great candidate for the contrib repo we've been discussing. |
the main motivation to add this theme was indeed the lack of minimal ones. most of existing themes are too bloated imo. the "pure" is closest to minimal among all, but for some reason it is 2-lines height... anyway, hope you'll merge this one, because some new users would be just scared by all those bells-n-whistles, funny prompt symbols etc while simply looking for zsh-with-prezto power and i'm pretty sure many of them would find "lean" just enough for their needs :) |
I like it 👍 |
@belak any news? hanging pr is a bad pr, let's do something with it |
I'm waiting for one other PR to be reviewed and a discussion on what we want to put in our contrib repo. I haven't forgotten about this PR. |
hey @belak , any news? 2 weeks already |
Unfortunately, same news as before. Still waiting on other people to review #1458 (as it's a pretty important change to module loading and I don't want to break anything) so we can start work on a contrib repo. I'd like to avoid as much confusion as possible, for when we (potentially) move some of the prompts to a separate module in a different repo. Especially for submodules, which have their own problems - I'd like for most external prompts to be included in the contrib repo, rather than core. I do appreciate you following up. EDIT: I've also opened #1483 to start a more official discussion about creating a contrib repo. |
@belak hi, i see that module loading PR was merged (finally). what should i do now? wait for zsh-contrib repo to be created and recreate this PR to that repo instead? |
In this case, I believe there is the problem discussed in this issue #1484 |
This was added to an experimental contrib repo in belak/prezto-contrib@c718baa |
@belak, nice! so now i can use that repo with the recently merged changes in prezto, right? if so, should we cancel this PR? |
Yep, you should be able to clone contrib and enable the I'm still deciding what to do with open PRs which are migrated to contrib. My plan is currently to tag everything which makes it into contrib and go back to close them when the contrib repo is moved to its home. |
huh? the file is in contrib!
README said "By default modules will be loaded from /modules and /contrib." note, the theme loaded fine. the issue is that this warning occurs every time |
Are you sure the main prezto repo is on the right branch? |
Works like a charm! Thanks good job |
|
woohoo, thanks for your question, made me check git status. removed garbage - now everything works :) let's close this PR |
Alright, sorry for not following up! Thanks for the original submission. I'm glad we could get it into contrib. :) |
updated instructions after sorin-ionescu/prezto#1472
Proposed Changes
people that like minimal prompts could find this theme very appealing. i've used "pure" theme for some time, but two-line prompt is too spacious. i've added a bit of prezto integration documentation to lean README.md.
screenshot of how can it look like is here https://imgur.com/IbNVB3M