feat(pageHeader): add tabs slot as alternative to icon + title

[PierreBrisorgueil]

Summary

• Add tabs slot to CorePageHeader that replaces icon + title area with navigation tabs
• Tabs mode detected via $slots.tabs — backward compatible, no script changes
• min-height: 56px in tabs mode for consistent header height

Test plan

• Existing pages without #tabs slot render unchanged
• Lint + unit tests pass
• Build succeeds

Closes #3780

Summary by CodeRabbit

• New Features
  • Page header now automatically switches between a tabs-focused layout and the traditional avatar + title/subtitle layout.
  • Tabs layout enforces consistent vertical spacing for a cleaner appearance.
  • Existing action area and previous header behavior remain supported for backward compatibility.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.22%. Comparing base (ff65d6f) to head (f64b5bc).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3781   +/-   ##
=======================================
  Coverage   99.22%   99.22%           
=======================================
  Files          27       27           
  Lines         898      898           
  Branches      235      235           
=======================================
  Hits          891      891           
  Misses          7        7           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Copilot]

Pull request overview

Adds an optional tabs slot to CorePageHeader so pages can render navigation tabs in the header’s left area instead of the standard icon + title layout, while keeping the actions area on the right.

Changes:

• Add conditional “tabs mode” rendering when tabs slot is present, otherwise render existing avatar/title/subtitle layout.
• Apply a min-height: 56px style when in tabs mode to keep header height consistent.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 88f176f5-1970-45b5-ba52-9c0e0730c271

📥 Commits

Reviewing files that changed from the base of the PR and between 12b07f9 and f64b5bc.

📒 Files selected for processing (1)

• src/modules/core/components/core.pageHeader.component.vue

---

Walkthrough

The PageHeader component now conditionally renders a tabs slot: if provided, the header shows only the tabs slot with min-height: 56px; otherwise it preserves the previous avatar + title + optional subtitle layout. The actions slot and component props are unchanged.

Changes

Cohort / File(s)	Summary
PageHeader tabs mode src/modules/core/components/core.pageHeader.component.vue	Template updated to branch on \$slots.tabs: when present render only <slot name="tabs"> and apply min-height: 56px; when absent render original avatar (icon fallback), title, and optional subtitle. actions slot and script/props remain unchanged.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main feature: adding a tabs slot as an alternative to the icon + title layout in the PageHeader component.
Description check	✅ Passed	The PR description covers the main changes, test plan, and links the related issue, though some template sections like validation checklist, guardrails, and detailed risk assessment are missing.
Linked Issues check	✅ Passed	The code changes fully satisfy issue #3780 requirements: tabs slot implemented via $slots.tabs detection, min-height: 56px applied in tabs mode, backward compatibility maintained, and no script changes made.
Out of Scope Changes check	✅ Passed	All changes are scoped to the CorePageHeader component's template to implement the tabs slot feature as specified in issue #3780; no unrelated modifications detected.
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
• Commit unit tests in branch feat/pageheader-tabs-slot

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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