jesseduffield / lazygit

simple terminal UI for git commands
MIT License
52.99k stars 1.85k forks source link

Support i18n #24

Open OdaDaisuke opened 6 years ago

OdaDaisuke commented 6 years ago

Lazygit doesn't support i18n?

TGhoul commented 6 years ago

Chinese character have some problems.

1533697357 1

spin6lock commented 6 years ago

It seems that the display is wrong... Some chinese character shows up and eat up following ascii character. Maybe I should raise an issue on gocui ?

jesseduffield commented 6 years ago

@TGhoul do those same characters have a fixed-width in regular use? It may be that your font isn't monospaced. I tried testing how the app behaved with some mandarin characters but they didn't even render: image

I'm thinking it may be configurable via cmd, but I'm not well versed in windows

TGhoul commented 6 years ago

@jesseduffield I think those characters is fixed-width, my problems appear in the "Commits" window, can you do something commit and write comment with Chinese simplified characters?

ponsfrilus commented 6 years ago

screenshot from 2018-08-08 12-00-59

Can't reproduce it in Ubuntu, maybe this is Windows-1252 related ?

itchyny commented 6 years ago

See https://github.com/jroimartin/gocui/pull/74.

TGhoul commented 6 years ago

@ponsfrilus @jesseduffield This issue may be related to gocui, I tested it in macOS.

image

jesseduffield commented 6 years ago

This should (or could) be fixed by https://github.com/jesseduffield/lazygit/pull/231 Looks like there are some little glitches that still occur but this is a good step in the right direction. Please let me know how the rendering works for you all :)

@OdaDaisuke @TGhoul

TGhoul commented 6 years ago

@jesseduffield Thank you for all your effort with i18n.

OdaDaisuke commented 6 years ago

@jesseduffield Thank youuuu!

glvr182 commented 6 years ago

@jesseduffield @OdaDaisuke is this issue fixed?

hello-liu commented 6 years ago

windos 有问题,ubuntu没得问题,头大

hello-liu commented 6 years ago

This should (or could) be fixed by #231 Looks like there are some little glitches that still occur but this is a good step in the right direction. Please let me know how the rendering works for you all :)

@OdaDaisuke @TGhoul

Chinese character have some problems.

hi,解决了么?,好像在ubuntu下没问题

faimin commented 6 years ago

@hello-liu It's OK on Mac OS.

hello-liu commented 6 years ago

change cmd GBK to UTF-8,it work

hello-liu commented 6 years ago

@hello-liu It's OK on Mac OS.

can you input chinese in the view ,half char no see

bushnerd commented 3 years ago

image

The filename is chinese, but it's crashed. But the content in chinese is OK.

jesseduffield commented 3 years ago

@scutxd could you provide some steps to reproduce that issue? I'm not able to reproduce locally on the latest lazygit master branch

asmwarrior commented 3 years ago

@scutxd could you provide some steps to reproduce that issue? I'm not able to reproduce locally on the latest lazygit master branch

I have the same issue here, if you add a file which has a Chinese name, than it will shown some \xxx like text in the commit message, let me give your a screen shot.

Chinese_font_error

There are two issues here: 1, see the arrow pointed as "1", the Chinese text is too width, I mean normally, one Chinese character's width equals two English character's width, but from the screen shot, it looks like it has 3 times of the English character's width.

2, the second issue the the same as the scutxd mentioned one.

Update It looks like the Chinese character's width = 3 * English character's width. Is it because that I set the font page as: chcp 65001, which means it is UTF8 format, and a Chinese character normally need 3 bytes.

asmwarrior commented 3 years ago

If I change the code page to chcp 936, which is simplified Chinese, then the character width issue look OK, but there are other layout issues, see the screen shot below:

Chinese_font_error3

asmwarrior commented 3 years ago

Bad view title drawing with unicode characters. · Issue #157 · jroimartin/gocui Maybe, this is some related.

jesseduffield commented 3 years ago

@asmwarrior I'm still struggling to replicate. Is there a public repo you could point me to which I could pull down to verify the issue?

asmwarrior commented 3 years ago

@asmwarrior I'm still struggling to replicate. Is there a public repo you could point me to which I could pull down to verify the issue?

I create a sample git repo for you to test the Chinese font issue, see here: https://github.com/asmwarrior/lazygit_issue24

jesseduffield commented 3 years ago

Thanks for making the sample repo :)

I'm not able to replicate on lazygit's latest master branch, on the iterm2 terminal in OSX: image

This may be a windows specific issue. With that said it's possible that the fault lies with termbox, one of our dependencies. There's a branch I'm working on called tcell-again which switches to tcell which is better for various reasons. Are you able to see if you still get the same issue if you're using lazygit from the tcell-again branch? You'll need to clone the lazygit repo and do a go install from within the repo's root directory but otherwise it should run the same as normal lazygit

asmwarrior commented 3 years ago

This may be a windows specific issue. With that said it's possible that the fault lies with termbox, one of our dependencies. There's a branch I'm working on called tcell-again which switches to tcell which is better for various reasons. Are you able to see if you still get the same issue if you're using lazygit from the tcell-again branch? You'll need to clone the lazygit repo and do a go install from within the repo's root directory but otherwise it should run the same as normal lazygit.

I have no experience of programming in go language. I mainly use C/C++. So for me, it may be hard to build the lazygit from the source under Windows. If you can create a simple executable file from this new branch, I can happily test it.

jesseduffield commented 3 years ago

@asmwarrior I've just compiled this from my windows PC. Does it work for you? lazygit.zip

asmwarrior commented 3 years ago

@asmwarrior I've just compiled this from my windows PC. Does it work for you? lazygit.zip

I tested this new binary under ComEmu, and here is the result.

With chcp 65001 setting, which means the UTF8 code page, I see still some errors, see the two images below:

chcp65001-error1

and

chcp65001-error2

Sometimes, the unwanted characters are shown.

With chcp 936, which is Chinese GB2312 code page, I see the frame border is not continued, but I don't see unwanted characters, see image below.

chcp936-good

In both of the code page settings, the Chinese character width issue is fixed! This is good!

But there are still same issues that the file path is still shown as a raw hex string, as the arrow 2 pointed in comment: the https://github.com/jesseduffield/lazygit/issues/24#issuecomment-801076179

diartyz commented 1 year ago

Git will quote "unusual" characters in the pathname. Try git config --global core.quotepath false