Skip to content

Conversation

@nakaterm
Copy link
Contributor

@nakaterm nakaterm commented Oct 25, 2025

「CSS による配置」で扱う内容を抜本的に見直し、例題のテーマも実用的なものに変更した

注意点

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 25, 2025

Deploying utcode-learn with  Cloudflare Pages  Cloudflare Pages

Latest commit: 56a3153
Status: ✅  Deploy successful!
Preview URL: https://3f320183.utcode-learn.pages.dev
Branch Preview URL: https://rewrite-css-arrangement.utcode-learn.pages.dev

View logs

@nakaterm nakaterm force-pushed the rewrite-css-arrangement branch 2 times, most recently from 3f0bd0b to 86bec36 Compare November 1, 2025 12:01
@nakaterm nakaterm force-pushed the rewrite-css-arrangement branch 2 times, most recently from b40ab58 to 4b0f703 Compare November 8, 2025 13:21
@nakaterm nakaterm closed this Nov 8, 2025
@nakaterm nakaterm force-pushed the rewrite-css-arrangement branch from 4b0f703 to cceb0dd Compare November 8, 2025 13:43
@nakaterm nakaterm reopened this Nov 8, 2025
@nakaterm nakaterm marked this pull request as ready for review November 8, 2025 13:52
Copy link
Contributor

@chvmvd chvmvd left a comment

Choose a reason for hiding this comment

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

途中ですが

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request revises the CSS layout documentation (「CSS による配置」を改訂) by simplifying and reorganizing the content. The PR introduces a new, more streamlined tutorial focused on building pricing plan cards while teaching CSS layout fundamentals.

Key Changes:

  • Creates new simplified CSS layout guide with step-by-step card building examples
  • Removes the advanced "CSS arrangement" content (box model, positioning, grid)
  • Adds multiple progressive code samples demonstrating width, border, padding, margin, flexbox, and responsive design

Reviewed Changes

Copilot reviewed 50 out of 80 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
docs/2-browser-apps/05-css-layout/index.mdx New tutorial teaching CSS layout through building pricing cards, covering width/height, borders, padding, margin, flexbox, and media queries
docs/2-browser-apps/05-css-layout/_samples/* Multiple sample code directories added to demonstrate each concept progressively
docs/2-browser-apps/05-css-layout/final-image.png Screenshot showing the completed pricing card layout
docs/2-browser-apps/05-css-arrangement/index.mdx Previous advanced CSS content removed (box model, position, grid)
docs/2-browser-apps/05-css-arrangement/_samples/* All previous sample code removed

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nakaterm
Copy link
Contributor Author

(メモ) 演習問題と、コードブロックのハイライトの CSS 以外を一旦修正

Copy link
Contributor

@chvmvd chvmvd left a comment

Choose a reason for hiding this comment

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

他はわかりやすく正確で非常にいいと思います!

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 50 out of 80 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 62 out of 100 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nakaterm nakaterm force-pushed the rewrite-css-arrangement branch from 7b743f7 to d87255a Compare December 14, 2025 13:44
Copy link
Contributor

@chvmvd chvmvd left a comment

Choose a reason for hiding this comment

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

すみません。操作を誤り、途中でレビューを完了してしまいました。

Copy link
Contributor

@chvmvd chvmvd left a comment

Choose a reason for hiding this comment

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

説明がわかりやすくかなり正確で非常に良いと思います!!!

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 62 out of 99 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@chvmvd chvmvd left a comment

Choose a reason for hiding this comment

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

もう問題ないと思います!
難易度も適切で、説明もわかりやすく、問題と演習問題の例も素敵だと思います。
ありがとうございます。

@nakaterm nakaterm requested a review from chvmvd December 29, 2025 12:49
Copy link
Contributor

@chvmvd chvmvd left a comment

Choose a reason for hiding this comment

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

完璧だと思います!

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