-
Notifications
You must be signed in to change notification settings - Fork 72
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
Table: Update Edit mode #2443
Table: Update Edit mode #2443
Conversation
✅ Deploy Preview for moduswebcomponents ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@ElishaSamPeterPrabhu Testing using the storybook required me to tab twice to edit the cell. So
Can we remove the extra Tab? |
Since there are two "states" for each cell initially when it appears in table cell the edit mode is not enabled , but when clicking enter or tab on that cell enables it and replaces the previous state with the respective editable mode. |
128eeda
to
c422fcb
Compare
c422fcb
to
0261fd1
Compare
0261fd1
to
28a882c
Compare
…b.com/trimble-oss/modus-web-components into issue-2437/table-tabbing-in-edit-mode
Description
References
#2437
Type of change
How Has This Been Tested?
Checklist