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

規格分類、配送方法、支払い方法の表示/非表示の切り替えとモーダルについて #131

Closed
okazy opened this issue Mar 26, 2018 · 9 comments
Assignees
Projects

Comments

@okazy
Copy link
Contributor

okazy commented Mar 26, 2018

3.n系では論理削除について見直しがされました。
その中で規格分類、配送方法、支払い方法は論理削除の機能がなくなり、物理削除の機能と表示/非表示の切り替えができる機能が加わりました。

対象画面

https://projects.invisionapp.com/d/main#/console/11807544/252428110/preview
https://projects.invisionapp.com/d/main#/console/11807544/269337332/preview
https://projects.invisionapp.com/d/main#/console/11807544/269343176/preview

これらの画面には

  • 表示/非表示の切り替え機能が必要です。
  • 削除は物理削除となるためモーダルで確認を出す必要があります。

こちらプロトタイプの修正をお願いできますでしょうか?

関連

EC-CUBE/ec-cube#2314
EC-CUBE/ec-cube#2491
EC-CUBE/ec-cube#2499
EC-CUBE/ec-cube#2500

@okazy okazy changed the title 規格分類、配送方法、支払い方法の表示/非表示の切り替えについて 規格分類、配送方法、支払い方法の表示/非表示の切り替えとモーダルについて Mar 26, 2018
@okazy okazy assigned okazy and witchdoktor and unassigned okazy Mar 26, 2018
@witchdoktor
Copy link
Collaborator

承知しました。ちょっとすぐに手を動かすことが難しく、今週金曜までに3画面上げる方向で順次プロトタイプを更新していく、という流れでお願いできればと思っております。よろしくお願いいたします!

@okazy
Copy link
Contributor Author

okazy commented Mar 26, 2018

ご確認ありがとうございます。
どうぞよろしくお願いいたします。

@witchdoktor
Copy link
Collaborator

witchdoktor commented Apr 2, 2018

遅くなり申し訳ありません、上記3画面、更新しました。提案の概要としては

  • 各項目右手、操作パネル内に表示 / 非表示ボタン(目のアイコン)を追加
  • 非表示時、項目名に「(非表示)」の文言を追加
  • 同じく非表示時、表示 / 非表示ボタンのアイコンが非表示アイコン(目に斜線のアイコン)に切り替わり、ボタン背景色もpale redに切り替え
  • 削除アイコンをクリック時、項目の削除ダイアログを表示。削除実行時には画面上部にアラートメッセージを出す(アラートメッセージの出現はモックアップに含まれません)

https://projects.invisionapp.com/d/main#/console/11807544/252428110/preview
https://projects.invisionapp.com/d/main#/console/11807544/269337332/preview
https://projects.invisionapp.com/d/main#/console/11807544/269343176/preview

@witchdoktor
Copy link
Collaborator

現在、項目の削除ダイアログには

この項目を完全に削除します
この項目を完全に削除します。これまでの販売データには影響はありませんが、この操作は取り消すことができません。削除しますか?

としていますが、当該データの物理削除によってこれまでの販売データ等への影響の有無はどうでしょうか?

@witchdoktor
Copy link
Collaborator

こちら、OKでしたらひとまずCloseしてくださいませー

@yasui05821
Copy link
Collaborator

yasui05821 commented Jun 12, 2018

@okazy
mocへの反映内容と対象ページについて整理しました。
一度認識に違いないかご確認をお願いします。問題なければ、実作業へと移らせて頂きます。

【mocへの反映内容】

  • 操作エリアに表示切り替えボタン(目のアイコン)を追加

  • 非表示の状態を作成
     - 目のアイコンが変わる(目のアイコンに斜線はいってるもの)
     - 表示切り替えボタンの背景が#FAF1F1
     - 非表示になっている対象のリスト名の後ろに (非表示)というテキストが付与される

  • 削除を実行後の画面の変化
     - モーダルが出現する。
     - 削除の実行後、削除完了アラートが出現(mocでは対応せずシステム側で対応)

【対象ページ】
▼規格管理(詳細)
デザイン:https://projects.invisionapp.com/d/main#/console/11807544/252428110/preview
moc:http://eccube3-admin-styleguide.herokuapp.com/product/standard/detail/

▼支払方法設定
デザイン:https://projects.invisionapp.com/d/main#/console/11807544/269337332/preview
moc:http://eccube3-admin-styleguide.herokuapp.com/setting/basis/payment/

▼配送方法設定
デザイン:https://projects.invisionapp.com/d/main#/console/11807544/252428110/preview
moc:http://eccube3-admin-styleguide.herokuapp.com/setting/basis/delivery/

@yasui05821 yasui05821 self-assigned this Jun 13, 2018
@okazy
Copy link
Contributor Author

okazy commented Jun 14, 2018

@yasui05821
内容確認しました。
まとめていただいた内容でよろしくお願いいたします。

@yasui05821
Copy link
Collaborator

モーダルに関してはissue #132 の方で実装しました。

操作ボタンの追加、「非表示」状態のマークアップをmocで実装しました。
.hideクラスを付与することで背景色が変わるよう調整しています。

devのテスト環境にて実装チェックしました。

▼規格管理(詳細)
http://eccube3-admin-styleguide-dev.herokuapp.com/product/standard/detail/

▼支払方法設定
http://eccube3-admin-styleguide-dev.herokuapp.com/setting/basis/payment/

▼配送方法設定
http://eccube3-admin-styleguide-dev.herokuapp.com/setting/basis/delivery/

@okazy
Copy link
Contributor Author

okazy commented Sep 10, 2018

現状のEC-CUBE本体の仕様でいきます。

@okazy okazy closed this as completed Sep 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
3.n対応
本体反映待ち
Development

No branches or pull requests

4 participants