search

bczsalba / pytermgui

Python TUI framework with mouse support, modular widget system, customizable and rapid terminal markup language and more!
https://ptg.bczsalba.com
MIT License
2.25k stars 56 forks source link

Fixes chinse chars being wrong len #118

Closed qoft closed 7 months ago

qoft commented 1 year ago

Fixes #98 image

bczsalba commented 1 year ago

Unfortunately this pulls in a new dependency, which I'm not super keen on. Since it's kind of impossible for me to do this manually in a completely perfect and reliable way I'm not against it, but please add it to pyproject.toml! :)

qoft commented 1 year ago

Sorry about the delay, added wcwidth to the pyproject.toml

Tired-Fox commented 1 year ago

@qoft Shouldn't the dependency go in the project dependencies list?

requires = [... , "wcwidth"]

to

[project]
...
dependencies = [... , "wcwidth"]
bczsalba commented 7 months ago

Implemented this myself since this PR seems abandoned, thanks for the idea :)