Skip to content

Add and refactor table attributes and related tests#166

Merged
phcurado merged 7 commits intophcurado:mainfrom
georgemarshall:codex/fix-missing-rest-includes-again
Apr 8, 2026
Merged

Add and refactor table attributes and related tests#166
phcurado merged 7 commits intophcurado:mainfrom
georgemarshall:codex/fix-missing-rest-includes-again

Conversation

@georgemarshall
Copy link
Copy Markdown
Contributor

This pull request enhances the flexibility of the td and th components in the DaisyUIComponents.Table module by allowing additional standard HTML table attributes to be passed through. It also updates the related tests to verify the correct handling of these new attributes.

Component enhancements:

  • Updated the td component to accept colspan, headers, and rowspan as additional HTML attributes via the :rest option.
  • Updated the th component to accept abbr, colspan, headers, rowspan, and scope as additional HTML attributes via the :rest option.

Test updates:

  • Extended tests in DaisyUIComponents.TableTest to ensure that the new attributes (abbr, colspan, headers, rowspan, scope) are correctly passed through and rendered on the td and th components.

Codex AI and others added 7 commits April 5, 2026 13:30
Co-authored-by: georgemarshall <132688+georgemarshall@users.noreply.github.com>
Co-authored-by: georgemarshall <132688+georgemarshall@users.noreply.github.com>
Co-authored-by: georgemarshall <132688+georgemarshall@users.noreply.github.com>
Co-authored-by: georgemarshall <132688+georgemarshall@users.noreply.github.com>
Co-authored-by: georgemarshall <132688+georgemarshall@users.noreply.github.com>
Co-authored-by: georgemarshall <132688+georgemarshall@users.noreply.github.com>
Co-authored-by: georgemarshall <132688+georgemarshall@users.noreply.github.com>
Copy link
Copy Markdown
Owner

@phcurado phcurado left a comment

Choose a reason for hiding this comment

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

💜

@phcurado phcurado merged commit 060aa2f into phcurado:main Apr 8, 2026
11 of 12 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.

3 participants