Skip to content

fix(tui): support resizing#7663

Merged
rgrinberg merged 1 commit intomainfrom
ps/rr/fix_tui___support_resizing
May 19, 2023
Merged

fix(tui): support resizing#7663
rgrinberg merged 1 commit intomainfrom
ps/rr/fix_tui___support_resizing

Conversation

@rgrinberg
Copy link
Copy Markdown
Member

@rgrinberg rgrinberg commented May 1, 2023

This PR allows the tui to be rerendered on resize events. This change is twofold:

  • Allow handling sigwinch in the tui thread by blocking it in every other thread.
  • Calling render whenever we get the winch.

Adding @snowleopard as a reviewer for the insignificant changes to scheduler to allow us to handle sigwinch.

Alizter
Alizter previously approved these changes May 1, 2023
@rgrinberg rgrinberg force-pushed the ps/rr/fix_tui___support_resizing branch 2 times, most recently from 8b1a1b2 to afe3b45 Compare May 1, 2023 04:39
@Alizter Alizter dismissed their stale review May 1, 2023 14:45

need to check again

@rgrinberg rgrinberg force-pushed the ps/rr/fix_tui___support_resizing branch from afe3b45 to e7c8a14 Compare May 1, 2023 16:16
@rgrinberg rgrinberg requested a review from snowleopard May 1, 2023 16:18
Copy link
Copy Markdown
Collaborator

@Alizter Alizter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resizing now renders reliably.

@Alizter Alizter mentioned this pull request May 2, 2023
9 tasks
@rgrinberg
Copy link
Copy Markdown
Member Author

ping @snowleopard

Copy link
Copy Markdown
Collaborator

@snowleopard snowleopard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good but I left a couple of comments.

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>

<!-- ps-id: 1e29eae2-40a1-4f0e-8ea1-81418eec9b72 -->
@rgrinberg rgrinberg force-pushed the ps/rr/fix_tui___support_resizing branch from e7c8a14 to 92d40ba Compare May 19, 2023 17:33
@rgrinberg rgrinberg merged commit fcdecfe into main May 19, 2023
@rgrinberg rgrinberg deleted the ps/rr/fix_tui___support_resizing branch May 19, 2023 17:48
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.

3 participants