feat: Add Products dropdown and Solutions dropdown to navbar#1703
Closed
feat: Add Products dropdown and Solutions dropdown to navbar#1703
Conversation
Member
Alek99
commented
Dec 5, 2025
- Created Products dropdown containing AI Builder, Open Source, and Cloud
- Created Solutions dropdown with App Types (Internal Tools, Data & AI Apps, Customer Facing Apps) and Industries (Finance, Healthcare, Energy, Enterprise)
- Updated navbar menu trigger to support active states for dropdowns
- Reorganized navbar items: Products, Docs, Resources, Solutions, Pricing
- Fixed icon names to use valid icons from the icon library
- Created Products dropdown containing AI Builder, Open Source, and Cloud - Created Solutions dropdown with App Types (Internal Tools, Data & AI Apps, Customer Facing Apps) and Industries (Finance, Healthcare, Energy, Enterprise) - Updated navbar menu trigger to support active states for dropdowns - Reorganized navbar items: Products, Docs, Resources, Solutions, Pricing - Fixed icon names to use valid icons from the icon library
Contributor
Greptile OverviewGreptile SummaryThis PR restructures the navbar to add two new dropdown menus - Products and Solutions. The Products dropdown consolidates AI Builder, Open Source, and Cloud offerings with conditional rendering based on the current route. The Solutions dropdown introduces organized sections for App Types (Internal Tools, Data & AI Apps, Customer Facing Apps) and Industries (Finance, Healthcare, Energy, Enterprise). Key changes:
Minor issue found:
Confidence Score: 4/5
Important Files ChangedFile Analysis
Sequence DiagramsequenceDiagram
participant User
participant Navbar
participant Router
participant ProductsDropdown
participant SolutionsDropdown
User->>Navbar: Hover over "Product" trigger
Navbar->>Router: Get current route path
Router-->>Navbar: Return path (e.g., "/docs/...")
Navbar->>Navbar: Calculate active state for Products
Note over Navbar: Checks if path contains:<br/>"ai-builder", "cloud", "hosting",<br/>or framework docs
Navbar->>ProductsDropdown: Display dropdown menu
ProductsDropdown->>Router: Check if on docs/ai-builder/cloud
Router-->>ProductsDropdown: Return path condition
alt On docs/ai-builder/cloud pages
ProductsDropdown->>User: Show internal doc links
else On other pages
ProductsDropdown->>User: Show external AI Builder + internal links
end
User->>Navbar: Hover over "Solutions" trigger
Navbar->>Router: Get current route path
Router-->>Navbar: Return path
Navbar->>Navbar: Calculate active state for Solutions
Note over Navbar: Checks if path contains "use-cases"
Navbar->>SolutionsDropdown: Display dropdown menu
SolutionsDropdown->>User: Show App Types and Industries columns
User->>SolutionsDropdown: Click "Finance" industry
SolutionsDropdown->>Router: Navigate to finance_use_case_page.path
Router->>User: Display Finance use case page
|
Comment on lines
+450
to
+456
| if active_str == "products": | ||
| is_docs = router_path.contains("docs") | ||
| is_open_source_page = router_path.contains("open-source") | ||
| not_cloud = ~(router_path.contains("cloud") | router_path.contains("hosting")) | ||
| not_ai_builder = ~router_path.contains("ai-builder") | ||
| is_framework = (is_docs & not_cloud & not_ai_builder) | is_open_source_page | ||
| active = router_path.contains("ai-builder") | router_path.contains("cloud") | router_path.contains("hosting") | is_framework |
Contributor
There was a problem hiding this comment.
style: duplicated logic from link_item function lines 69-75 - extract this "products" active state calculation into a shared helper function
Prompt To Fix With AI
This is a comment left during a code review.
Path: pcweb/components/docpage/navbar/navbar.py
Line: 450:456
Comment:
**style:** duplicated logic from `link_item` function lines 69-75 - extract this "products" active state calculation into a shared helper function
How can I resolve this? If you propose a fix, please make it concise.409bacc to
9f18d34
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.