Skip to content

Conversation

@bsastregx
Copy link
Collaborator

@bsastregx bsastregx commented Oct 10, 2025

Adds the showLines property to the tabular grid component. This property defines which lines (or borders) should be displayed, similar to spreadsheet border options.

Note: Currently, this property does not affect rendering and only reflects its value as an HTML attribute. In the near future, ch-tabular-grid-render will render actual borders that match the selected property value.

Default: "all"

Options:

  • "all"
  • "all-inside"
  • "column"
  • "column-inside"
  • "row"
  • "row-inside"
  • "none"
showLines value representation
all-inside all-inside
all all
column-inside column-inside
column column
none none
row-inside row-inside
row row

@bsastregx bsastregx added feature Feature implementation pull request target: minor This PR is targeted for the next minor release labels Oct 10, 2025
@netlify
Copy link

netlify bot commented Oct 10, 2025

Deploy Preview for gx-chameleon ready!

Name Link
🔨 Latest commit 34f7cd2
🔍 Latest deploy log https://app.netlify.com/projects/gx-chameleon/deploys/68ed06e07d74350008790fb5
😎 Deploy Preview https://deploy-preview-559--gx-chameleon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@bsastregx bsastregx changed the title [ch-tabular-grid] Add showLines property [ch-tabular-grid] Add showLines property (placeholder, no effect yet) Oct 10, 2025
@bsastregx bsastregx requested a review from dangmarm October 10, 2025 19:12
@bsastregx bsastregx merged commit 9436bf0 into main Oct 13, 2025
8 checks passed
@bsastregx bsastregx deleted the feat/tabular-grid-show-lines branch October 13, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Feature implementation pull request target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants