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

Productsの実装 #126

Merged
merged 8 commits into from
Apr 17, 2021
Merged

Productsの実装 #126

merged 8 commits into from
Apr 17, 2021

Conversation

yamachu
Copy link
Contributor

@yamachu yamachu commented Apr 12, 2021

close: #98

@YuMuuu
Copy link
Contributor

YuMuuu commented Apr 12, 2021

RangeError: WebAssembly.Memory(): could not allocate memory
CIガチャっぽい

https://github.com/oystersjp/oystersjp.github.io/pull/126/checks?check_run_id=2319747585#step:6:19

↑違った

@YuMuuu
Copy link
Contributor

YuMuuu commented Apr 12, 2021

cyrilwanner/next-optimized-images#196 (comment)

cyrilwanner/next-optimized-images#186 (comment)

この変が原因かもしれないです

@mii288
Copy link
Member

mii288 commented Apr 14, 2021

#128 直してみました
画像リサイズしたらエラーでなくなりました 🤔

Products section
==================================================================*/

.products-tags:not(:last-child) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

poem: 書き手の好みですが、 :not 使うと詳細度高くなるので、隣接セレクタでかいても良いかもしれないですね。
(このデザインいつまで使うのかという問題もありますがw)

.products-tags + .products-tags {
  margin-left: 4px;
}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

未だに詳細度の優先度わかってないからなるほどなとなった
高すぎると上書きづらいし良さそう

yamachu added 2 commits April 15, 2021 23:47
Productsの画像のリサイズ(CIのエラー修正)
#126 (comment)
詳細度高いと上書きづらいし良さそうに思える
@yamachu yamachu merged commit 97d262c into master-nextjs Apr 17, 2021
@yamachu yamachu deleted the feat/products branch April 17, 2021 10:16
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

Successfully merging this pull request may close these issues.

3 participants