Trevor Slocum

Results 65 comments of Trevor Slocum

Thanks @foxcpp. I am interested in merging your changes into [cview](https://gitlab.com/tslocum/cview). I'll let you know once I've had a chance to review your changes.

Hey @foxcpp, I only see your optimization to use TrimRightFunc in your updated branches. Were you able to review the benchmark results which seem to show these changes cause decreases...

I believe you have the results backwards: when not wrapping text, Draw is improved 60% while Write is up to 600% slower (if old is master and new is textview-opts).

Please share the text that causes this breakage as well as the application code responsible for displaying it.

I am glad to rebase this and other PRs if there is interest in merging them. Please let me know. Changes I have submitted are available now in [cview](https://gitlab.com/tslocum/cview), a...

Tested and merged into [cview](https://git.sr.ht/~tslocum/cview) with [this additional commit](https://git.sr.ht/~tslocum/cview/commit/c90e430fdfdad1d5584765f46d34d88044113785) adding a message argument to Form.NewCheckbox

[TabbedPanels](https://gitlab.com/tslocum/cview/-/blob/master/tabbedpanels.go) is an example of extracting this functionality to a new class. Let me know if you'd like a PR.

Without this change, when setting the content of a TextView with its background set to ColorDefault to a shorter length than it previously was, anywhere not drawn upon with new...

Setting the foreground and background to ColorDefault allows the usage of users' default terminal colors. Filling with a solid color instead may affect terminal transparency (Alacritty+Linux) and/or not combine aesthetically...

In the mean time, you can assign and change an identifier whenever cancelling a Seq: ```go streamer := beep.Seq(audioFile.Streamer, beep.Callback(func() { if playingFileID != thisFileID { return } go nextTrack()...