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

【Playground Flow】グリッド表でのid値検証 #179

Closed
snext1220 opened this issue Apr 12, 2020 · 6 comments
Closed

【Playground Flow】グリッド表でのid値検証 #179

snext1220 opened this issue Apr 12, 2020 · 6 comments
Labels
enhancement 新機能提案* Testing テスト中(現在のテスト対象項目)

Comments

@snext1220
Copy link
Owner

snext1220 commented Apr 12, 2020

snext1220/snext-management#18 (comment) を受けてのIsueです。

・敵のIDはm+数字で書かないと機能しないのですが、数字だけで書いても入力、シーン指定できてしまうので、あとで混乱しました。

アイテム、フラグ、敵、実績のid値を入力値にチェックし、適切なプレフィックスがついていないものに対してエラーを表示するようにしています。GitHubのみ反映済みです。

すべてのチェックは(主にメンテナンス的な意味で)難しいかもしれませんが、「この辺は間違えやすい」などあれば、本Issueにてぜひお教えください。

@snext1220 snext1220 added enhancement 新機能提案* Testing テスト中(現在のテスト対象項目) labels Apr 12, 2020
@snext1220 snext1220 added this to the June 2020 Update milestone Apr 12, 2020
@cocotori
Copy link
Collaborator

警告が出るのはわかりやすくていいですね。

ただ警告が出る状態でもタブを移動できるので、

例えばフラグのタブで間違ってn01と入力して、
敵のタブに移動して、入力しようとすると
「idは「f~」形式で入力してください」と表示されて
少し混乱するので、
「フラグのidは「f~」形式で入力してください」等に
してくれると更に親切かと思いました。

@toki-sor1
Copy link
Collaborator

Win 8.1/Chrome 81.0.4044.129(64bit)にて動作確認。
警告が表示されることで、自分が何を間違えたのかすぐに分かるところが良いと思います。

「フラグのidは「f~」形式で入力してください」等に
してくれると更に親切かと思いました。

確かに現在ではタブを移動しても他のタブの警告が表示されるので、ちょっと混乱しちゃいますね。
ココさんのご意見に賛成です!

@snext1220 snext1220 added Confirmed 議論終了&実装待ち(実装時に仕様修正の可能性あり) and removed Testing テスト中(現在のテスト対象項目) labels May 3, 2020
@snext1220
Copy link
Owner Author

snext1220 commented May 3, 2020

ご確認ありがとうございます。
メッセージ表示については各項目共通になっていたため、項目単位にばらしてみました。
GitHub Pagesのみ反映済みです。

@snext1220 snext1220 added Testing テスト中(現在のテスト対象項目) and removed Confirmed 議論終了&実装待ち(実装時に仕様修正の可能性あり) labels May 4, 2020
@cocotori
Copy link
Collaborator

cocotori commented Jun 6, 2020

確認しました。
よりわかりやすくなったと思います!

@toki-sor1
Copy link
Collaborator

Win8.1/Chrome 83.0.4103.97(64bit)にて動作確認。
正常に表示されていました。分かり易くなって良いと思います!

@Salvadors-cabin
Copy link
Collaborator

Win10+Edge環境で問題ないことを確認しましたー

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 新機能提案* Testing テスト中(現在のテスト対象項目)
Projects
None yet
Development

No branches or pull requests

4 participants