fix: add home section #769
Conversation
github-actions
bot
added
the
enhancement
…hinx-theme into feat/home-section
src/ansys_sphinx_theme/__init__.py
Outdated
| return | ||
|
|
||
| home_entry = ( | ||
| nodes.Text("Package Home"), |
There was a problem hiding this comment.
Are we imposing this to always be Package Home? I thought we agreed on just writing Home here.
There was a problem hiding this comment.
Mia suggested having the package at home in this comment last week, but we can discuss this further.
There was a problem hiding this comment.
I agree.. "Package Home" looks too verbose and weird .. I'd sugggest moving to "Home"
There was a problem hiding this comment.
I feel like Package Home is too long.. Home is consistent with the logo 🏠 we see in the breadcrumps. To me @mia-guo-ux didn't mean Package in the label.
There was a problem hiding this comment.
pinging @mia-guo-ux here , for insights
There was a problem hiding this comment.
To avoid confusion, we initially adopted the term Package Home, since Home alone could be misinterpreted as referring to the broader PyAnsys home.
One potential improvement would be to simplify the label to just Home, while providing additional context through a tooltip. For example, hovering over "Home" could display a tooltip like PyAEDT Home, clearly indicating the specific package being referenced.
There was a problem hiding this comment.
Added the tool tip, and renamed to home
Fix #702, fix #498, fix #257