Skip to content

feat(layout): impl grid layout#31

Open
TtTRz wants to merge 29 commits intomasterfrom
feat-grid
Open

feat(layout): impl grid layout#31
TtTRz wants to merge 29 commits intomasterfrom
feat-grid

Conversation

@TtTRz
Copy link
Member

@TtTRz TtTRz commented Jan 30, 2026

No description provided.

@TtTRz TtTRz requested a review from Copilot January 30, 2026 06:40
@TtTRz TtTRz changed the title Feat: impl grid layout feat(layout): impl grid layout Jan 30, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request implements CSS Grid Layout functionality for the float-pigment layout engine, adding support for display: grid and display: inline-grid with core grid properties like grid-template-rows, grid-template-columns, grid-auto-flow, justify-items, and justify-self.

Changes:

  • Implements CSS Grid Layout Module Level 1 algorithms (placement, track sizing, alignment)
  • Adds SizingMode enum to support min-content/max-content sizing for grid items
  • Extends layout traits with grid-specific properties and methods
  • Adds comprehensive test coverage for grid layout features

Reviewed changes

Copilot reviewed 65 out of 66 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
float-pigment-layout/src/lib.rs Adds grid-related trait methods, exports SizingMode, comments out #![no_std]
float-pigment-layout/src/unit.rs Integrates SizingMode into layout computation, adds min_content_size to results
float-pigment-layout/src/algo/grid/* New grid layout algorithm implementation (placement, track sizing, alignment, matrix)
float-pigment-layout/src/types.rs Adds grid template types and Clone trait bounds
float-pigment-layout/Cargo.toml Adds grid crate dependency
float-pigment-css/src/typing.rs Adds grid-related CSS types and properties
float-pigment-forest/* Integrates grid layout into forest implementation and adds extensive tests

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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