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

WORKFLOW内のAlgorithmブロックが ドラッグに追従してしまう (oist_0214_7) #438

Closed
itutu-tienday opened this issue Mar 23, 2023 · 7 comments · Fixed by #464
Labels
bug Something isn't working frontend For frontend app tasks
Milestone

Comments

@itutu-tienday
Copy link
Collaborator

itutu-tienday commented Mar 23, 2023

Algorithmブロック内の[OUTPUT]ボタンをクリックした後に表示されるモーダル内において、チャートをドラッグすると、WORKFLOW内のAlgorithmブロックも追従して動いてしまいます。
解析結果には影響はないのですが、Algorithmブロックが思わぬ場所に移動していたりするので、不便に感じました

image

  • 事象発生箇所(モーダルのダイアログ・ツールチップが表示される箇所全般)
    • OUTPUTダイアログ
    • SELECT DATAダイアログ
    • エラーログ ダイアログ
    • etc.
@itutu-tienday itutu-tienday added enhancement New feature or request bug Something isn't working and removed enhancement New feature or request labels Mar 23, 2023
@itutu-tienday itutu-tienday added the frontend For frontend app tasks label Mar 30, 2023
@Reactplus-Hao
Copy link

Trong modal được hiển thị sau khi click vào button [OUTPUT] trong block Algorithm,
thì khi drag chat thì block Algorithm trong WORKFLOW cũng bị di chuyển theo.
Phần bug này không ảnh hưởng đến kết quả phân tích, nhưng việc cho di chuyển block Algorithm đến chỗ không mong muốn gây bất tiện.

=> Nên KH nhờ bên mình est trước xem tốn bn effort để fix lỗi này.
Sau đó họ sẽ điều chỉnh độ ưu tiên của task xem có cần xử lý luôn không.

※Nội dung note trên ảnh đính kèm: Sau khi di chuyể drag trong chat, và sau đó back lại WORKFLOW thì block cũng bị drag theo và bị di chuyển đến chỗ khác

@Reactplus-Hao
Copy link

Thông tin bổ sung:

  • Chỗ xảy ra issue này(Tất cả những chỗ hiển thị dialog modal・tooltip)
    ・Dialog OUTPUT
    ・Dialog SELECT DATA
    ・Dialog Log error
    etc.

@Reactplus-Hao
Copy link

@itutu-tienday
このチケットの事象発生箇所はエラーログ ダイアログとツールチップが表示される箇所を含めていますが、
弊社の方でこの2つの箇所を表示する方法が分かりませんので、
どうすればこの2つの箇所(エラーログ ダイアログとツールチップ)を表示させるか、教えて頂けますでしょうか。

@itutu-tienday
Copy link
Collaborator Author

itutu-tienday commented Apr 7, 2023

@Reactplus-Hao
エラーダイアログ(ツールチップ)は、添付イメージのものです。
※添付イメージのように(ここではpipelineがつながっていない)、簡易にエラーを再現可能です。

wf-ss

弊社の方でこの2つの箇所を表示する方法が分かりませんので、

当方で現在認識しているエラーの表示形式は、上記の1種類になります。

@Reactplus-Hao
Copy link

Về phần dialog Error log (Chính là tooltip) thì như ảnh đính kèm dưới đây
※Bên mình có thể dễ dàng tái hiện error như ảnh đính kèm (Ở ảnh này là lỗi đang không kết nối pipeline)

Image

----> KH đang hiểu là hiện tại format hiển thị lỗi chỉ có 1 loại như trên thôi ạ

@Reactplus-Hao
Copy link

Reactplus-Hao commented Apr 7, 2023

@itutu-tienday
#438について、全ての対象箇所に対して修正とテスト実施しました。
テスト実施結果は下記のファイルに記入し、動作のデモ動画は下記のフォルダに格納いたしました。
テスト実施結果ファイルhttps://docs.google.com/spreadsheets/d/1yAeSzsBtXc6YW0si-7OIk9emkZJq59isnjrzEi0UPSQ/edit#gid=0
動作のデモ動画https://drive.google.com/drive/u/0/folders/1vycRQCPVGSiJFICFDFiPmObYo657clvW
お手数ですが、ご確認お願い致します。

@Reactplus-Hao
Copy link

@itutu-tienday
このチケットの原因と内容内容について、下記の通りとなります。

  • 原因
    既存のソースコードでは、ポップアップのコンポーネントがReactFlowライブラリに存在しているためです。
  • 対応内容
    ・ポップアップのコンポーネント(下記の3つの部分)をReactFlowライブラリの外に移動して、関連の関数を修正しました。
        ・AlgorithmOutputDialog, FileSelectDialog
        ・エラーログのダイアログについて、命名されていないため、下記の画像のソースコードをそのままコピーして、ReactFlowライブラリの外に移動しました。

Screen Shot 2023-04-10 at 16 52 57

  • 備考:詳細までレビューしたい時に、再度develop-mainブランチへ他のPRを作成することで、下記の画像のファイルの位置と詳細修正内容を確認できると思います。

Screen Shot 2023-04-10 at 16 53 59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend For frontend app tasks
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants