From 63e32a94c061971d5080f0258ef5ce61ab42aa90 Mon Sep 17 00:00:00 2001 From: nick black Date: Sat, 14 Nov 2020 23:30:25 -0500 Subject: [PATCH] =?UTF-8?q?Sextants:=20fix=20arithmetic=20mistake=20that?= =?UTF-8?q?=20broke=20=F0=9F=AC=93,=20=F0=9F=AC=A6,=20=F0=9F=AC=A7?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The scheme used in draw_sextant works well enough, but the feed was broken. 21 and 42 need to be skipped, not 20 and 40. Without this change, the following glyphs are broken: U+1FB13 BLOCK SEXTANT-35 🬓 renders as LEFT HALF U+1FB26 BLOCK SEXTANT-46 🬦 renders as SEXTANT-146 U+1FB27 BLOCK SEXTANT-146 🬧 renders as RIGHT HALF With this change, the entirety of the sextant block works correctly. Found debugging the Notcurses sextant blitter: https://github.com/dankamongmen/notcurses/issues/1112 --- kitty/fonts/box_drawing.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kitty/fonts/box_drawing.py b/kitty/fonts/box_drawing.py index 00cdbd30a..5393dba74 100644 --- a/kitty/fonts/box_drawing.py +++ b/kitty/fonts/box_drawing.py @@ -955,7 +955,7 @@ for i in range(256): c = 0x1fb00 for i in range(1, 63): - if i in (20, 40): + if i in (21, 42): continue box_chars[chr(c)] = [p(sextant, which=i)] c += 1