-
Couldn't load subscription status.
- Fork 96
global.css設定、ヒーローエリア画像追加・実装、不要な画像削除 #57
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
Conversation
|
ローカルで確認しました! ディレクトリに関しては、まだApp Routerのキャッチアップをしてないので何とも言えないのですが、
くらいは分けてもいいかな?と思いました。 Tailwindは設定まわりのこと全然知らないので、自分も調べてみます……! |
|
追記です
を作成して、
するのはどうでしょうか? コンポーネントファイルの作成は各コンポーネント作成者に任せる、という形でタスクの切り分けができるといいかなーと考えてます💭 |
|
さっそくありがとうございます!
良いと思います!余裕があれば、レスポンシブ対応を残課題にしてイシューにしても良いですね!
こちら、srcの中で、appと同じ階層でcomponentsディレクトリを作成するのはどうでしょう? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
下記2点ご確認よろしくお願いいたします🙌
|
遅くなりましたー! 詳細ディレクトリについてfrom @kagomen さん
from @kazzyfrog さん
kagomeさんのとの意見もあわせて ディレクトリ、ちょっとこのプルリク以外でissue立てて整理します! ~ ~ ~ お二人ともありがとうございました! 実装の流れ
とりあえずディレクトリはほぼ仮で決まったような気がするので CSSはとりあえずTailwindCSSをclassNameにちまちまと入れていき ((それまでの間に個人的に理解できていないCSSについて調べられる...時間がかせげる...!というのもあります...笑)) ここで書きましたが、実装の流れDiscordにもっていきますね! |
残課題
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM✌️
|
@kagomen さんにマージしてもらう!!!! Squashで!!!! |
|
LGTM! |
概要
関連するIssue
closed #32
その他
コンポーネントに分けようと考えたのですが、ディレクトリで悩んだので→ issue化 必要なディレクトリ・ファイルを作成する #61page.tsxに記述した参考
残課題
tailwind.config.tsに設定するのか、global.cssに設定するのかわからなかったprimary-color: XXXXとしてどこかで設定して、Tailwind CSSでtext-red-600のように設定しなくて良いようにするfirst-contributions-jaというサイト名を定数化して必要箇所で読み込めるようにしたい上記あたり #62 にて issue化済み