Conversation
c40503a to
eca354e
Compare
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
c40503a to
eca354e
Compare
b1b0bbc to
eca354e
Compare
10979aa to
eca354e
Compare
|
Pull request #4 has conflicts that needs to be resolved. |
|
Pull request #4 has conflicts that needs to be resolved. |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 229 out of 2430 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…nt rules, and adding oxlint package rule
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 229 out of 2430 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…mmutable header - Replace static FeatureCarousel/FeatureShowcase imports with defineAsyncComponent + useIntersectionObserver (rootMargin: 400px) so their JS chunks are excluded from the home page initial bundle and fetched only as the #explore section approaches the viewport. Handles direct #explore anchor navigation via onMounted hash check. Adds min-h-200 (800px) to the section to prevent CLS during lazy load. - Remove Inter font weight 900 from nuxt.config.ts: font-black has zero usages across the codebase (confirmed by grep). - Add /_nuxt/* → Cache-Control: public, max-age=31536000, immutable to public/_headers — Nuxt Vite build chunks go to /_nuxt/, not /assets/*, so the existing /assets/* rule never matched them on Netlify CDN.
|
@coderabbitai summary |
✅ Actions performedSummary regeneration triggered. |
…ble zero-JS button loading Co-authored-by: RedStar071 <76824516+RedStar071@users.noreply.github.com>
…state loads Co-authored-by: RedStar071 <76824516+RedStar071@users.noreply.github.com>
…t for layout shift fix Co-authored-by: RedStar071 <76824516+RedStar071@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 229 out of 2430 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.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 229 out of 2431 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| | Name | Type | | ||
| | ------- | ----- | | ||
| | on | `Ref` | | ||
| | off | `Ref` | | ||
| | trigger | `Ref` | | ||
| | clear | `Ref` | |
There was a problem hiding this comment.
The return types are documented as Ref, but these are methods on the hook (functions). Please update the table to reflect callable types and/or the correct EventHook type shape so consumers don’t misunderstand how to use the API.
| | Name | Type | | |
| | ------- | ----- | | |
| | on | `Ref` | | |
| | off | `Ref` | | |
| | trigger | `Ref` | | |
| | clear | `Ref` | | |
| | Name | Type | | |
| | ------- | ---------------------------------- | | |
| | on | `(fn: (param: T) => void) => void` | | |
| | off | `(fn: (param: T) => void) => void` | | |
| | trigger | `(param: T) => void` | | |
| | clear | `() => void` | |
| ## ::component-code | ||
|
|
||
| ignore: | ||
|
|
||
| - items | ||
| - class | ||
| - defaultValue | ||
| external: | ||
| - items | ||
| externalTypes: | ||
| - TimelineItem[] | ||
| props: | ||
| defaultValue: 2 | ||
| items: - date: 'Mar 15, 2025' | ||
| title: 'Project Kickoff' | ||
| description: 'Kicked off the project with team alignment. | ||
| - `color`: to change the color of the active items in a Timeline. |
There was a problem hiding this comment.
This section appears malformed/corrupted (indentation, YAML-ish blocks, and an unclosed quoted string on description). As written, it will likely render incorrectly and is hard to maintain. Please fix the formatting (proper fenced block / valid MDC component syntax) and ensure strings are closed so the doc page parses cleanly.
…ding users, guilds, channels, members, roles, settings, and health check
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 229 out of 2440 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| | Option | Type | Default | Description | | ||
| | ------------ | ------------------------------------------------------- | ------- | --------------------------------------------------- | | ||
| | ignore | `MaybeRefOrGetter<(MaybeElementRef \| string)[]>` | - | List of elements that should not trigger the event, | | ||
| | capture | `boolean` | true | Use capturing phase for internal event listener. | | ||
| | detectIframe | `boolean` | false | Run handler function if focus moves to an iframe. | | ||
| | controls | `Controls` | false | Use controls to cancel/trigger listener. | |
There was a problem hiding this comment.
This doc misrepresents the API shape: onClickOutside(...) returns a stop function by default, and controls changes the return value to include control functions—these are not Refs. Also, the controls option type/default is inconsistent (Controls vs false). Please align the options and returns tables with the actual VueUse typings (functions vs refs, and the conditional return when controls is enabled).
| Wrappers around RxJS's and to allow them to accept s. | ||
|
|
||
| **Package:** `@vueuse/rxjs` | ||
| **Category:** '@RxJS' | ||
|
|
||
| ## Reference | ||
|
|
||
| [VueUse Docs](https://vueuse.org/core/from/) |
There was a problem hiding this comment.
The description is truncated/garbled (RxJS's and / accept s.) which makes the page hard to use. Also, since this is @vueuse/rxjs, the reference URL likely shouldn't be under /core/. Please rewrite the one-line summary with complete terms (e.g., which RxJS creators/operators are wrapped and what inputs they accept) and update the docs link to the correct section.
| Wrappers around RxJS's and to allow them to accept s. | |
| **Package:** `@vueuse/rxjs` | |
| **Category:** '@RxJS' | |
| ## Reference | |
| [VueUse Docs](https://vueuse.org/core/from/) | |
| Wrappers around RxJS's `from` and `fromEvent` creators to allow them to accept Vue refs and other reactive sources. | |
| **Package:** `@vueuse/rxjs` | |
| **Category:** '@RxJS' | |
| ## Reference | |
| [VueUse Docs](https://vueuse.org/rxjs/from/) |
| Wrappers around RxJS's and to allow them to accept s. | ||
|
|
||
| **Package:** `@vueuse/rxjs` | ||
| **Category:** '@RxJS' | ||
|
|
||
| ## Reference | ||
|
|
||
| [VueUse Docs](https://vueuse.org/core/from/) |
There was a problem hiding this comment.
The description is truncated/garbled (RxJS's and / accept s.) which makes the page hard to use. Also, since this is @vueuse/rxjs, the reference URL likely shouldn't be under /core/. Please rewrite the one-line summary with complete terms (e.g., which RxJS creators/operators are wrapped and what inputs they accept) and update the docs link to the correct section.
| Wrappers around RxJS's and to allow them to accept s. | |
| **Package:** `@vueuse/rxjs` | |
| **Category:** '@RxJS' | |
| ## Reference | |
| [VueUse Docs](https://vueuse.org/core/from/) | |
| Wrappers around RxJS `from` and `fromEvent` creators that allow them to accept Vue refs, computed refs, and other reactive sources. | |
| **Package:** `@vueuse/rxjs` | |
| **Category:** '@RxJS' | |
| ## Reference | |
| [VueUse Docs](https://vueuse.org/rxjs/from/) |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 229 out of 2440 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ## Returns | ||
|
|
||
| | Name | Type | | ||
| | ---- | ----- | | ||
| | next | `Ref` | | ||
| | done | `Ref` | |
There was a problem hiding this comment.
makeDestructurable does not return { next, done } refs; this 'Returns' section appears to be from an iterator API and is misleading. Replace it with the actual return shape/behavior of makeDestructurable (and ideally include a short usage snippet).
| | Name | Type | | ||
| | ------ | ----- | | ||
| | stop | `Ref` | | ||
| | cancel | `Ref` | |
There was a problem hiding this comment.
The 'Returns' table documents stop/cancel as Ref, but onClickOutside returns a stop function (or control functions when controls is enabled). Update the return types to reflect callable functions (and consider documenting when cancel exists).
| | Name | Type | | |
| | ------ | ----- | | |
| | stop | `Ref` | | |
| | cancel | `Ref` | | |
| | Name | Type | | |
| | ---- | ------------ | | |
| | stop | `() => void` | |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 229 out of 2444 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| **Package:** `@vueuse/math` | ||
| **Category:** '@Math' |
There was a problem hiding this comment.
The reference URL points to /core/logicOr/ but the package is @vueuse/math, so the link is very likely incorrect. Also, the Category value @Math is oddly quoted and inconsistent with other category labels. Please update the docs URL to the math section and normalize the category label.
|
|
||
| ## Reference | ||
|
|
||
| [VueUse Docs](https://vueuse.org/core/logicOr/) |
There was a problem hiding this comment.
The reference URL points to /core/logicOr/ but the package is @vueuse/math, so the link is very likely incorrect. Also, the Category value @Math is oddly quoted and inconsistent with other category labels. Please update the docs URL to the math section and normalize the category label.
| import { computedEager } from '@vueuse/core' | ||
|
|
||
| const todos = ref([]) | ||
| const hasOpenTodos = computedEager(() => !!todos.length) | ||
|
|
||
| console.log(hasOpenTodos.value) // false | ||
| toTodos.value.push({ title: 'Learn Vue' }) |
There was a problem hiding this comment.
The example code is internally inconsistent and will fail: todos is a ref so todos.length should be todos.value.length, and toTodos is referenced but never defined (likely meant todos). Please fix the example so it runs as-written.
| import { computedEager } from '@vueuse/core' | |
| const todos = ref([]) | |
| const hasOpenTodos = computedEager(() => !!todos.length) | |
| console.log(hasOpenTodos.value) // false | |
| toTodos.value.push({ title: 'Learn Vue' }) | |
| import { ref } from 'vue' | |
| import { computedEager } from '@vueuse/core' | |
| const todos = ref([]) | |
| const hasOpenTodos = computedEager(() => !!todos.value.length) | |
| console.log(hasOpenTodos.value) // false | |
| todos.value.push({ title: 'Learn Vue' }) |
Summary by CodeRabbit
New Features
Infrastructure
Documentation
✏️ Tip: You can customize this high-level summary in your review settings.