Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(v2): Implement TableField component #2945

Merged
merged 38 commits into from
Nov 5, 2021

Conversation

karrui
Copy link
Contributor

@karrui karrui commented Oct 17, 2021

Problem

This PR implements a field renderer for the Table field.

Closes #2485, #2621

Solution

Features:

  • feat: add TableField component (and subcomponents)
  • style(Table): add column-stripe table theme variant

Improvements:

  • chore: run lint, remove unused import in Utilities story
  • feat(FormLabel): add overriding FormLabel component props
  • feat(FormLabel): if isRequired is explicitly provided, ignore form control context value.

Before & After Screenshots

New stories have been added in Storybook.

Deploy Notes

New environment variables:

  • env var : env var details

New scripts:

  • script : script details

New dependencies:

  • type-fest : update versions in shared to v2.5.0, install in frontend
  • react-table: headless renderer for tables in React

karrui added 30 commits October 17, 2021 13:54
Want to use new type RequireAllOrNone
when addMoreRows is present, maximumRows must also be present
requires wrapping the entire table with a wrapper (instead of setting display in the table tag) to correctly set width of each table cell
# Conflicts:
#	frontend/package-lock.json
#	frontend/package.json
#	frontend/src/utils/fieldValidation.ts
# Conflicts:
#	frontend/package-lock.json
#	frontend/package.json
@karrui karrui merged commit 208c81d into form-v2/develop Nov 5, 2021
@karrui karrui deleted the form-v2/table-field-component branch November 5, 2021 19:56
@justynoh justynoh mentioned this pull request Oct 5, 2022
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