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

チャットに改行を入力・表現可能に #19

Closed

Conversation

blhsrwznrghfzpr
Copy link
Contributor

@blhsrwznrghfzpr blhsrwznrghfzpr commented Jul 29, 2018

#概要
チャットメッセージ内の改行を表現するものです。またそれに加えて、チャットウィンドウからの入力にて改行を行えるように機能修正しています。
チャットウインドウからの入力にて、Ctrl+Enter Shift+Enterで改行を入力できるようになります。

テスト環境: http://yoshis-islands.sakura.ne.jp/udondev/

@TK11235 TK11235 self-requested a review July 31, 2018 13:14
@TK11235
Copy link
Owner

TK11235 commented Jul 31, 2018

PRありがとうございます。
現在作業中という認識で良いのでしょうか。

動作確認してみたのですが、<textarea>でのCtrl+Enterはまだ動作していないようです。

ChatMessageComponentの改行表示は正しく動作しており、
改行を含むテキストを<textarea>にコピー&ペーストして送信する場合は改行が反映されます。

@blhsrwznrghfzpr
Copy link
Contributor Author

blhsrwznrghfzpr commented Jul 31, 2018

確認ありがとうございます。作業は終了しています。
PRの文中の「Ctrl+Enterで改行を入力」というのは記載ミスで、正しくは「Shift+Enterで改行を入力」でした。申し訳ありません。

動作のイメージは

  • 通常のチャット送信時は通常のEnter
  • 改行入力時はShift+Enter

で、Ctrl+Enterには何も制御を入れてません。

@TK11235
Copy link
Owner

TK11235 commented Aug 1, 2018

  • 通常のチャット送信時は通常のEnter
  • 改行入力時はShift+Enter

了解です。こちらも確認が十分ではありませんでした。

コードは問題ありませんのでdevelop/1.6.xブランチにマージします。
v1.6.0の機能追加に含めてリリースする予定です。

※補足
チャット欄におけるShift+EnterCtrl+EnterAlt+Enterの挙動については議論の余地があるかもしれません。まずは実装してみて、問題が出たら考えます。

@TK11235 TK11235 closed this Aug 2, 2018
@blhsrwznrghfzpr blhsrwznrghfzpr deleted the chat-show-newline branch August 2, 2018 14:02
usaturn pushed a commit to usaturn/udonarium that referenced this pull request Aug 7, 2021
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.

2 participants