Skip to content

Allow dashboard rows to wrap beyond twelve columns - #127

Merged
TanayBensuYurtturk merged 4 commits into
mainfrom
TanayBensuYurtturk/allow-wrapping-rows
Oct 28, 2026
Merged

TanayBensuYurtturk merged 4 commits into
mainfrom
TanayBensuYurtturk/allow-wrapping-rows

Conversation

@TanayBensuYurtturk

@TanayBensuYurtturk TanayBensuYurtturk commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Allow widgets in a dashboard row to span more than 12 columns in total, so wrapping layouts such as col: 8 followed by col: 8 pass validation. Each widget keeps its existing column bounds.

Google Slides export puts each wrapped line on a continuation slide, preserving widget order, column widths, and query results. Existing rows that fit within 12 columns retain their layout. Update layout/export documentation and dashboard creation guidance.

Validation: make test GO_PACKAGES='./pkg/dashboard/... ./pkg/slides/...' and make format-go GO_PACKAGES='./pkg/dashboard/... ./pkg/slides/...' pass. Regression tests cover wrapping, implicit widths, slide boundaries, and query results for continuation slides, including tabbed widgets.

@greptile-apps

greptile-apps Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

RetriggerView in Greptile

[Medium impact] The PR appears safe to merge; no blocking issues remain.

Reviews (4) · Last reviewed commit: "Remove local Slides test demo from the P..." · Reviewed by Greptile

Comment thread pkg/dashboard/validator.go Resolved
@TanayBensuYurtturk
TanayBensuYurtturk merged commit 9258445 into main Oct 28, 2026
3 checks passed
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.

2 participants