Skip to content

Feat/talking about - #682

Merged
Luizgomess merged 3 commits into
masterfrom
feat/talking-about
Aug 7, 2026
Merged

Feat/talking about#682
Luizgomess merged 3 commits into
masterfrom
feat/talking-about

Conversation

@Luizgomess

@Luizgomess Luizgomess commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

What was done? 📝

adding all the articles I found on the internet about Zard

Screenshots or GIFs 📸

image

Type of change 🏗

  • New feature (non-breaking change that adds functionality)
  • Bug fix (non-breaking change that fixes an issue)
  • Refactor (non-breaking change that improves the code or technical debt)
  • Chore (none of the above, such as upgrading libraries)

Checklist 🧐

  • Tested on Chrome
  • Tested on Safari
  • Tested on Firefox
  • No errors in the console

Summary by CodeRabbit

  • New Features

    • Added a Featured Articles page with grouped article cards, language navigation, metadata, and contribution instructions.
    • Added article cover-image fallbacks for unavailable images.
    • Enabled access to Featured Articles through the sidebar and a dedicated route.
    • Added initial English featured articles with links to external posts.
  • Bug Fixes

    • Improved resilience when article cover images fail to load.

Adds /docs/featured/articles as the written-content counterpart to the
YouTube page: an aggregator of community articles about Zard UI, with the
original cover image of each post and a submission section pointing to the
same address used for video submissions.

Seeded with the two dev.to articles currently available. Adding a new one
is a single entry in featured-articles.ts.
Featured lists community content, not API reference, so sitting between
Get Started and Components broke the reading order of the sidebar. It now
comes last, after Contribute.

SIDEBAR_PATHS backs the sidebar, the mobile menu, the command palette and
the sitemap, so all four follow the new order. Prerendered routes are
unaffected.
@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Adds a Featured Articles page with article data, grouped rendering, scroll-spy navigation, SEO metadata, cover-image fallback handling, tests, lazy routing, sidebar integration, and prerendering.

Changes

Featured articles

Layer / File(s) Summary
Article data contract and content
apps/web/src/app/domain/pages/featured/data/featured-articles.ts
Defines featured article metadata and adds two English dev.to article records.
Article page rendering and behavior
apps/web/src/app/domain/pages/featured/articles/*
Adds grouped and sorted article cards, scroll-spy navigation, SEO metadata, contribution instructions, cover fallbacks, and tests.
Featured navigation and route integration
apps/web/src/app/domain/pages/featured/featured.routes.ts, apps/web/src/app/shared/constants/routes.constant.ts, apps/web/prerender-routes.txt
Adds the lazy route, enables and reorders the sidebar entry, and prerenders /docs/featured/articles.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Sidebar
  participant AngularRouter
  participant ArticlesPage
  participant ArticlesPageTemplate
  Sidebar->>AngularRouter: Navigate to /docs/featured/articles
  AngularRouter->>ArticlesPage: Lazy-load page component
  ArticlesPage->>ArticlesPageTemplate: Render grouped article data
  ArticlesPageTemplate->>ArticlesPage: Report failed cover image
  ArticlesPage->>ArticlesPageTemplate: Render placeholder cover
Loading

Possibly related PRs

  • zard-ui/zardui#673: Extends the same Featured section with another page and shared navigation and routing changes.
  • zard-ui/zardui#675: Modifies the scroll-spy directive used by the new Articles page.
  • zard-ui/zardui#677: Uses the same lazy-route, prerender, SEO, navigation, and testing patterns for another documentation page.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is vague and does not clearly identify the new Articles page or its featured community content. Rename the pull request to clearly describe the main change, such as "Add featured Articles page for Zard UI community posts."
✅ Passed checks (4 passed)
Check name Status Explanation
Description check ✅ Passed The description covers the feature, screenshot, change type, and most checklist items, but omits the issue link and breaking-change section.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/talking-about

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

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 current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@apps/web/src/app/domain/pages/featured/articles/articles.page.html`:
- Around line 40-46: Update the cover image in the article link to use an empty
alt value instead of including article.title, while leaving the adjacent h3
heading and other image attributes unchanged.

In `@apps/web/src/app/domain/pages/featured/articles/articles.page.ts`:
- Around line 35-51: Update the articles page component metadata by importing
ViewEncapsulation from `@angular/core` and setting encapsulation to
ViewEncapsulation.None in the `@Component` decorator.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 50ee69bb-c4b0-4672-828e-1c84a1bbcf08

📥 Commits

Reviewing files that changed from the base of the PR and between 04320cb and 2e5b528.

⛔ Files ignored due to path filters (2)
  • apps/web/public/docs/featured/articles.md is excluded by !apps/web/public/** and included by apps/**
  • carousel-upgrade.md is excluded by none and included by none
📒 Files selected for processing (7)
  • apps/web/prerender-routes.txt
  • apps/web/src/app/domain/pages/featured/articles/articles.page.html
  • apps/web/src/app/domain/pages/featured/articles/articles.page.spec.ts
  • apps/web/src/app/domain/pages/featured/articles/articles.page.ts
  • apps/web/src/app/domain/pages/featured/data/featured-articles.ts
  • apps/web/src/app/domain/pages/featured/featured.routes.ts
  • apps/web/src/app/shared/constants/routes.constant.ts

Comment thread apps/web/src/app/domain/pages/featured/articles/articles.page.html
Comment thread apps/web/src/app/domain/pages/featured/articles/articles.page.ts
@Luizgomess
Luizgomess merged commit 6d7c338 into master Aug 7, 2026
9 checks passed
@Luizgomess
Luizgomess deleted the feat/talking-about branch August 7, 2026 22:45
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.

1 participant