Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support tmux #38

Merged
merged 1 commit into from
Feb 10, 2022
Merged

Support tmux #38

merged 1 commit into from
Feb 10, 2022

Conversation

IdlePhysicist
Copy link
Owner

Supporting tmux required updating to the latest cview (v1.5.7) which
drew in changes in tcell.

In this commit we also move to cview's new repo, and finally remove the
import alias which aliased cview to tview.

Most the changes in internal/gui are what was required to compile
cave-logger on this newer cview, as there have been several breaking API
changes. The most affecting being the removinal of chaining. While
Trevor had good reasons for this it was a functionality that cave-logger
used extensively.

Also remove unused channel from page structs.

Supporting tmux required updating to the latest cview (v1.5.7) which
drew in changes in tcell.

In this commit we also move to cview's new repo, and finally remove the
import alias which aliased cview to tview.

Most the changes in `internal/gui` are what was required to compile
cave-logger on this newer cview, as there have been several breaking API
changes. The most affecting being the removinal of chaining. While
Trevor had good reasons for this it was a functionality that cave-logger
used extensively.

Also remove unused channel from page structs.
@IdlePhysicist IdlePhysicist changed the base branch from master to develop February 10, 2022 04:57
@IdlePhysicist IdlePhysicist merged commit 8b37bc3 into develop Feb 10, 2022
@IdlePhysicist IdlePhysicist deleted the support-tmux branch February 10, 2022 05:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant