Ux small changes - #12
Merged
Merged
Conversation
- Change operation_detail.html to use same format as try_it_out.html - Use <span class="param-name-wrapper"> container for parameter names - Move required asterisk outside <code> tag with spacing - Move colon outside the wrapper for consistent styling
Step 2 & 3 of incremental CSS modularization to manage growing stylesheet. Changes: - Created extract_detail_page_css.py to extract API detail page CSS - Created extract_skill_page_css.py to extract skill page CSS - Generated detail_page.css (2,014 lines) with 273 detail-page-specific rules - Generated skill_page.css (1,113 lines) with 143 skill-page-specific rules - Updated detail_page.html to include detail_page.css - Updated skill_page.html to include skill_page.css - Restored stats CSS to styles.css (was needed in detail pages) CSS Architecture (Complete): - styles.css: 7,548 lines (foundation, loaded on all pages) - homepage.css: 278 lines (39 rules, homepage only) - detail_page.css: 2,014 lines (273 rules, API detail pages only) - skill_page.css: 1,113 lines (143 rules, skill pages only) - Total: 10,953 lines, properly organized by page context Detail Page CSS includes: - Operation details, parameters, schemas, responses - Try It Out panel and authentication - Navigation sidebar and breadcrumbs - Stats grid, badges, method badges - Schema tables and examples Skill Page CSS includes: - Step-by-step workflow UI and terminal styling - Interactive playground and execution panels - Variables table and curl command blocks - Install modal and copy feedback - Evaluation inputs and results Benefits: - Reduced CSS payload per page (only load what's needed) - Better maintainability with isolated page-specific styles - Clear separation of concerns - Easier to identify and remove unused CSS in future
Reverted CSS extraction to page-specific files as it incorrectly identified shared components (badges, cards, buttons, catalog) as page-specific, causing broken layouts. The monolithic styles.css approach is correct for this codebase due to extensive component reuse across all pages. Updated schema_table.html to use param-name-wrapper format matching try_it_out.html for consistent parameter display across request and response schemas.
Ferticidio
approved these changes
Apr 17, 2026
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
2 participants
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.
No description provided.