feat(e2e): add Percy visual regression tests for priority component areas#2
Open
devin-ai-integration[bot] wants to merge 1 commit intomainfrom
Open
Conversation
…reas - Add enhanced tests for Form Components (input-box, checkbox, radio-button) - Disabled states, hint text, placeholder, error states - Hover, focus, and validation error states - Add enhanced tests for Layout & Responsive Components (fluid-grid, box, card) - Mobile width breakpoint testing - Context menu open states - Hover states - Add enhanced tests for Interactive Overlay Components (dropdown, autocomplete, colorpicker) - Hover and focus states for buttons - Menu item hover states - Highlighted result states - Preset color selection - Add enhanced tests for Critical User Flows (error, tabs, paging) - Mobile width responsive testing - Action button hover and focus states - Tab navigation states - Paging button states and navigation Co-Authored-By: benc@cognition.ai <Benc@windsurf.com>
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
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.
feat(e2e): add Percy visual regression tests for priority component areas
Summary
This PR expands Percy visual regression test coverage across priority component areas in the SKY UX component library. The new tests capture interactive states (hover, focus), validation states, and responsive behavior at mobile breakpoints.
Changes by area:
Form Components - Added tests for checkbox (validation error, hover, focus), input-box (disabled, hint text, placeholder, error+button states), and radio-button (focus, hover, validation error)
Layout & Responsive - Added mobile width breakpoint tests for fluid-grid, box, and card components, plus context menu open state for box and hover state for card
Interactive Overlays - Added hover/focus states for dropdown buttons and menu items, autocomplete focus and result hover states, colorpicker hover/focus and preset selection
Critical User Flows - Added mobile responsive tests and action button states for error pages, tab navigation states (hover, focus, navigated), and paging button states with navigation
Review & Testing Checklist for Human
.sky-paging-btn,.sky-error-action button,.sky-btn-tabwere based on existing patterns but not verified against live componentsnx e2e <project>-e2efor affected projects to verify tests workMOBILE_WIDTHSinside singleit()blocks; verify this is acceptable for the test frameworkRecommended test plan:
npm run lint:affectedto verify lint passes (already done)Notes
E2eVariations.forEachTheme(),DISPLAY_WIDTHS, andMOBILE_WIDTHSutilities consistentlyLink to Devin run: https://app.devin.ai/sessions/c19c2f1df206434b9433e55d9aa21547
Requested by: @bcmake