-
Notifications
You must be signed in to change notification settings - Fork 9
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
SIMSBIOHUB 335: Tab Through Observation Edit #1156
Conversation
…elements in the edit view of the observations table to be Tab / Shift+Tab through
Openshift URLs for the PR Deployment: |
Codecov Report
@@ Coverage Diff @@
## dev-staging #1156 +/- ##
===============================================
- Coverage 61.87% 61.81% -0.06%
===============================================
Files 547 549 +2
Lines 16597 16639 +42
Branches 2540 2541 +1
===============================================
+ Hits 10269 10286 +17
- Misses 5645 5665 +20
- Partials 683 688 +5
... and 15 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ran this locally and works great
app/src/components/data-grid/autocomplete/AsyncAutocompleteDataGridEditCell.tsx
Outdated
Show resolved
Hide resolved
app/src/components/data-grid/autocomplete/AutocompleteDataGridEditCell.tsx
Outdated
Show resolved
Hide resolved
SonarCloud Quality Gate failed. 0 Bugs No Coverage information Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally and it works perfectly 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👨🍳 💋
Openshift URLs for the PR Deployment: |
* Modified each custom renderEditCell component of the ObservationTable to manually set hasFocus. This allows the user to Tab forward to the next cell or Shift+Tab back to the previous. * Wrapped some of the simpler TextField / DatePicker renders in their own functional components so that the same hasFocus pattern could be used.
Links to Jira Tickets
Description of Changes