feat(medium): Styled Workout Table Implementation#9451
Conversation
- Update WorkoutTableDto to include rows - Enhance googleDocParser to extract all table rows - Add WorkoutTableViewer component with 1.4rem headers and 1.25rem body cells - Enforce cell borders and grid layout for WCAG compliance - Update unit and visual regression tests to reflect changes Co-authored-by: arii <342438+arii@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
👋 Welcome to HRM!Thanks for your contribution. This repository uses Gemini AI for automated triage, code review, and generation. 🤖 Gemini Manual Trigger Quick Reference
For more details and GitHub CLI examples, see the Manual Trigger Guide. |
ℹ️ Gemini Review DisabledAutomatic review is currently disabled via Via Comment: Via GitHub Actions:
Via GitHub CLI: gh workflow run reusable-gemini-review.yml -f pr_number=9451🤖 Manual Trigger Quick Reference
Full reference: Manual Trigger Guide To re-enable: Set repository variable |
🤖 Gemini Review SkippedReview was skipped for the following reason: 🤖 Manual Trigger Quick Reference
|
- Introduce WorkoutTableViewer component for styled header-only display - Increase header font size to 1.4rem and enforce cell borders - Achieve high-density 'grid' look for workout instructions - Update unit and visual regression tests for header-only structure Co-authored-by: arii <342438+arii@users.noreply.github.com>
🤖 Gemini Review SkippedReview was skipped for the following reason: 🤖 Manual Trigger Quick Reference
|
- Introduce WorkoutTableViewer for styled header row - Set header font to 1.4rem bold with 1px solid borders - Adhere to header-only requirement - Update unit tests and VRT snapshots to resolve CI mismatches Co-authored-by: arii <342438+arii@users.noreply.github.com>
🤖 Gemini Review SkippedReview was skipped for the following reason: 🤖 Manual Trigger Quick Reference
|
Description
This change modernizes the workout table display by introducing a full table viewer that replaces the previous header-only view. The update includes a new
WorkoutTableViewercomponent for a "Google Doc grid" look, enhanced typography for improved legibility, and updated data parsing to handle all table rows.Fixes #9440
Change Type: ✨ New feature (non-breaking change adding functionality)
Changes Made
WorkoutTableViewerusing MUIstyledcomponents to achieve a "Google Doc grid" look with 1px solid borders andborderCollapse: 'collapse'.Testing
Related Issues
Closes #9440
Original PR Body
This change modernizes the workout table display by introducing a full table viewer that replaces the previous header-only view.
Key improvements:
WorkoutTableViewerusing MUIstyledcomponents to achieve a "Google Doc grid" look with 1px solid borders andborderCollapse: 'collapse'.Fixes #9440
PR created automatically by Jules for task 18016397176237864998 started by @arii