8870 - Datagrid Inline Editor SelectOnEdit

[tjamesallen15]

Explain the details for making this change. What existing problem does the pull request solve?

This pull request will fix selecting issue on inline editor by adding selectOnEdit setting.

Related github/jira issue (required):
Closes #8870

Steps necessary to review your pull request (required):

• Pull this git repository, build, and run the app
• Go to http://localhost:4000/components/datagrid/test-editable-with-inline-editor.html
• Click the value inside the inline editor
• It should not be select on click

Included in this Pull Request:

• An e2e or functional test for the bug or feature.
• A note to the change log.

[jbrcna]

[n-ace-ancog]

Working on my end.

Screen.Recording.2024-07-25.at.7.07.39.PM.mov

[tmcconechy]

@rcoseng can you test this?

I was thinking that selectOnEdit would focus all the text when you click in. Was this the idea? I think the behavior is correct now but maybe we need this to be the default a selectOnEdit selects all the text.

[ericangeles]

Looks good. I think this is a good way to provide options.

Can you add this to NG as well, @tjamesallen15 ?

[rcoseng]

@rcoseng can you test this?

I was thinking that selectOnEdit would focus all the text when you click in. Was this the idea? I think the behavior is correct now but maybe we need this to be the default a selectOnEdit selects all the text.

@tmcconechy Hi, yes that is the idea. selectOnEdit = true would focus all the text. I have also tested it and it looks good. Thanks!