Skip to content

Use focus Modifier on Login Form - #9484

Open
jrjohnson wants to merge 2 commits into
ilios:masterfrom
jrjohnson:focus-screenshot-race
Open

Use focus Modifier on Login Form#9484
jrjohnson wants to merge 2 commits into
ilios:masterfrom
jrjohnson:focus-screenshot-race

Conversation

@jrjohnson

Copy link
Copy Markdown
Member

Might be more accessible, but I'm primarily trying to resolve a race condition in our screenshots.

@netlify

netlify Bot commented Aug 25, 2026

Copy link
Copy Markdown

Deploy Preview for ilios-frontend ready!

Name Link
🔨 Latest commit aa4a38e
🔍 Latest deploy log https://app.netlify.com/projects/ilios-frontend/deploys/6a90cb793af62000087ce95b
😎 Deploy Preview https://deploy-preview-9484--ilios-frontend.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

✅ Visual Diff Approved

❌ Visual Diff Report — FAILED

2024 images compared: 3 different · 2021 identical

Details

Differences (3)

File Diff % Notes
desktop-Acceptance _ Auth Errors_request for preferences unauthorized__dark.png 0.32%
laptop-Acceptance _ Admin_can search for users_search results dropdown_dark.png 0.04%
laptop-Acceptance _ Auth Errors_request for preferences unauthorized__dark.png 0.27%

Download the results.

@jrjohnson
jrjohnson force-pushed the focus-screenshot-race branch 3 times, most recently from b229e05 to a97b968 Compare August 26, 2026 17:17
Might be more accessible, but I'm primarily trying to resolve a race
condition in our screenshots.
We get race conditions on the input focus in screenshots in this test,
need to explicitly wait for the focus so we take a useful picture.
@jrjohnson
jrjohnson force-pushed the focus-screenshot-race branch from a97b968 to aa4a38e Compare August 27, 2026 23:42
@jrjohnson jrjohnson added the approve visual diff Manually Approve the Results of a Visual Diff Run label Aug 28, 2026
@jrjohnson
jrjohnson marked this pull request as ready for review August 28, 2026 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approve visual diff Manually Approve the Results of a Visual Diff Run

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant