boxed-sheet:0.1.2#3868
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request updates the Boxed-Sheet package from version 0.1.1 to version 0.1.2. The package provides a colorful and organized cheatsheet template for Typst that allows users to create overview documents of learning notes with colored bounding boxes and line titles.
Changes:
- Updated version number from 0.1.1 to 0.1.2
- Minor change to package description ("A automatical" to "An automatical")
- All core functionality files (lib.typ, template files, README, LICENSE) are included for the new version
Reviewed changes
Copilot reviewed 6 out of 14 changed files in this pull request and generated 26 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/preview/boxed-sheet/0.1.2/typst.toml | Package metadata file with version 0.1.2, updated description |
| packages/preview/boxed-sheet/0.1.2/src/lib.typ | Core library implementing boxedsheet functions and styling |
| packages/preview/boxed-sheet/0.1.2/template/boxedsheet.typ | Template example using JavaScript cheat sheet content |
| packages/preview/boxed-sheet/0.1.2/template/boxedsheet-scaling.typ | Alternative template with custom page sizing |
| packages/preview/boxed-sheet/0.1.2/README.md | Documentation for package usage and configuration |
| packages/preview/boxed-sheet/0.1.2/LICENSE | MIT License for the package |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ```js | ||
| Wed Jun 11 2025 18:31:19 GMT+0800 (中国标准时间) | ||
| var d = new Date(); | ||
| 1749637879070 miliseconds passed since 1970 |
There was a problem hiding this comment.
The word "miliseconds" is misspelled. It should be "milliseconds".
| 1749637879070 miliseconds passed since 1970 | |
| 1749637879070 milliseconds passed since 1970 |
| var esc = 'I don\'t \n know'; // \n new line | ||
| var len = abc.length; // string length | ||
| abc.indexOf("lmno"); // find substring, -1 if doesn't contain | ||
| abc.lastIndexOf("lmno"); // last occurance |
There was a problem hiding this comment.
The word "occurance" is misspelled. It should be "occurrence".
| abc.lastIndexOf("lmno"); // last occurance | |
| abc.lastIndexOf("lmno"); // last occurrence |
| authors = ["Zhihao Li <https://lzhms.github.io>"] | ||
| repository = "https://github.com/LZHMS/boxed-sheet.git" | ||
| license = "MIT" | ||
| description = "An automatical colored and well organised cheatsheet template for writing daily notes with colored bounding box and line titles." |
There was a problem hiding this comment.
The word "automatical" should be "automatic" or "automatically". In this context, "automatically" would be more appropriate as it's an adverb modifying "colored".
| description = "An automatical colored and well organised cheatsheet template for writing daily notes with colored bounding box and line titles." | |
| description = "An automatically colored and well organised cheatsheet template for writing daily notes with colored bounding box and line titles." |
| !(a == b) // logical not | ||
| a != b // not equal | ||
| typeof a // type (number, object, function...) | ||
| x << 2 x >> 3 // minary shifting |
There was a problem hiding this comment.
The word "minary" is misspelled. It should be "binary".
| x << 2 x >> 3 // minary shifting | |
| x << 2 x >> 3 // binary shifting |
| ```js | ||
| Wed Jun 11 2025 18:31:19 GMT+0800 (中国标准时间) | ||
| var d = new Date(); | ||
| 1749637879070 miliseconds passed since 1970 |
There was a problem hiding this comment.
The word "miliseconds" is misspelled. It should be "milliseconds".
| 1749637879070 miliseconds passed since 1970 | |
| 1749637879070 milliseconds passed since 1970 |
| !(a == b) // logical not | ||
| a != b // not equal | ||
| typeof a // type (number, object, function...) | ||
| x << 2 x >> 3 // minary shifting |
There was a problem hiding this comment.
The word "minary" is misspelled. It should be "binary".
| x << 2 x >> 3 // minary shifting | |
| x << 2 x >> 3 // binary shifting |
| ``` | ||
|
|
||
| #inline("While Loop") | ||
| ```python |
There was a problem hiding this comment.
The code block is incorrectly marked as python but contains JavaScript code. It should be marked as js to match the other code blocks in this JavaScript cheat sheet.
| ```python | |
| ```js |
| #inline("Defination") | ||
| + `var` defines the variable in the function scope and become global variable if it's defined in the outside of function. It can be used with the value of `undefined` before defination and be alse defined repeatly. | ||
| + `let` defines the variable in the block scope, such as `for`, `if` `while` or `{}`. It cann't be used before defination and not be defined repreatly. |
There was a problem hiding this comment.
The word "defination" is misspelled. It should be "definition".
| #inline("Defination") | |
| + `var` defines the variable in the function scope and become global variable if it's defined in the outside of function. It can be used with the value of `undefined` before defination and be alse defined repeatly. | |
| + `let` defines the variable in the block scope, such as `for`, `if` `while` or `{}`. It cann't be used before defination and not be defined repreatly. | |
| #inline("Definition") | |
| + `var` defines the variable in the function scope and become global variable if it's defined in the outside of function. It can be used with the value of `undefined` before definition and be alse defined repeatly. | |
| + `let` defines the variable in the block scope, such as `for`, `if` `while` or `{}`. It cann't be used before definition and not be defined repreatly. |
| var esc = 'I don\'t \n know'; // \n new line | ||
| var len = abc.length; // string length | ||
| abc.indexOf("lmno"); // find substring, -1 if doesn't contain | ||
| abc.lastIndexOf("lmno"); // last occurance |
There was a problem hiding this comment.
The word "occurance" is misspelled. It should be "occurrence".
| abc.lastIndexOf("lmno"); // last occurance | |
| abc.lastIndexOf("lmno"); // last occurrence |
| x = addNumbers(1, 2); | ||
| ``` | ||
| #inline("Edit DOM element") | ||
| Code for modifying the DOM (Document Object Model). `JavaScript` code will be execute to dynamicly change the HTML elements. |
There was a problem hiding this comment.
The word "dynamicly" is misspelled. It should be "dynamically".
| Code for modifying the DOM (Document Object Model). `JavaScript` code will be execute to dynamicly change the HTML elements. | |
| Code for modifying the DOM (Document Object Model). `JavaScript` code will be execute to dynamically change the HTML elements. |
|
Do you want to fix the typos reported by Copilot before merging or are you ignoring its review? |
|
Yes. I have fixed the typos reported by Copilot. |
I am submitting
Description: Explain what the package does and why it's useful.
I have read and followed the submission guidelines and, in particular, I
typst.tomlfile with all required keysREADME.mdwith documentation for my packageLICENSEfile or linked one in myREADME.mdexcluded PDFs or README images, if any, but not the LICENSE