-
Notifications
You must be signed in to change notification settings - Fork 0
Seri/teach js html css #1
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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文の外) | ||
| ``` | ||
|
|
Copilot
AI
Aug 19, 2025
There was a problem hiding this comment.
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.
No description provided.