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

Add Click-To-Edit functionality #250

Merged
merged 3 commits into from
Sep 26, 2024
Merged

Add Click-To-Edit functionality #250

merged 3 commits into from
Sep 26, 2024

Conversation

schreiaj
Copy link
Collaborator

@schreiaj schreiaj commented Sep 25, 2024

  • Adds the click to edit functions for the review and edit page

    • Return will commit changes
    • Escape will revert changes since last commit
    • Clicking outside of the text box will commit changes
  • If there are errors Submit button on review page will be disabled

  • Hovering over content will display icon (transition of icon will respect
    reduce motion preferences)

  • Minor tsconfig tweak to remove error when importing
    syntheticDefaultImports

  • Use SortableTable on Review page

  • Update SortableTable to pass formatter function (value, idx, row) to allow more complex formatting

  • Fix bug with Sortable Table passing in wrong index to formatter
    functions

Demo:
Kapture 2024-09-25 at 15 14 31

Updates SortableTable to pass in index and entire row data into
formatters

Use SortableTable on Review page

Create custom formatter for adding exclamation icon and coloring rows
red
Adds the click to edit functions for the review and edit page
Return will commit changes
Escape will revert changes since last commit
Clicking outside of the text box will commit changes

If there are errors Submit button on review page will be disabled

Hovering over content will display icon (transition of icon will respect
reduce motion preferences)

Minor tsconfig tweak to remove error when importing
syntheticDefaultImports

Fix bug with Sortable Table passing in wrong index to formatter
functions
@schreiaj schreiaj marked this pull request as ready for review September 25, 2024 20:15
@schreiaj schreiaj added this pull request to the merge queue Sep 26, 2024
Merged via the queue into main with commit 21d86a1 Sep 26, 2024
4 checks passed
@schreiaj schreiaj deleted the schreiaj/click-to-edit branch September 26, 2024 17:54
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.

2 participants