Website: update features comparison table on homepage#46293
Conversation
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughThis PR simplifies the homepage comparison feature by consolidating from dual IT and Security comparison modes to IT only. The JavaScript data state now uses Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
Updates the Fleet website homepage to simplify the “Moving to Fleet?” feature comparison section by removing the audience switch / security-focused table and trimming the remaining comparison UI + script accordingly.
Changes:
- Removed the comparison-table mode switch and deleted the “For CISOs” (security) comparison tables from the homepage template.
- Reduced the remaining “IT” comparison table content (rows/features) and removed some previously listed competitor options from selectors.
- Simplified the homepage page script by removing now-unused state/methods related to the deleted table/mode.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| website/views/pages/homepage.ejs | Removes the switch + security comparison tables and updates the remaining IT comparison table rows/selectors. |
| website/assets/js/pages/homepage.page.js | Removes security comparison state/methods and updates mounted behavior accordingly. |
Comments suppressed due to low confidence (1)
website/views/pages/homepage.ejs:359
- The comparison table dropdown no longer offers ansible/puppet/chef (and also no longer offers tanium), but this row still contains vendor-specific branches for those values. With the current selector options, the
['puppet', 'chef', 'ansible']branch is unreachable, which makes the template harder to maintain and can hide mistakes if the vendor list changes again.
Consider either (a) removing these vendor checks across the table now that those vendors aren’t selectable, or (b) re-adding those vendors to the selector so the conditions remain meaningful.
<div purpose="feature-name"><p>Open source</p></div>
<div purpose="fleet-column">
<img class="mx-auto" alt="checkmark" purpose="checkmark" src="/images/icon-checkmark-green-16x16@2x.png">
</div>
<div purpose="comparison-column" v-if="['puppet', 'chef', 'ansible'].includes(comparisonModeForIt)">
<p>Only free version</p>
</div>
<div purpose="comparison-column" v-else>
<img class="mx-auto" alt="❌" purpose="red-x" src="/images/icon-emoji-x-12x12@2x.png">
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Changes:
Summary by CodeRabbit