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

【PgFlow】Flow2Editor移動時のグレーアウト #305

Open
snext1220 opened this issue Aug 8, 2021 · 10 comments
Open

【PgFlow】Flow2Editor移動時のグレーアウト #305

snext1220 opened this issue Aug 8, 2021 · 10 comments
Labels
enhancement 新機能提案* Testing テスト中(現在のテスト対象項目)

Comments

@snext1220
Copy link
Owner

snext1220 commented Aug 8, 2021

#297 (comment) 由来のIssueです。
Flow2Editor/Editor2Flowでタブを移動した時に、移動元のウィンドウをグレーアウトしてみました。
操作できなくなるわけではありませんが、アクティブでないことが明確なので、誤操作はなくなると思われます。
あくまで簡易な誤操作防止機能ですが、如何なものでしょうか。

# 最初はタブ自体のクローズを検討していたのですが、セキュリティの問題でコードが複雑になりそうだったので、このような形に改めています。

@snext1220 snext1220 added enhancement 新機能提案* Compatibility 現在/将来の互換性に影響すると思われる変更 labels Aug 8, 2021
@snext1220 snext1220 added this to the 2021 Autumn Update milestone Aug 8, 2021
@snext1220 snext1220 changed the title 【PgFlow】Flow2Editor移動時のタブクローズ 【PgFlow】Flow2Editor移動時のグレーアウト Aug 8, 2021
@snext1220 snext1220 added Testing テスト中(現在のテスト対象項目) and removed Compatibility 現在/将来の互換性に影響すると思われる変更 labels Aug 8, 2021
@toki-sor1
Copy link
Collaborator

Win 8.1/Chrome 92.0.4515.131(64bit)にて動作確認。
移動元のウィンドウがグレーアウトし、作業中のウィンドウはグレーアウトしていないことを確認しました!
うわーーー早速のご対応ありがとうございます!!
編集中かそうでないかが一目で分かるので、これで誤操作も防げます~!助かりました!!

@snext1220
Copy link
Owner Author

@toki-sor1 さん:
こちらこそご指摘&早速のご確認ありがとうございました!
Flow/Editor連携はまだまだ課題もありそうですが、少しずつ直していければと思うので、引き続きお気づきの点は何なりとです~

@cocotori
Copy link
Collaborator

chromeで確認。

editor,flowともに移動した後、移動元がグレーアウトされています。

@snext1220
Copy link
Owner Author

snext1220 commented Aug 11, 2021

@cocotori さん:
ご確認ありがとうございます!

#306 (comment) で戴いていた件、確かに初見の方にとってはギョッとするかもなので、移動元で、他が編集中になっている旨を示すトーストを追加しました。如何なものでしょうか。

@toki-sor1
Copy link
Collaborator

更新お疲れ様です!
現在このように本番環境Playgroundがズレて表示されていますが、私の環境だけでしょうか;;
image

@snext1220
Copy link
Owner Author

とと、大変失礼しました。
最後に行った微細な修正でミスが混在していたのを気づいていませんでした;(キャッシュ...)
先ほど再修正しましたので、ご確認くださいませ~

@toki-sor1
Copy link
Collaborator

お忙しいところ修正お疲れ様です~!
キャッシュ削除後、Playground FLowが正常に表示されているのを確認しました!

確かに初めてグレーアウトしているのを見るとビックリしてしまう可能性がありますね!
トーストも正常に表示されていましたが、良いと思います!

@snext1220
Copy link
Owner Author

ご確認ありがとうございます!
そして、トーストの件も良いとの事、ありがとうございます~

オマケですが、Editor2Flow/Flow2Editorで行き来する際の「再読み込みしますか?」アラームがうるさく感じられたため、カットしました(正確にはアクティブ状態の場合のアンロードは警告しますが、非アクティブ状態のアンロードはそのまま通します~)。

@toki-sor1
Copy link
Collaborator

なるほど、非アクティブ状態だと閉じるだけなので、アラームカット良いと思います!(正常に動作してました!)

@snext1220
Copy link
Owner Author

ご確認&ご賛同ありがとうございます!!

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

3 participants