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

見た目がヤバいなという感じだった部分を直した #36

Merged
merged 2 commits into from
Dec 26, 2020

Conversation

KanadeNishizawa
Copy link
Contributor

やったこと

見た目があまりにもヤバいなという感じだった部分を直した。

  • Twin:teとは?の説明用の画像の素材を作り直して差し替えた
  • 不要そうな装飾を削除した(Q&Aの吹き出しとか特徴欄のbackground-colorとか)
  • paddingやmarginのremで良さそうな部分を置き換えた
  • 画面によって大きすぎたりしてしまっていた要素を直した
after before

気になった部分を取り急ぎやったので、提供してもらった大学の画像に差し替えたりとかその他もう少し時間がかかりそうな修正事項は後日別commitでやります。

確認したこと

  • 動く

備考

今回は単なる微修正なので、以前やると言っていたlanding pageの刷新ではないです。
次verのTwin:teリリースにあたってLP自体は新しく作り替えなくてはいけなくなるので、vueへの移行とか大幅なデザインの変更はこのLPではもうやらなくて良いかなという気持ちです。次のLPではちゃんとしたいです。

Copy link
Member

@HikaruEgashira HikaruEgashira left a comment

Choose a reason for hiding this comment

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

コラ画像のサイズが大きいのとかも気になったけどそれはオッケーにしとく!
browserslist使ったのはどこ情報ですか?

package.json Outdated Show resolved Hide resolved
src/index.html Outdated Show resolved Hide resolved
@KanadeNishizawa
Copy link
Contributor Author

コラ画像のサイズが大きいのとかも気になったけどそれはオッケーにしとく!
browserslist使ったのはどこ情報ですか?

押忍、直した
yarn start時にUnknown versionでerrorになったブラウザを片っ端から消してました・・・。
これ→ parcel-bundler/parcel#5164

@KanadeNishizawa
Copy link
Contributor Author

コラ画像のサイズが大きいのとかも気になったけどそれはオッケーにしとく!

画像サイズが大きいのはただの圧縮忘れでしたわ・・・
直しといた

Copy link
Member

@HikaruEgashira HikaruEgashira left a comment

Choose a reason for hiding this comment

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

:good_higero:

@KanadeNishizawa KanadeNishizawa force-pushed the kanade/update-styles branch 2 times, most recently from e3461ca to 86a9607 Compare December 26, 2020 10:05
@KanadeNishizawa KanadeNishizawa merged commit cca60b5 into next Dec 26, 2020
@KanadeNishizawa KanadeNishizawa deleted the kanade/update-styles branch December 26, 2020 10:27
@KanadeNishizawa KanadeNishizawa restored the kanade/update-styles branch December 27, 2020 17:08
@KanadeNishizawa KanadeNishizawa deleted the kanade/update-styles branch December 27, 2020 17:41
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.

2 participants