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

SIMSBIOHUB 335: Tab Through Observation Edit #1156

Merged
merged 6 commits into from
Nov 1, 2023

Conversation

GrahamS-Quartech
Copy link
Contributor

Links to Jira Tickets

Description of Changes

  • 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. It seems that if you don't override the renderEditCell, it does this by default, but it is necessary for any custom component. Docs here.
  • Wrapped some of the simpler TextField / DatePicker renders in their own functional components so that the same hasFocus pattern could be used.

Copy link

codecov bot commented Nov 1, 2023

Codecov Report

Merging #1156 (1fefe0e) into dev-staging (a284963) will decrease coverage by 0.06%.
Report is 1 commits behind head on dev-staging.
The diff coverage is 0.00%.

@@               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     
Files Coverage Δ
...ts/data-grid/taxonomy/TaxonomyDataGridEditCell.tsx 0.00% <ø> (ø)
...autocomplete/AsyncAutocompleteDataGridEditCell.tsx 0.00% <0.00%> (ø)
...grid/autocomplete/AutocompleteDataGridEditCell.tsx 0.00% <0.00%> (ø)
app/src/components/data-grid/TextFieldDataGrid.tsx 0.00% <0.00%> (ø)
...pp/src/components/data-grid/TimePickerDataGrid.tsx 0.00% <0.00%> (ø)
...eatures/surveys/observations/ObservationsTable.tsx 0.00% <0.00%> (ø)

... and 15 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Contributor

@al-rosenthal al-rosenthal left a 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

Copy link

sonarqubecloud bot commented Nov 1, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
8.1% 8.1% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@curtisupshall curtisupshall added the Staging Will merge into staging branch label Nov 1, 2023
Copy link
Contributor

@curtisupshall curtisupshall left a 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

Copy link
Contributor

@JeremyQuartech JeremyQuartech left a 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 👍

Copy link
Contributor

@al-rosenthal al-rosenthal left a comment

Choose a reason for hiding this comment

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

👨‍🍳 💋

@GrahamS-Quartech GrahamS-Quartech marked this pull request as ready for review November 1, 2023 21:49
@GrahamS-Quartech GrahamS-Quartech merged commit d71b919 into dev-staging Nov 1, 2023
29 of 32 checks passed
@GrahamS-Quartech GrahamS-Quartech deleted the SIMSBIOHUB-335 branch November 1, 2023 21:51
KjartanE pushed a commit that referenced this pull request Nov 6, 2023
* 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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Staging Will merge into staging branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants