Skip to content

Conversation

@seri-jig
Copy link
Owner

No description provided.

@seri-jig seri-jig requested a review from Copilot August 19, 2025 04:06
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 PR appears to be adding educational content for teaching HTML, CSS, and JavaScript fundamentals. The changes include creating new documentation files and JavaScript examples, along with fixing table formatting issues in existing documentation.

  • Adds comprehensive JavaScript learning materials with practical examples
  • Creates new HTML/CSS documentation with detailed tutorials
  • Fixes markdown table formatting in existing documentation

Reviewed Changes

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

Show a summary per file
File Description
javascript/neta.md New comprehensive JavaScript tutorial covering basic operations, string manipulation, and logical operations with extensive code examples
javascript/memo.js New JavaScript file with mathematical operations examples and demonstrations
javascript/docd_cp.md New detailed JavaScript documentation covering fundamentals from Hello World to advanced topics like async/await and browser APIs
html-css/DOCS.md Minor formatting fix for markdown table structure
html-css/DOCS copy.md New complete HTML/CSS tutorial covering everything from basic structure to responsive design and JavaScript integration
Comments suppressed due to low confidence (5)

javascript/docd_cp.md:820

  • There's a typo in the image path 'igms/../imgs/css-font-family-smple.png'. It should be 'imgs/css-font-family-sample.png' - both 'igms' should be 'imgs' and 'smple' should be 'sample'.
**javascript/docd_cp.md:43**
* The TOC links for grid layout and flexbox are incorrectly numbered as 3-4-1 and 3-4-2, but the actual sections are numbered 3-6-1 and 3-6-2. This will cause broken internal links.
- [7-1. `Promise` / `then()` / `catch()`](#7-1-promise--then--catch)
**javascript/docd_cp.md:44**
* The TOC link for flexbox is incorrectly numbered as 3-4-2, but the actual section is numbered 3-6-2. This will cause a broken internal link.
- [7-2. `async` / `await`](#7-2-async--await)
**javascript/docd_cp.md:1031**
* The section header is numbered 3-4-1 but should be 3-6-1 to match the parent section 3-6 '思いどおりに配置しよう'.

関数定義を一つだけにしたい、await の使用箇所を一つにしたいという理由で非推奨なコードをサンプルで使用しました。

**javascript/docd_cp.md:1092**
* The section header is numbered 3-4-2 but should be 3-6-2 to match the parent section 3-6 '思いどおりに配置しよう'.

const geolocation = navigator.geolocation;

</details>



---

<sub>**Tip:** Customize your code reviews with copilot-instructions.md. <a href="/seri-jig/intern-dev-tutorial/new/main/.github?filename=copilot-instructions.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Create the file</a> or <a href="https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">learn how to get started</a>.</sub>

print("iは1です") # さらに深いインデント(ネストしたif文)
print("ループ終了") # インデントなし(for文の外)
```

Copy link

Copilot AI Aug 19, 2025

Choose a reason for hiding this comment

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

The code block is marked as Python but contains Python syntax examples in a JavaScript tutorial file. This should either be removed if not relevant to JavaScript learning or properly labeled.

Suggested change

Copilot uses AI. Check for mistakes.
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