Skip to content

Conversation

@ylimezhang
Copy link
Collaborator

No description provided.

@ylimezhang ylimezhang requested a review from Copilot June 4, 2025 01:01
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR incorporates usability feedback by standardizing spelling, refining tutorial headings, and adjusting UI text and behaviors for improved consistency and clarity.

  • Standardize spelling of “visualizing” to British “visualising” across docs and tests
  • Update tutorial titles and content data to reference “IRV RLAs”
  • Tweak UI text formatting (dashes, paragraphs) and tooltip interactions

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
test/prettyprint_assertions_and_pictures.test.ts Corrects British spelling of “visualising” in comments
app/tutorial/outcomes/page.mdx Updates heading to British “Visualising Outcomes”
app/tutorial/introduction/page.md Changes title to “IRV RLAs” and British spelling in list item
app/tutorial/components/data-content.ts Synchronizes navigation titles for introduction and outcomes
app/tutorial/assertion/page.mdx Updates subheading to British “Visualising assertions”
app/steps.tsx Adjusts dash style in instructional text
app/dashboard/components/step-by-step-view/index.tsx Rewords assertion tree message for clarity
app/dashboard/components/lazyload-view/index.tsx Refactors winner-tree description into clearer paragraphs
app/dashboard/components/Information-icon-text.tsx Alters tooltip trigger logic and removes mouse-leave handler
Comments suppressed due to low confidence (3)

app/dashboard/components/Information-icon-text.tsx:45

  • Removing the onMouseLeave handler means the tooltip won’t close when the cursor leaves, preventing users from dismissing it. Consider re-adding the handler or using Tooltip props to ensure proper close behavior.
onMouseLeave={handleMouseLeave}

app/steps.tsx:152

  • [nitpick] Replacing the em dash (—) with a hyphen (-) may conflict with the project’s typography guidelines; consider using an em dash for proper punctuation consistency.
<Brown> reaching the top - no cuts, no doubts!</Brown>

app/dashboard/components/lazyload-view/index.tsx:1022

  • Using
    tags for line breaks can hinder accessibility and screen-reader flow. Consider splitting this content into separate

    elements or using CSS for spacing.

<br />

@ylimezhang ylimezhang requested review from Mirrorgo and lvyl9909 June 4, 2025 01:03
@Mirrorgo Mirrorgo merged commit 3628bb5 into release Jun 4, 2025
1 of 3 checks passed
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.

4 participants