Skip to content

feat(Table): Added Table Component in react vanilla components - #180

Open
armaang1729 wants to merge 7 commits into
mainfrom
tableComponent
Open

armaang1729 wants to merge 7 commits into
mainfrom
tableComponent

Conversation

@armaang1729

Copy link
Copy Markdown
Collaborator

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@armaang1729 armaang1729 changed the title feat(Table): Added Table Component in react vanilla components testing 2 Aug 16, 2026
@armaang1729 armaang1729 changed the title testing 2 feat(Table): Added Table Component in react vanilla components Aug 24, 2026
…ponent

af-core wraps repeatable panels in a generated array-type container with
:type 'panel' (not 'table-row'), causing the row filter to silently drop
all repeatable rows. Fix the filter to also match item.type === 'array'
and route those to a new RepeatableTableRow component that mirrors the
Accordion/RepeatableItem pattern — gets live instances via form.getElement,
renders one TableRow per instance, and dispatches AddItem/RemoveItem on
add/remove button clicks. TableRow is simplified to accept onAdd/onRemove/
showAdd/showRemove props instead of managing form state internally.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@armaang1729 armaang1729 reopened this Aug 24, 2026
Armaan Gupta and others added 4 commits August 24, 2026 16:19
Each row now dispatches AddItem(index + 1) to insert after itself and
RemoveItem(index) to remove itself, instead of always appending/removing
from the end.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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