DOC-13935 new page-capella-type for AI and app-services#196
DOC-13935 new page-capella-type for AI and app-services#196
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request adds support for two new Capella types ("AI" and "App Services") to the documentation theme's label system, allowing pages to be tagged with these categories alongside existing "columnar" and "operational" types.
Changes:
- Added conditional rendering for "AI" and "App Services" labels with corresponding icons
- Introduced new color variables for the green and purple brand colors used by the new labels
- Created CSS styles for the new label types matching the existing pattern
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/partials/labels.hbs | Added conditional blocks to render AI and App Services labels with sparkles and mobile-signal icons |
| src/css/vars.css | Defined new brand color variables (green and purple) for the new Capella types |
| src/css/labels.css | Added CSS styling for .ai and .app-services classes with appropriate colors and borders |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| } | ||
|
|
||
| /* Capella types: columnar vs operational */ | ||
| /* Capella types: columnar, operational, ai, app-services */ |
There was a problem hiding this comment.
Extra space between 'columnar,' and 'operational' - should be single space for consistency.
| /* Capella types: columnar, operational, ai, app-services */ | |
| /* Capella types: columnar, operational, ai, app-services */ |
Rebecca-Martinez007
left a comment
Rebecca-Martinez007
left a comment
There was a problem hiding this comment.
I feel like we should keep it consistent with naming elsewhere and have it be "Capella AI Services". Open to thoughts and objections.
| <li class="columnar"> | ||
| <i class="fas fa-chart-line"></i> | ||
| Capella Analytics</li> | ||
| {{else if (eq this "AI")}} |
There was a problem hiding this comment.
I just realized we need this to be "AI" to match with antora.yaml right?
How would we get the label to be "Capella AI Services"? In the local images shared, it says "Capella AI"
There was a problem hiding this comment.
You're commenting on the wrong line of code :)
There was a problem hiding this comment.
that is just the value of :page-capella-type: e.g. https://github.com/couchbaselabs/docs-ai/blob/main/antora.yml#L11
| <li class="columnar"> | ||
| <i class="fas fa-chart-line"></i> | ||
| Capella Analytics</li> | ||
| {{else if (eq this "AI")}} |
There was a problem hiding this comment.
I just realized we need this to be "AI" to match with antora.yaml right?
How would we get the label to be "Capella AI Services"? In the local images shared, it says "Capella AI"
| <li class="columnar"> | ||
| <i class="fas fa-chart-line"></i> | ||
| Capella Analytics</li> | ||
| {{else if (eq this "AI")}} |
There was a problem hiding this comment.
You're commenting on the wrong line of code :)
src/partials/labels.hbs
Outdated
| {{else if (eq this "AI")}} | ||
| <li class="ai"> | ||
| <i class="fas fa-sparkles"></i> | ||
| Capella AI</li> |
There was a problem hiding this comment.
This is the visible output, e.g. the label HTML.
So that's what you'd want to update @Rebecca-Martinez007
Co-authored-by: sarahlwelton <110928505+sarahlwelton@users.noreply.github.com>
|
Can't get Have replcaed with |
See https://preview.docs-test.couchbase.com/docs-capella-app-services-main/ai/build/integrate-agent-with-catalog.html
I've updated docs-ai and docs-capella-app-services with the new
:page-capella-type:NB: the icons aren't rendering, I'll look at that again...