Skip to content

Conversation

@rohitpaulk
Copy link
Member

@rohitpaulk rohitpaulk commented Dec 3, 2025

Update start-workflow-tutorial-screen to use an animated checkmark
via Rive instead of the static SVG icon. This improves visual feedback
and enhances user experience on the workflow completion modal.


Note

Swap static SVG status icons for Rive animations in two course-page modals.

  • UI:
    • Replace SVG icons with Rive animations in course modals:
      • app/components/course-page/course-stage-step/tests-passed-modal/choose-action-step.hbs: use RiveAnimation with checkmark_icon.riv instead of svg-jar "check-circle".
      • app/components/course-page/previous-steps-incomplete-modal.hbs: use RiveAnimation with warning_icon.riv instead of svg-jar "exclamation".

Written by Cursor Bugbot for commit 5129794. This will update automatically on new commits. Configure here.

@github-actions
Copy link

github-actions bot commented Dec 3, 2025

Test Results

  1 files  ±0    1 suites  ±0   9m 49s ⏱️ +21s
677 tests ±0  625 ✅  - 1  51 💤 ±0  0 ❌ ±0  1 🔥 +1 
677 runs  ±0  624 ✅  - 2  51 💤 ±0  1 ❌ +1  1 🔥 +1 

For more details on these errors, see this check.

Results for commit 5129794. ± Comparison against base commit 1eca78e.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Dec 3, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
626 1 625 51
View the full list of 1 ❄️ flaky test(s)
Chrome 142.0::Acceptance | course-page | test-results-bar | resize: can resize test results bar using touch

Flake rate in main: 35.96% (Passed 57 times, Failed 32 times)

Stack Traces | 2.7s run time
Test results bar should be resized using touch

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@codecov
Copy link

codecov bot commented Dec 3, 2025

Bundle Report

Changes will decrease total bundle size by 1.5kB (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
client-array-push 38.81MB -1.5kB (-0.0%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: client-array-push

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/chunk.*.js -55 bytes 40.85kB -0.13%
assets/chunk.*.js -1.57kB 3.33MB -0.05%
assets/chunk.*.js -27 bytes 40.9kB -0.07%
assets/chunk.*.js 155 bytes 402.32kB 0.04%

@rohitpaulk rohitpaulk force-pushed the pk-branch-11 branch 3 times, most recently from 85bdb47 to bf66b08 Compare December 3, 2025 16:29
Update start-workflow-tutorial-screen to use an animated checkmark  
via Rive instead of the static SVG icon. This improves visual feedback  
and enhances user experience on the workflow completion modal.
feat(ui): replace exclamation SVG with Rive warning animation

Update the previous steps incomplete modal to use a Rive animation
instead of a static SVG icon for the warning indicator. This change
enhances the visual feedback by providing a more engaging and dynamic
animation, improving user experience and drawing attention more
effectively to incomplete course steps.
fix(ui): increase checkmark icon size in tutorial screen

Increase the checkmark icon size from 48x48px to 64x64px in the
StartWorkflowTutorialScreen to improve visibility and user feedback
during the course setup process.
Update the choose-action-step component to use a Rive animation for
the checkmark icon instead of a static SVG. This enhances visual
feedback by adding smooth, scalable animations, improving user
experience in the tests passed modal.
@rohitpaulk rohitpaulk merged commit 5d7813c into main Dec 3, 2025
4 of 6 checks passed
@rohitpaulk rohitpaulk deleted the pk-branch-11 branch December 3, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants