Skip to content

fix(skills): omit column widths by default in generated grid code#2250

Open
onlyexeption wants to merge 2 commits into
masterfrom
ibarakov/fix-1696-cli
Open

fix(skills): omit column widths by default in generated grid code#2250
onlyexeption wants to merge 2 commits into
masterfrom
ibarakov/fix-1696-cli

Conversation

@onlyexeption

Copy link
Copy Markdown
Contributor

Closes #1696

Description

Provide a clear and concise summary of the changes in this PR and the motivation behind them

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that causes existing functionality to change)
  • Documentation update
  • Refactoring (code improvements without functional changes)

Related Issues

Closes #

Testing

Checklist

  • My code follows the project's coding standards
  • I have tested my changes locally
  • I have updated documentation if needed
  • Breaking changes are documented in the description

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR appears intended to address #1696 by ensuring generated grid code does not include column width by default, but the current diff only updates the “gotchas” reference documentation for the image-to-design generation skill.

Changes:

  • Added a new “gotcha” section advising omitting column width for Grid Lite and Advanced Grid to keep columns fluid by default.
  • Documented a narrow exception note about resizable Grid Lite columns using px.

Comment thread skills/igniteui-wc-generate-from-image-design/references/gotchas.md
Comment thread skills/igniteui-wc-generate-from-image-design/references/gotchas.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CLI MCP] [Blazor]: Grid columns are generated with fixed pixel widths, leaving large empty space in the grid

2 participants