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

背景をグラデーションにする #368

Merged
merged 2 commits into from
Feb 6, 2024
Merged

Conversation

takashi0602
Copy link
Member

close #308

概要

CSSをグラデーションにしましたので確認お願いします。

※ 当初はコンポーネントで対応しようと思ってましたが、CSSオンリーで対応できたので特にコンポーネント化はしてません。

スクリーンショット

PC

青のグラデーションの背景にofficialの文字が記載されたWebサイトの画面。PCサイズ。

SP

青のグラデーションの背景にofficialの文字が記載されたWebサイトの画面。スマホサイズ。

@takashi0602 takashi0602 self-assigned this Jan 29, 2024
@uyupunpopunpo
Copy link
Contributor

@tyokinuhata
Copy link
Member

こういうのって設計的にコンポーネントじゃなくてグローバルに適用するほうが一般的?

@takashi0602
Copy link
Member Author

こういうのって設計的にコンポーネントじゃなくてグローバルに適用するほうが一般的?

コンポーネントにするとReact.Fragment(<></>こういうやつ)を返すことになるので、グローバルに適応するのが一般的かな
後は、bodyにスタイルを当てたいというところもあるかな

@tyokinuhata
Copy link
Member

コンポーネントにするとReact.Fragment(<></>こういうやつ)を返すことになるので、グローバルに適応するのが一般的かな
後は、bodyにスタイルを当てたいというところもあるかな

なるほど。ちなみにFragmentを返すことは問題あるの?

@takashi0602
Copy link
Member Author

Fragmentを返すこと自体は問題ないけど、ちょっとJSのサイズが大きくなる感じかな

@tyokinuhata
Copy link
Member

コンポーネントにするとReact.Fragment(<></>こういうやつ)を返すことになるので、グローバルに適応するのが一般的かな

ちなみに「適応」じゃなくて「適用」ッス

Copy link
Member

@tyokinuhata tyokinuhata left a comment

Choose a reason for hiding this comment

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

LGTM

@takashi0602 takashi0602 merged commit 60151ab into main Feb 6, 2024
6 checks passed
@takashi0602 takashi0602 deleted the feat/308_layout branch February 6, 2024 02:42
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