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

feat(todoapp): アプリの構成要素について #447

Merged
merged 15 commits into from
May 4, 2018
Merged

Conversation

azu
Copy link
Collaborator

@azu azu commented May 4, 2018

#4

  • HTMLとCSSが主な内容
  • このセクションでHTMLとCSSは完成してこれ以降は触らない
  • CSSを外部ファイルにしていいのかが悩みどころ
    • コピーしてもありだとは思う
    • 正直手書きするのは微妙なので内容は書きたくはない感じ
    • todoapp に index.css がある方が構成としてはキレイではあるけど

https://github.com/asciidwango/js-primer/blob/master/source/use-case/todoapp/OUTLINE.md の "アプリの構造"が該当する内容。

このセクションではHTMLでアプリの構造を定義し、CSSでアプリのスタイルを定義しました。
次のセクションではJavaScriptモジュールを作成していき、現在は空であるTodoリストを更新していきます。

現在のTodoアプリのディレクトリ構造は次のようになります。
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

結構この章長い気がするから、毎セクション毎に構造と変更した内容と現在のURLをおいて確認できるようにしたい

--output, -o output path of image

Examples
$ screenshot-dev-tools --url "http://127.0.0.1:8080/final/final/" --output ./output.png
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Firefoxで開発者ツールを開いてスクショを撮るツール

--output, -o output path of image

Examples
$ screenshot --url "http://127.0.0.1:8080/final/final/" --output ./output.png
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Firefoxで開発者ツールなしのスクショを撮るツール

@bot-user
Copy link

bot-user commented May 4, 2018

Deploy preview for js-primer ready!

Built with commit c448ce3

https://deploy-preview-447--js-primer.netlify.com

@azu azu merged commit 940cbd0 into master May 4, 2018
@azu azu deleted the app-structure branch May 4, 2018 11:19
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