~rjarry/aerc-devel

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
6 5

[PATCH aerc v3] vaxis: update to 0.9.2

Details
Message ID
<20240617150524.387724-2-tim@timculverhouse.com>
DKIM signature
pass
Download raw message
Patch: +3 -3
Fixes several behind the scenes issues, but notably addresses scrolling
of CJK characters in the terminal widget as well as wrapping of wide
characters

Reported-by: Tianhao Wang <shrik3@mailbox.org>
Reported-by: ~runxiyu
Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>
---
v3: Fix an issue v0.9.1 created regarding wrapping of text 

 go.mod | 2 +-
 go.sum | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/go.mod b/go.mod
index e95e4c0c1dc9..660e1b02af69 100644
--- a/go.mod
+++ b/go.mod
@@ -5,7 +5,7 @@ go 1.18
require (
	git.sr.ht/~rjarry/go-opt v1.4.0
	git.sr.ht/~rockorager/go-jmap v0.5.0
	git.sr.ht/~rockorager/vaxis v0.8.5
	git.sr.ht/~rockorager/vaxis v0.9.2
	github.com/ProtonMail/go-crypto v1.0.0
	github.com/arran4/golang-ical v0.2.7
	github.com/danwakefield/fnmatch v0.0.0-20160403171240-cbb64ac3d964
diff --git a/go.sum b/go.sum
index f9a7cc3d6e1f..72e2a427930b 100644
--- a/go.sum
+++ b/go.sum
@@ -2,8 +2,8 @@ git.sr.ht/~rjarry/go-opt v1.4.0 h1:YHUKRXOuoy6d57Jt2b0DRSLbxq0mz5biq+6P/npFK5s=
git.sr.ht/~rjarry/go-opt v1.4.0/go.mod h1:oEPZUTJKGn1FVye0znaLoeskE/QTuyoJw5q+fjusdM4=
git.sr.ht/~rockorager/go-jmap v0.5.0 h1:Xs8NeqpA631HUz4uIe6V+0CpWt6b+nnHF7S14U2BVPA=
git.sr.ht/~rockorager/go-jmap v0.5.0/go.mod h1:aOTCtwpZSINpDDSOkLGpHU0Kbbm5lcSDMcobX3ZtOjY=
git.sr.ht/~rockorager/vaxis v0.8.5 h1:9R8P6n05wuk9+SoA11Wo+yI18L2/qPYjWmWdvctWF6k=
git.sr.ht/~rockorager/vaxis v0.8.5/go.mod h1:h94aKek3frIV1hJbdXjqnBqaLkbWXvV+UxAsQHg9bns=
git.sr.ht/~rockorager/vaxis v0.9.2 h1:OKPaCG3S4b6jWnwUPC0zX+Zw16hhLcSgC2mLw/nwsV0=
git.sr.ht/~rockorager/vaxis v0.9.2/go.mod h1:h94aKek3frIV1hJbdXjqnBqaLkbWXvV+UxAsQHg9bns=
github.com/ProtonMail/go-crypto v0.0.0-20230923063757-afb1ddc0824c/go.mod h1:EjAoLdwvbIOoOQr3ihjnSoLZRtE8azugULFRteWMNc0=
github.com/ProtonMail/go-crypto v1.0.0 h1:LRuvITjQWX+WIfr930YHG2HNfjR1uOfyf5vE0kC2U78=
github.com/ProtonMail/go-crypto v1.0.0/go.mod h1:EjAoLdwvbIOoOQr3ihjnSoLZRtE8azugULFRteWMNc0=
-- 
2.45.2

[aerc/patches] build success

builds.sr.ht <builds@sr.ht>
Details
Message ID
<D22DX8763X6E.3690ZEP597RQ5@fra02>
In-Reply-To
<20240617150524.387724-2-tim@timculverhouse.com> (view parent)
DKIM signature
missing
Download raw message
aerc/patches: SUCCESS in 2m0s

[vaxis: update to 0.9.2][0] v3 from [Tim Culverhouse][1]

[0]: https://lists.sr.ht/~rjarry/aerc-devel/patches/53343
[1]: tim@timculverhouse.com

✓ #1253310 SUCCESS aerc/patches/alpine-edge.yml https://builds.sr.ht/~rjarry/job/1253310
✓ #1253311 SUCCESS aerc/patches/openbsd.yml     https://builds.sr.ht/~rjarry/job/1253311
Details
Message ID
<D22HES7TOJ58.1F3IHDLZ83IPL@mailbox.org>
In-Reply-To
<20240617150524.387724-2-tim@timculverhouse.com> (view parent)
DKIM signature
pass
Download raw message
On Mon Jun 17, 2024 at 5:04 PM CEST, Tim Culverhouse wrote:
> Fixes several behind the scenes issues, but notably addresses scrolling
> of CJK characters in the terminal widget as well as wrapping of wide
> characters
>
> Reported-by: Tianhao Wang <shrik3@mailbox.org>
> Reported-by: ~runxiyu
> Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>
> ---
> v3: Fix an issue v0.9.1 created regarding wrapping of text 
>
>  go.mod | 2 +-
>  go.sum | 4 ++--
>  2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/go.mod b/go.mod
> index e95e4c0c1dc9..660e1b02af69 100644
> --- a/go.mod
> +++ b/go.mod
> @@ -5,7 +5,7 @@ go 1.18
>  require (
>  	git.sr.ht/~rjarry/go-opt v1.4.0
>  	git.sr.ht/~rockorager/go-jmap v0.5.0
> -	git.sr.ht/~rockorager/vaxis v0.8.5
> +	git.sr.ht/~rockorager/vaxis v0.9.2
>  	github.com/ProtonMail/go-crypto v1.0.0
>  	github.com/arran4/golang-ical v0.2.7
>  	github.com/danwakefield/fnmatch v0.0.0-20160403171240-cbb64ac3d964
> diff --git a/go.sum b/go.sum
> index f9a7cc3d6e1f..72e2a427930b 100644
> --- a/go.sum
> +++ b/go.sum
> @@ -2,8 +2,8 @@ git.sr.ht/~rjarry/go-opt v1.4.0 h1:YHUKRXOuoy6d57Jt2b0DRSLbxq0mz5biq+6P/npFK5s=
>  git.sr.ht/~rjarry/go-opt v1.4.0/go.mod h1:oEPZUTJKGn1FVye0znaLoeskE/QTuyoJw5q+fjusdM4=
>  git.sr.ht/~rockorager/go-jmap v0.5.0 h1:Xs8NeqpA631HUz4uIe6V+0CpWt6b+nnHF7S14U2BVPA=
>  git.sr.ht/~rockorager/go-jmap v0.5.0/go.mod h1:aOTCtwpZSINpDDSOkLGpHU0Kbbm5lcSDMcobX3ZtOjY=
> -git.sr.ht/~rockorager/vaxis v0.8.5 h1:9R8P6n05wuk9+SoA11Wo+yI18L2/qPYjWmWdvctWF6k=
> -git.sr.ht/~rockorager/vaxis v0.8.5/go.mod h1:h94aKek3frIV1hJbdXjqnBqaLkbWXvV+UxAsQHg9bns=
> +git.sr.ht/~rockorager/vaxis v0.9.2 h1:OKPaCG3S4b6jWnwUPC0zX+Zw16hhLcSgC2mLw/nwsV0=
> +git.sr.ht/~rockorager/vaxis v0.9.2/go.mod h1:h94aKek3frIV1hJbdXjqnBqaLkbWXvV+UxAsQHg9bns=
>  github.com/ProtonMail/go-crypto v0.0.0-20230923063757-afb1ddc0824c/go.mod h1:EjAoLdwvbIOoOQr3ihjnSoLZRtE8azugULFRteWMNc0=
>  github.com/ProtonMail/go-crypto v1.0.0 h1:LRuvITjQWX+WIfr930YHG2HNfjR1uOfyf5vE0kC2U78=
>  github.com/ProtonMail/go-crypto v1.0.0/go.mod h1:EjAoLdwvbIOoOQr3ihjnSoLZRtE8azugULFRteWMNc0=


Thanks! This fixes broken CJK text, however hindi is still ghosting (but
perhaps that's different issue), could you reproduce with the following
Lorem ipsum?

screenshot:
    https://0x0.st/XTHr.png

पढाए हिंदी रहारुप अनुवाद कार्यलय मुख्य संस्था सोफ़तवेर निरपेक्ष उनका आपके बाटते आशाआपसमुख्यतह उश
की करता। शुरुआत संस्था कुशलता मेंभटृ अनुवाद गएआप विशेष सकते परिभाषित लाभान्वित प्रतिदेकर समजते
दिशामे प्राप्त जैसे वर्णन संस्थान निर्माता प्रव्रुति भाति चुनने उपलब्ध बेंगलूर अर्थपुर्ण


Tested-by: Tianhao Wang <shrik3@mailbox.org>

cheers,
wth
Details
Message ID
<D22KYYI2CF0C.2AJWAPJBASPQ6@timculverhouse.com>
In-Reply-To
<D22HES7TOJ58.1F3IHDLZ83IPL@mailbox.org> (view parent)
DKIM signature
pass
Download raw message
On Mon Jun 17, 2024 at 12:51 PM CDT, Tianhao Wang wrote:
>
> पढाए हिंदी रहारुप अनुवाद कार्यलय मुख्य संस्था सोफ़तवेर निरपेक्ष उनका आपके बाटते आशाआपसमुख्यतह उश
> की करता। शुरुआत संस्था कुशलता मेंभटृ अनुवाद गएआप विशेष सकते परिभाषित लाभान्वित प्रतिदेकर समजते
> दिशामे प्राप्त जैसे वर्णन संस्थान निर्माता प्रव्रुति भाति चुनने उपलब्ध बेंगलूर अर्थपुर्ण
>
>
> Tested-by: Tianhao Wang <shrik3@mailbox.org>
>
> cheers,
> wth

Thanks for testing - I can definitely reproduce this one. I'll look into it
tomorrow.

-- 
Tim
Details
Message ID
<D23AS90NK915.34EKIHDD2KSDX@timculverhouse.com>
In-Reply-To
<D22HES7TOJ58.1F3IHDLZ83IPL@mailbox.org> (view parent)
DKIM signature
pass
Download raw message
On Mon Jun 17, 2024 at 12:51 PM CDT, Tianhao Wang wrote:
> पढाए हिंदी रहारुप अनुवाद कार्यलय मुख्य संस्था सोफ़तवेर निरपेक्ष उनका आपके बाटते आशाआपसमुख्यतह उश
> की करता। शुरुआत संस्था कुशलता मेंभटृ अनुवाद गएआप विशेष सकते परिभाषित लाभान्वित प्रतिदेकर समजते
> दिशामे प्राप्त जैसे वर्णन संस्थान निर्माता प्रव्रुति भाति चुनने उपलब्ध बेंगलूर अर्थपुर्ण

I can't seem to get much to agree on how long these strings should be. The
top line measured in a few different ways:

Golang (uniseg):              80 columns
Golang (wcwidth):             95 columns
Golang (ignore ZWJ):          80 columns
Zig (zg):                     63 columns
Alacritty (??):               80 columns
Ghostty (handrolled):         68 columns
Kitty (wcswidth):             68 columns
Foot (utf8proc):              80 columns

There's no clear consensus nor shared feature for how to decide how to measure
it. I think the best we can do is provide a keybind to do a complete repaint of
the screen for if these characters end up duplicated.

Anyone else have any ideas??

-- 
Tim
Details
Message ID
<0847e8a1-3f47-4de5-be80-7fc7c1dcbd86@ferdinandy.com>
In-Reply-To
<D23AS90NK915.34EKIHDD2KSDX@timculverhouse.com> (view parent)
DKIM signature
missing
Download raw message
2024. jún. 18. 18:53:27 Tim Culverhouse <tim@timculverhouse.com>:

> On Mon Jun 17, 2024 at 12:51 PM CDT, Tianhao Wang wrote:
>> पढाए हिंदी रहारुप अनुवाद कार्यलय मुख्य संस्था सोफ़तवेर निरपेक्ष उनका 
>> आपके बाटते आशाआपसमुख्यतह उश
>> की करता। शुरुआत संस्था कुशलता मेंभटृ अनुवाद गएआप विशेष सकते परिभाषित 
>> लाभान्वित प्रतिदेकर समजते
>> दिशामे प्राप्त जैसे वर्णन संस्थान निर्माता प्रव्रुति भाति चुनने उपलब्ध 
>> बेंगलूर अर्थपुर्ण
>
> I can't seem to get much to agree on how long these strings should be. 
> The
> top line measured in a few different ways:
>
> Golang (uniseg):              80 columns
> Golang (wcwidth):             95 columns
> Golang (ignore ZWJ):          80 columns
> Zig (zg):                     63 columns
> Alacritty (??):               80 columns
> Ghostty (handrolled):         68 columns
> Kitty (wcswidth):             68 columns
> Foot (utf8proc):              80 columns

The median seems to be 80.


>
> There's no clear consensus nor shared feature for how to decide how to 
> measure
> it. I think the best we can do is provide a keybind to do a complete 
> repaint of
> the screen for if these characters end up duplicated.

A repaint ability might be useful anyway, _stuff_ can happen anytime :)

The problem here is that you sort of need to guess what the host terminal 
will do?

>
> Anyone else have any ideas??

Send the above text to all the above projects and ask them to figure it 
out 🤣

>
> --
> Tim

Applied: [PATCH aerc v3] vaxis: update to 0.9.2

Details
Message ID
<171917324085.363076.13539783619577098262@ringo.local>
In-Reply-To
<20240617150524.387724-2-tim@timculverhouse.com> (view parent)
DKIM signature
pass
Download raw message
Tim Culverhouse <tim@timculverhouse.com> wrote:
> Fixes several behind the scenes issues, but notably addresses scrolling
> of CJK characters in the terminal widget as well as wrapping of wide
> characters
>
> Reported-by: Tianhao Wang <shrik3@mailbox.org>
> Reported-by: ~runxiyu
> Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>
> ---
> v3: Fix an issue v0.9.1 created regarding wrapping of text

Acked-by: Robin Jarry <robin@jarry.cc>

Applied, thanks.

To git@git.sr.ht:~rjarry/aerc
   910bc95af6e8..159746ff9ed9  master -> master
Reply to thread Export thread (mbox)