urwid / urwid / 21588891692

74%

Build Type

push

travis-pro

Committed by web-flow

Commit Message

Bugfix some kinds of emojis with grapheme boundry support (#1100)

* TDD failing tests, to be resolved by integration

This requires integration with wcwidth>=0.3 with new iter_graphemes()
function.

It is true, though, that sometimes measuring things individually by a
character can accidentally be correct, so some tests do succeed,

* wcwidth>=0.3.5 integration

Uses ``wcwidth.iter_graphemes()`` and a dumb hack to go previous
grapheme for now, I am working on a new
wcwidth.grapheme_boundary_before() function, maybe in wcwidth>=0.4.

WIP, still in testing

* improve edit_text.txt with more examples

* also fix this NoneType exception on exit ??

* require yet to be released wcwidth

* integrate with next wcwidth release

* reduce test file length, focus on important codepoints

* match origin/master better

* change wcswidth -> width+control_codes='ignore'

Added some raw control characters to docs/examples/edit_text.txt, there
is a strange effect of the cursor, it will not "move" into them, as each
cell is measured as 0 -- but at least it doesn't crash!

(For an editor to handle control characters, it needs to change the
displayed presentation, eg \x00 -> "^A" and usually bolded by tradition,
it also needs to allow input, eg "^V^A")

* addressing feedback and linting

* raise DeprecationWarning as suggested

* Fix ruff warnings

Run Details

1916 of 2558 branches covered (74.9%)

Branch coverage included in aggregate %.

41 of 46 new or added lines in 3 files covered. (89.13%)

2 existing lines in 1 file now uncovered.

9245 of 12510 relevant lines covered (73.9%)

7.52 hits per line