Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8
Date: Wed, 15 Dec 2021 23:52:54 -0500
Message-Id: <CGGGGJEKKITC.1Q7YMR0X5RUUU@pine64-pinephone>
Cc: 
Subject: Re: [PATCH senpai] config: allow gaps to be customized
From: "Zach DeCook" <zachdecook@librem.one>
To: "Hubert Hirtz" <hubert@hirtz.pm>, <~taiite/public-inbox@lists.sr.ht>
References: <20211214211927.8827-1-zachdecook@librem.one>
 <2fe3814d-c75a-d773-55ab-5a9dbf2fdfa0@hirtz.pm>
In-Reply-To: <2fe3814d-c75a-d773-55ab-5a9dbf2fdfa0@hirtz.pm>

On Wed Dec 15, 2021 at 10:06 AM EST, Hubert Hirtz wrote:
> Hi thanks for the patch. IIUC you want gaps to be one cell wide, is
> that right?
Right now I'm using it with gaps set to 0. (the different style of the colu=
mns is enough to tell them apart).
> The gaps have been set to 2 originally because highlight nicks have
> padding. This padding would need to take the gap width into account,
Is this padding really necessary?
(e.g. if it was used to have something besides a space printed in that colu=
mn then my changes to printIdent would be invalid)

> senpai hasn't the kind of UI that is made for narrow windows anyway.

(I also was planning to add a patch to reduce / expand the width with diffe=
rent date formats, like =F0=9F=95=9E
(which would remove the last hard-coded width value: `5`))

You're right, the grid-based timeline is not *ideal* for narrow screens. (m=
ine is 38 columns).
But this patch has already made this program usable for me (I can discern n=
icknames at width 7, and the wrapping isn't too much to bear).

If there's another approach you think would be better, I would be happy to =
try it out,
but I think this change is in line with the current configuration options o=
f senpai.

> I don't think it makes sense to add all this code ...
If you like, I could probably refactor some option code so there's a net ne=
gative in lines (rather than +4 new lines). ;)

-Zach