Skip to content

Fix issues #3820 expand=False didn't work when it contains Columns#3823

Open
Stranger-Jie wants to merge 3 commits into
Textualize:mainfrom
Stranger-Jie:issues3820
Open

Fix issues #3820 expand=False didn't work when it contains Columns#3823
Stranger-Jie wants to merge 3 commits into
Textualize:mainfrom
Stranger-Jie:issues3820

Conversation

@Stranger-Jie

@Stranger-Jie Stranger-Jie commented Aug 7, 2025

Copy link
Copy Markdown

Type of changes

  • Bug fix
  • New feature
  • Documentation / docstrings
  • Tests
  • Other

Checklist

  • I've run the latest black with default args on new code.
  • I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
  • I've added tests for new code.
  • I accept that @willmcgugan may be pedantic in the code review.

Description

#3820 , This is a simple fix that adds a method similar to the __rich_measure__ in Panel to Conlumns class.
image

@willmcgugan

Copy link
Copy Markdown
Member

Can you please add a test for this.

@Stranger-Jie

Copy link
Copy Markdown
Author

Thanks for the review! I've gone ahead and added a test for this change. Let me know if there's anything else.

@microy

microy commented Mar 11, 2026

Copy link
Copy Markdown

This patch works !

Here is a test:

Copie d'écran_20260311_190907

Please merge this request !

Thank you !

@Stranger-Jie

Copy link
Copy Markdown
Author

I've updated the relevant code to fix the bug @willmcgugan

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