Skip to content

Cursor should show at the end when clicking in dialpad textfield#2576

Merged
carocao-msft merged 5 commits intomainfrom
carocao/dialpad
Dec 7, 2022
Merged

Cursor should show at the end when clicking in dialpad textfield#2576
carocao-msft merged 5 commits intomainfrom
carocao/dialpad

Conversation

@carocao-msft
Copy link
Copy Markdown
Contributor

What

Cursor should show at the end when clicking in dialpad textfield

Why

https://skype.visualstudio.com/SPOOL/_workitems/edit/3015832

How Tested

dialpad.mov

Process & policy checklist

  • I have updated the project documentation to reflect my changes if necessary.
  • I have read the CONTRIBUTING documentation.

Is this a breaking change?

  • This change causes current functionality to break.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 6, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 6, 2022

Calling bundle size is increased❗.

  • Current size: 5475464
  • Base size: 5475382
  • Diff size: 82

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 6, 2022

Chat bundle size is not changed.

  • Current size: 5571811
  • Base size: 5571811
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 6, 2022

CallWithChat bundle size is increased❗.

  • Current size: 5862503
  • Base size: 5862421
  • Diff size: 82

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 6, 2022

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 6, 2022

Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot.

@edwardlee-msft
Copy link
Copy Markdown
Contributor

This is an interesting scenario and not sure if I agree. I can see how this would make sense for a call that is already dialling, but for a call that has yet to be dialled I can see a need to edit the middle of the number.

But regardless, I can see that Alex P provided his input so no issues either.

@carocao-msft
Copy link
Copy Markdown
Contributor Author

This is an interesting scenario and not sure if I agree. I can see how this would make sense for a call that is already dialling, but for a call that has yet to be dialled I can see a need to edit the middle of the number.

But regardless, I can see that Alex P provided his input so no issues either.

To edit the middle of number you can click and then using the left arrow on keyboard to move cursor to the middle. I was concerned about this as well so I made sure we can still do that

Comment thread packages/react-components/src/components/Dialpad/Dialpad.tsx
@carocao-msft carocao-msft enabled auto-merge (squash) December 7, 2022 18:08
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 7, 2022

@carocao-msft carocao-msft merged commit 028b7ce into main Dec 7, 2022
@carocao-msft carocao-msft deleted the carocao/dialpad branch December 7, 2022 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants