[00:26] *** librasteve_ left
[03:26] <Geth> ¦ Terminal-Widgets/main: 8eed93247e | (Geoffrey Broadwell)++ | lib/Terminal/Widgets/Viewer/Tree.rakumod

[03:26] <Geth> ¦ Terminal-Widgets/main: Split Viewer::Tree.select-{,display-}node

[03:26] <Geth> ¦ Terminal-Widgets/main: 

[03:26] <Geth> ¦ Terminal-Widgets/main: This is a mild adaptation of patrickbkr++'s PR #40.  Thank you Patrick!

[03:26] <Geth> ¦ Terminal-Widgets/main: review: https://github.com/japhb/Terminal-Widgets/commit/8eed93247e

[04:38] <Geth> ¦ Terminal-Widgets: patrickbkr++ created pull request #42: Cursor and scroll hooks

[04:38] <Geth> ¦ Terminal-Widgets: review: https://github.com/japhb/Terminal-Widgets/pull/42

[04:44] <Geth> ¦ Terminal-Widgets/main: 49cedbffbd | (Geoffrey Broadwell)++ | 5 files

[04:44] <Geth> ¦ Terminal-Widgets/main: Wire up wide-context for RenderSpans

[04:44] <Geth> ¦ Terminal-Widgets/main: 

[04:44] <Geth> ¦ Terminal-Widgets/main: Also allows correct use of is-monospace-core/duospace-width-core, which

[04:44] <Geth> ¦ Terminal-Widgets/main: until now had hardwired the wide-context parameter to 0.

[04:44] <Geth> ¦ Terminal-Widgets/main: 

[04:44] <Geth> ¦ Terminal-Widgets/main: This does *not* include any place where the wide-context flag is actually

[04:44] <Geth> ¦ Terminal-Widgets/main: set to True; everywhere either passes on the flag from somewhere else or

[04:44] <Geth> ¦ Terminal-Widgets/main: defaults it to False.

[04:44] <Geth> ¦ Terminal-Widgets/main: review: https://github.com/japhb/Terminal-Widgets/commit/49cedbffbd

[04:44] <Geth> ¦ Terminal-Widgets/main: 511fe7d3ab | (Geoffrey Broadwell)++ | 3 files

[04:44] <Geth> ¦ Terminal-Widgets/main: Wire up setting wide-context based on terminal.caps

[04:44] <Geth> ¦ Terminal-Widgets/main: review: https://github.com/japhb/Terminal-Widgets/commit/511fe7d3ab

[04:47] <Geth> ¦ Terminal-Widgets/main: bc64c3b19e | (Patrick Böker)++ | lib/Terminal/Widgets/Scrollable.rakumod

[04:47] <Geth> ¦ Terminal-Widgets/main: Provide a `scrolled` hook

[04:47] <Geth> ¦ Terminal-Widgets/main: 

[04:47] <Geth> ¦ Terminal-Widgets/main: Called whenever the scrolling of a Widget changes.

[04:47] <Geth> ¦ Terminal-Widgets/main: review: https://github.com/japhb/Terminal-Widgets/commit/bc64c3b19e

[04:47] <Geth> ¦ Terminal-Widgets/main: 9bcfa0b596 | (Patrick Böker)++ | lib/Terminal/Widgets/WrappableBuffer.rakumod

[04:47] <Geth> ¦ Terminal-Widgets/main: Provide a `cursor-moved` hook

[04:47] <Geth> ¦ Terminal-Widgets/main: 

[04:47] <Geth> ¦ Terminal-Widgets/main: Called whenever the cursor of a WrappableBuffer changes.

[04:47] <Geth> ¦ Terminal-Widgets/main: 

[04:47] <Geth> ¦ Terminal-Widgets/main: When moving the cursor along a rugged edge, the cursor-x is kept at the

[04:47] <Geth> ¦ Terminal-Widgets/main: original X position, even when the line is shorter. The event will return

[04:47] <Geth> ¦ Terminal-Widgets/main: the X coordinate of the last char of that line in that case instead.

[04:47] <Geth> ¦ Terminal-Widgets/main: review: https://github.com/japhb/Terminal-Widgets/commit/9bcfa0b596

[04:47] <Geth> ¦ Terminal-Widgets/main: 1e45c78331 | (Geoffrey Broadwell)++ (committed using GitHub Web editor) | 2 files

[04:47] <Geth> ¦ Terminal-Widgets/main: Merge pull request #42 from patrickbkr/cursor-and-scroll-hooks

[04:47] <Geth> ¦ Terminal-Widgets/main: 

[04:47] <Geth> ¦ Terminal-Widgets/main: Cursor and scroll hooks

[04:47] <Geth> ¦ Terminal-Widgets/main: review: https://github.com/japhb/Terminal-Widgets/commit/1e45c78331

[04:47] <japhb> And thank you again, patrickb++

[04:47] <patrickb> o/ Hi there!

[04:47] <japhb> o/.

[04:47] <japhb> o/

[04:48] <japhb> How are you doing?

[04:48] <patrickb> Good! On a full day trip. So lots of hacking time ahead. \o/

[04:49] <japhb> Wheeeeee

[04:49] <patrickb> How is the new job going?

[04:49] <japhb> LOTS to do.  But good.  My team and org are awesome.

[04:50] <patrickb> Oh, that sounds good!

[04:50] <japhb> Yeah.  We're bootstrapping a lot of newness into the org, so those of us that joined recently are going "HOW MANY THINGS CAN WE CHANGE AT ONCE?"

[04:51] <japhb> What about you?  How are things going on your continent?  :-)

[04:52] <patrickb> Oh, I know the feeling. Same at my place. My company is growing massively and we somehow have to keep the processes and the tech stack keep up with all the progress / changes.

[04:52] <japhb> Never a dull moment.  :-D

[04:53] <japhb> Do you have more things you'd like to PR before I make a new release?

[04:54] <patrickb> Apart from work, me and family entered into the (by now traditional) pre-birth frenzy. No.3 ETA in two months.

[04:54] <japhb> \o/

[04:54] <japhb> Awesome, congrats!

[04:56] <patrickb> Re more changes - I can't tell. These things show up while I go. Unless there is reason to release fast, I'm fine with piling up more changes. Currently I'm working off of T-W main anyways.

[04:56] <patrickb> Thank you!

[05:01] <japhb> Ah excellent, and you're welcome!

[05:09] <patrickb> Currently colorizing the tree view requires subclassing and doing post-formatting in `node-content`. Is there a deeper reason to not allow `short-name` to return spans directly?

[05:29] <patrickb> I'll do a quick test drive and maybe make a PR.

[08:32] <Geth> ¦ Terminal-Widgets: patrickbkr++ created pull request #46: Fix `T-W::Tree.select-display-node`

[08:32] <Geth> ¦ Terminal-Widgets: review: https://github.com/japhb/Terminal-Widgets/pull/46