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

Remove paddings #1

Merged
merged 1 commit into from
Aug 19, 2014
Merged

Remove paddings #1

merged 1 commit into from
Aug 19, 2014

Conversation

dxops
Copy link
Contributor

@dxops dxops commented Aug 18, 2014

No description provided.

@sunaku
Copy link
Owner

sunaku commented Aug 19, 2014

Thanks for the patch, but my prompt looks exactly the same after applying it on my Debian Sid system.

What OS are you using? Could you provide a screenshot showing what the effect is on your system?

@dxops
Copy link
Contributor Author

dxops commented Aug 19, 2014

I'm using os x mavericks, just replaced plus and minus with arrows

before
2014-08-19 21 26 10

and after
2014-08-19 21 28 19

@sunaku
Copy link
Owner

sunaku commented Aug 19, 2014

Thanks for the screenshots. ✨ What ZSH version do you have? Mine is:

$ zsh --version
zsh 5.0.5 (x86_64-pc-linux-gnu)

@dxops
Copy link
Contributor Author

dxops commented Aug 19, 2014

❯ zsh --version                                                                               
zsh 5.0.5 (x86_64-apple-darwin13.3.0)

@sunaku
Copy link
Owner

sunaku commented Aug 19, 2014

Alright, I'll merge it since it fixes the problem for you and doesn't introduce any new problems for me. 😅

sunaku added a commit that referenced this pull request Aug 19, 2014
zsh: remove padding from ahead/behind-by counters
@sunaku sunaku merged commit 1c5c6f0 into sunaku:master Aug 19, 2014
@dxops
Copy link
Contributor Author

dxops commented Aug 19, 2014

👍

sunaku added a commit that referenced this pull request Jun 28, 2023
sunaku added a commit that referenced this pull request Jul 4, 2023
sunaku added a commit that referenced this pull request Jul 4, 2023
sunaku added a commit that referenced this pull request Sep 24, 2024
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.

2 participants