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

テーブルにbackground-colorが付いているとsuggestiveShadowが見えない #34

Closed
airi-miyamoto opened this issue Sep 13, 2024 · 3 comments

Comments

@airi-miyamoto
Copy link

概要

  • suggestiveShadowがbackgroundのlinear-gradientで設定されているため、テーブルに不透明な背景色が付いているとその部分だけ影が見えない

Screenshot by Dropbox Capture

解決法提案

  • suggestiveShadowがtrueの場合、.scroll-hintをラップする要素を追加し、その::before,::afterでshadowを設定する。

JavaScript

  • defaultsにscrollHintShadowWrapClassとして.scroll-hint-shadow-wrapを追加
  • suggestiveShadowtrueの場合、.scroll-hintをラップする.scroll-hint-shadow-wrapを追加

CSS

  • 擬似要素はpositionで設定し、pointer-events: noneにする。
  • JavaScript側の設定によるが、.scroll-hint-shadow-wrap自体にアクティブクラスがついた時か、.scroll-hintにアクティブクラスがついた時にshadowをvisibility: visible, opacity: 1で見えるようにする。

▼scroll-hint.css

.scroll-hint-shadow-wrap {
  position: relative;
}

.scroll-hint-shadow-wrap::after {
  content: "";
  width: 20px;
  height: 100%;
  background: linear-gradient(270deg, rgba(0, 0, 0, .15) 0, rgba(0, 0, 0, 0) 16px, rgba(0, 0, 0, 0));
  position: absolute;
  top: 0;
  right: 0;
  pointer-events: none;
  z-index: 1;
  opacity: 0;
  visibility: hidden;
}

.scroll-hint-shadow-wrap::before {
  content: "";
  width: 20px;
  height: 100%;
  background: linear-gradient(90deg, rgba(0, 0, 0, .15) 0, rgba(0, 0, 0, 0) 16px, rgba(0, 0, 0, 0));
  position: absolute;
  top: 0;
  left: 0;
  pointer-events: none;
  z-index: 1;
  opacity: 0;
  visibility: hidden;
}

.scroll-hint-shadow-wrap:has(.scroll-hint.is-right-scrollable)::after,
.scroll-hint-shadow-wrap.is-right-scrollable::after
 {
  opacity: 1;
  visibility: visible;
}

.scroll-hint-shadow-wrap:has(.scroll-hint.is-left-scrollable)::before,
.scroll-hint-shadow-wrap.is-left-scrollable::before
 {
  opacity: 1;
  visibility: visible;
}
@uidev1116
Copy link
Contributor

@airi-miyamoto
確認が遅れてすみません。提案内容を拝見させていただきました。問題ないかと思うので、プルリクエストお待ちしております!

また、今後は issue なしで最初からプルリクエスト送っていただいても大丈夫です!なにか質問事項や懸念点があるときに issue を利用していただけるとありがたいです!

@pixeliumjp
Copy link

@airi-miyamoto
@uidev1116
進捗が見られないため、私が修正を行ってみました。
PRすること自体が初めてで間違い等があるかもしれませんが、動作確認の上、マージをお願いいたします。

uidev1116 added a commit that referenced this issue Jan 14, 2025
#34 の不具合に対する修正
@uidev1116
Copy link
Contributor

#36 で解決済みとして close させていただきます。

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

No branches or pull requests

2 participants