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

Adding fiilegroup name label to the input box and auto focus of newly added row #24536

Merged
merged 6 commits into from
Oct 2, 2023

Conversation

ssreerama
Copy link
Contributor

@ssreerama ssreerama commented Sep 28, 2023

This PR adds a label to the filegroup input box and fixes the issue of row filegroup table remove button (where the button adds the rows instead of removing it). Also, Add and remove button automatically focuses on the newly added input box and next row after delete respectively.

NewOneFileGroupUpdate

@corivera
Copy link
Member

Would it be possible to automatically select the newly added row when there are already other rows in the table?

@ssreerama ssreerama requested a review from corivera September 29, 2023 17:11
@ssreerama ssreerama changed the title Sai/filegroups Adding fiilegroup name label to the input box and auto focus of newly added row Oct 2, 2023
Copy link
Contributor

@erinstellato-ms erinstellato-ms left a comment

Choose a reason for hiding this comment

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

Changes looks good, will definitely do some testing when it's merged in!

@ssreerama ssreerama merged commit 549464d into main Oct 2, 2023
6 checks passed
@ssreerama ssreerama deleted the sai/filegroups branch October 2, 2023 17:12
siyao-Siyang pushed a commit that referenced this pull request Oct 12, 2023
… added row (#24536)

* adds nameupdate textbox

* fixed rows table remove button

* input value update on row selection

* using display prop instead of visible css property

* select newly added cell automatically

* sets remove button to next row after deleting a row
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Confusing how to set name of new row in database properties
3 participants