Skip to content

gpui: Add grid_rows_min_content and grid_rows_max_content methods - #61837

Merged
MrSubidubi merged 2 commits into
zed-industries:mainfrom
bowenxuuu:grid-rows-min-max-content
Jul 29, 2026
Merged

gpui: Add grid_rows_min_content and grid_rows_max_content methods#61837
MrSubidubi merged 2 commits into
zed-industries:mainfrom
bowenxuuu:grid-rows-min-max-content

Conversation

@bowenxuuu

@bowenxuuu bowenxuuu commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Objective

  • GPUI provides grid_cols_min_content() and grid_cols_max_content(), but the grid_rows_*_content() methods are missing.

Solution

  • This PR adds the grid_rows_min_content() and grid_rows_max_content() methods, and renames the enum TemplateColumnMinSize to GridTemplateMinSize.

Testing

  • I have tested locally.

Self-Review Checklist:

  • I've reviewed my own diff for quality, security, and reliability
  • Unsafe blocks (if any) have justifying comments
  • The content adheres to Zed's UI standards (UX/UI and icon guidelines)
  • Tests cover the new/changed behavior
  • Performance impact has been considered and is acceptable

Release Notes:

  • N/A

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jul 29, 2026
@MrSubidubi MrSubidubi changed the title gpui: Add grid_rows_min_content() and grid_rows_max_content() methods gpui: Add grid_rows_min_content and grid_rows_max_content methods Jul 29, 2026

@MrSubidubi MrSubidubi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thank you!

@MrSubidubi
MrSubidubi enabled auto-merge July 29, 2026 09:45
auto-merge was automatically disabled July 29, 2026 09:56

Head branch was pushed to by a user without write access

@MrSubidubi
MrSubidubi enabled auto-merge July 29, 2026 10:13
@MrSubidubi
MrSubidubi added this pull request to the merge queue Jul 29, 2026
Merged via the queue into zed-industries:main with commit 5ccbbbd Jul 29, 2026
36 checks passed
@bowenxuuu
bowenxuuu deleted the grid-rows-min-max-content branch July 31, 2026 07:54
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
…zed-industries#61837)

# Objective

- GPUI provides `grid_cols_min_content()` and `grid_cols_max_content()`,
but the `grid_rows_*_content()` methods are missing.

## Solution

- This PR adds the `grid_rows_min_content()` and
`grid_rows_max_content()` methods, and renames the enum
`TemplateColumnMinSize` to `GridTemplateMinSize`.

## Testing

- I have tested locally.

## Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [ ] Unsafe blocks (if any) have justifying comments
- [ ] The content adheres to Zed's UI standards
([UX/UI](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
and
[icon](https://github.com/zed-industries/zed/blob/main/crates/icons/README.md)
guidelines)
- [ ] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable


Release Notes:

- N/A
HyacinthHaru pushed a commit to HyacinthHaru/z3rm that referenced this pull request Aug 16, 2026
…zed-industries#61837)

# Objective

- GPUI provides `grid_cols_min_content()` and `grid_cols_max_content()`,
but the `grid_rows_*_content()` methods are missing.

## Solution

- This PR adds the `grid_rows_min_content()` and
`grid_rows_max_content()` methods, and renames the enum
`TemplateColumnMinSize` to `GridTemplateMinSize`.

## Testing

- I have tested locally.

## Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [ ] Unsafe blocks (if any) have justifying comments
- [ ] The content adheres to Zed's UI standards
([UX/UI](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
and
[icon](https://github.com/zed-industries/zed/blob/main/crates/icons/README.md)
guidelines)
- [ ] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

Release Notes:

- N/A

(cherry picked from commit 5ccbbbd)
playdohface pushed a commit to playdohface/zed that referenced this pull request Aug 29, 2026
…zed-industries#61837)

# Objective

- GPUI provides `grid_cols_min_content()` and `grid_cols_max_content()`,
but the `grid_rows_*_content()` methods are missing.

## Solution

- This PR adds the `grid_rows_min_content()` and
`grid_rows_max_content()` methods, and renames the enum
`TemplateColumnMinSize` to `GridTemplateMinSize`.

## Testing

- I have tested locally.

## Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [ ] Unsafe blocks (if any) have justifying comments
- [ ] The content adheres to Zed's UI standards
([UX/UI](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
and
[icon](https://github.com/zed-industries/zed/blob/main/crates/icons/README.md)
guidelines)
- [ ] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable


Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants