Skip to content

docs(nuxt-feather-icons): Update navigation icons in usage documentation#74

Merged
4sllan merged 1 commit into
mainfrom
feat/docs
Mar 16, 2026
Merged

docs(nuxt-feather-icons): Update navigation icons in usage documentation#74
4sllan merged 1 commit into
mainfrom
feat/docs

Conversation

@4sllan

@4sllan 4sllan commented Mar 16, 2026

Copy link
Copy Markdown
Owner
  • Replace icon reference in basic usage example
  • Replace icon reference in dynamic icons example

Summary by CodeRabbit

  • Documentation
    • Updated navigation icons in the Basic Usage and Dynamic Icons documentation pages.

- Replace icon reference in basic usage example
- Replace icon reference in dynamic icons example
@coderabbitai

coderabbitai Bot commented Mar 16, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

Documentation navigation icons updated across two usage guide files. ZapIcon replaced with Settings2Icon in basic usage documentation, and the dynamic icons guide icon reference changed from i-ph-arrows-left-right to ArrowRightLeftIcon. No functional code changes.

Changes

Cohort / File(s) Summary
Documentation Navigation Icons
docs/content/2.usage/1.basic-usage.md, docs/content/2.usage/3.dynamic-icons.md
Updated icon references in YAML front matter metadata. Basic usage doc icon changed from ZapIcon to Settings2Icon; dynamic icons doc icon changed from i-ph-arrows-left-right to ArrowRightLeftIcon.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 ✨ Icons dance left and right,
From zappers to gears, a metadata flight,
Two docs refreshed with symbols new,
Simple swaps make the guides shine through! ⚙️

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: updating navigation icons in usage documentation across two files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/docs
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

You can disable the changed files summary in the walkthrough.

Disable the reviews.changed_files_summary setting to disable the changed files summary in the walkthrough.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/content/2.usage/1.basic-usage.md`:
- Line 5: Replace the incorrect Lucide-only icon name by changing the
frontmatter/icon reference from Settings2Icon to SettingsIcon so the document
uses the Feather icon set consistently; locate the icon key (icon:
Settings2Icon) and update it to icon: SettingsIcon to match the other examples
(e.g., HomeIcon, UsersIcon) and the navigation example that already uses
SettingsIcon.

In `@docs/content/2.usage/3.dynamic-icons.md`:
- Line 5: Replace the invalid Feather icon reference `ArrowRightLeftIcon` with a
valid Feather icon name (for example `ArrowRightIcon`, `ArrowDownIcon`, or
`ShuffleIcon`) wherever `icon: ArrowRightLeftIcon` appears; ensure the value
passed to useFeatherIcon() matches a real Feather icon identifier so the dynamic
icon lookup succeeds.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6b56247c-f2b3-46e0-9ee2-b3b408cc4144

📥 Commits

Reviewing files that changed from the base of the PR and between ddc3851 and c4de8ed.

📒 Files selected for processing (2)
  • docs/content/2.usage/1.basic-usage.md
  • docs/content/2.usage/3.dynamic-icons.md

Comment thread docs/content/2.usage/1.basic-usage.md
Comment thread docs/content/2.usage/3.dynamic-icons.md
@4sllan
4sllan merged commit b78aed1 into main Mar 16, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in nuxt-feather-icons Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant