Skip to content

[Chat] Fix for a cursor position during message editing#3233

Merged
vhuseinova-msft merged 6 commits intomainfrom
vhuseinova/message-edit-focus-fix
Jun 20, 2023
Merged

[Chat] Fix for a cursor position during message editing#3233
vhuseinova-msft merged 6 commits intomainfrom
vhuseinova/message-edit-focus-fix

Conversation

@vhuseinova-msft
Copy link
Copy Markdown
Member

What

  • Fixed an issue with a cursor position when multiline parameter is set to true

Why

The cursor position should be set to the end of the string instead of the beggining of it when edit is started

How Tested

Storybook

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
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 20, 2023

Chat bundle size is increased❗.

  • Current size: 2198311
  • Base size: 2198249
  • Diff size: 62

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 20, 2023

CallWithChat bundle size is increased❗.

  • Current size: 10608932
  • Base size: 10608870
  • Diff size: 62

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 20, 2023

Calling bundle size is not changed.

  • Current size: 9966617
  • Base size: 9966617
  • Diff size: 0

@vhuseinova-msft vhuseinova-msft enabled auto-merge (squash) June 20, 2023 16:28
Comment thread packages/react-components/src/components/InputBoxComponent.tsx
@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@vhuseinova-msft vhuseinova-msft merged commit adc7ffb into main Jun 20, 2023
@vhuseinova-msft vhuseinova-msft deleted the vhuseinova/message-edit-focus-fix branch June 20, 2023 17:21
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.

3 participants