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

[DON'T MERGE] LP改修の親ブランチ #19

Closed
wants to merge 6 commits into from
Closed

Conversation

eiciemm
Copy link

@eiciemm eiciemm commented Jan 15, 2021

1/24
本ブランチを切った時にmasterが最新でなかった為にコンフリクトが多発してしまったので、
改めて最新のMasterからLP回収の親ブランチを切り直します。なので、本ブランチはCLOSEになる予定です。

Types of changes

スクリーンショット 2021-01-23 15 57 11

Figmaはこちら

こちらがFrourio LP改修プロジェクトの親ブランチになります。
このPRから子ブランチを切ってコーディング作業を行い、このブランチに対してPull Requestを送ります。

Pull Requestは区切りのいい粒度で出しますので、レビュー後、問題なければマージして頂ければと思います。

最終的にこのブランチをmasterにマージして、作業完了となります。

① メインエリア

  • コーディング作業

② SET UP

  • コーディング作業

③ 型定義駆動開発

  • コードブロック機能を使用して実際のコードを埋め込む(レイアウトに影響するのでコード量は要確認)
  • コーディング作業

④ レスポンシブ対応

  • コーディング作業

⑤ 翻訳

  • 日本語を英文にする作業(量は少ない)

⑥ 全体の調整

  • デザイン周りで細かい調整

@eiciemm eiciemm requested a review from solufa January 15, 2021 14:18
@eiciemm eiciemm self-assigned this Jan 15, 2021
@eiciemm eiciemm added the design / style Involving brand design or UI label Jan 15, 2021
@eiciemm
Copy link
Author

eiciemm commented Jan 16, 2021

@solufa
How to Setupの下にくる「Type-Driven development」では、ソースコードを表示させると思うんですが、このやり方以外に何かありますか?

<pre><code>
  let tmp = document.getElementsByClassName("hoege"); 
  const char="testID"; 
  tmp[0].setAttribute("id", char);
</code></pre>

@solufa
Copy link
Member

solufa commented Jan 16, 2021

コードブロックはこれ使ってるみたいなので加工できるかも
https://github.com/FormidableLabs/prism-react-renderer

Docsはこのあたり
https://v2.docusaurus.io/docs/2.0.0-alpha.66/markdown-features#code-blocks

@su8ru su8ru mentioned this pull request Jan 30, 2021
@eiciemm eiciemm closed this Jan 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design / style Involving brand design or UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants