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

Texts are not aligned and position change while I scroll down... #184

Closed
jamespanda opened this issue Apr 15, 2017 · 10 comments · Fixed by #185
Closed

Texts are not aligned and position change while I scroll down... #184

jamespanda opened this issue Apr 15, 2017 · 10 comments · Fixed by #185

Comments

@jamespanda
Copy link

work fine while all texts are English or numbers...
2017-04-15-165730_1919x1020_scrot
2017-04-15-170349_1919x1019_scrot

@mptre
Copy link
Owner

mptre commented Apr 15, 2017

Hi,
Could you submit the following info:

  • Version of pick: pick -v

  • OS: uname -a

  • Shell: echo $SHELL

  • Locale: echo $LC_CTYPE

  • The input file used in your screenshots

@jamespanda
Copy link
Author

version: 1.5.3
os: FreeBSD 11.0
shell: bash (all shell return the same result)
locale: zh_TW.UTF-8

I try...(contain 下載 , 文件 and 圖片 subdirectory in home)
echo "abc\nbcd\n123\n下載\n文件\n圖片" | pick
find ~ -type d -maxdepth 1 | sort | pick

@mptre
Copy link
Owner

mptre commented Apr 16, 2017

Ah, this has been on my todo for a long while. The current
implementation is naive and assumes every character will occupy exactly
one column once displayed. Whereas the characters in your example
occupies 3 columns. I will take a look at this later this week and let
you know when I got something working. Thanks for the bug-report.

@jamespanda
Copy link
Author

I will wait for it
thanks

@mptre
Copy link
Owner

mptre commented Apr 16, 2017

Please go ahead and try #185.

@jamespanda
Copy link
Author

It works.
Thanks ^_^

@mptre
Copy link
Owner

mptre commented Apr 17, 2017

Great! I will let you know when this is released.

@jamespanda
Copy link
Author

OK

@mptre
Copy link
Owner

mptre commented Apr 19, 2017

Hi, v1.6.0 was just released with the fix included.

@jamespanda
Copy link
Author

thanks

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 a pull request may close this issue.

2 participants