タスク更新画面へのemojiパレットの追加

Lycheejam / twe-todo-app

.Net Core todo app for twitter

0 stars 0 forks source link

Closed Lycheejam closed 5 years ago

Lycheejam commented 5 years ago

タスク更新画面にてタスクごとの現在ステータスを入力可能だが現状ではint型で決まったコード値しか入力できない仕様となっている。（0 = " "、1 = "○"、2 = "×"）

元々ボタン形式としていたが操作がしっくりこないため一時的にテキストボックスとしている。

ステータス項目をint型からstring型へ変更し絵文字を入力可能とする変更を実施する。本実装を実施する際に1文字のみ入力可能なバリデーションを同時に実装する必要がある。

Lycheejam commented 5 years ago

0~2のパラメータ方式から1文字制限のTextBoxに変更 8c254100c10ebb285a2661ae4662a0c822f723f6

ただしクライアント側でのバリデーションが未実装

Lycheejam commented 5 years ago

080ac0dd01c6a3a9d58d8dbd0779f1a4a12d86d6 絵文字ピッカーを実装した。またhtmlのmaxLengthタグを使用してステータスに使用できる文字数を1文字とした。

※ASP.NET Coreの機能を使用したバリデーションの実装も行ったが絵文字が4バイト文字であるため1文字の制限をかけると全角2字と判定され入力できない事態が発生した。

Lycheejam commented 5 years ago

絵文字ピッカーにより絵文字の入力が可能となったことからユーザによるキーボードからの入力制限を検討する必要がある。

Lycheejam commented 5 years ago

絵文字ピッカーについて fontawesomeのバージョンが古いことから（2019/3/10現在5.7.2）使用できない絵文字が存在する可能性がある。（未調査）

単純にCDNの参照バージョンを変更するだけでは機能しなかったため OSSへのプルリクも含め時間があれば対応する。 https://github.com/OneSignal/emoji-picker