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

TOPページ お知らせアニメーション #3765

Closed
wants to merge 2 commits into from

Conversation

takashi0602
Copy link
Contributor

@takashi0602 takashi0602 commented Sep 5, 2018

概要(Overview・Refs Issue)

close EC-CUBE/Eccube-Styleguide#212
TOPページ内にある、お知らせのアニメーションの修正

実装に関する補足(Appendix)

動きがおかしかったのは、 margin がアニメーション前になかったからと、アニメーションの動きが max-height: 100vh; になっていたからです。なので、

  • アニメーション前にmarginを付ける。
  • height の値を取得し、 max-height: 0; にする。そして、ボタンを押すと取得した値を max-height の値に変え、再度ボタンを押すと max-height: 0; に変える。

この2つのように変更しました。

相談(Discussion)

現在、リロードせず画面サイズを変更すると、お知らせの説明文の行数が変わり、 overflow: hidden; の影響で、説明文が途中で切れてしまいます。
原因は最初に height の値を取得しているので、途中で height の値が変わると対応できないからです。

@takashi0602
Copy link
Contributor Author

@mikakane レビューお願いします。

@takashi0602 takashi0602 changed the title TOPページ お知らせアニメーション WIP TOPページ お知らせアニメーション Sep 5, 2018
@ryo-endo ryo-endo added this to the 4.0 milestone Sep 6, 2018
@takashi0602 takashi0602 changed the title WIP TOPページ お知らせアニメーション TOPページ お知らせアニメーション Sep 10, 2018
@okazy okazy self-assigned this Sep 11, 2018
@okazy
Copy link
Contributor

okazy commented Sep 11, 2018

以下の問題があったので別のプルリクで修正しました。
#3812

  • 新着情報の�非表示、削除等を行うと不具合が発生する。
  • scssに不要なコードが残っていた
  • アニメーションのjsが冗長
  • htmlタグのidが被りそうなのでつけたくない

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TOPページ お知らせアニメーション
3 participants