Skip to content

Adjust User Guide Link#9100

Merged
jrjohnson merged 1 commit intoilios:masterfrom
jrjohnson:user-guide-visual
Feb 5, 2026
Merged

Adjust User Guide Link#9100
jrjohnson merged 1 commit intoilios:masterfrom
jrjohnson:user-guide-visual

Conversation

@jrjohnson
Copy link
Copy Markdown
Member

@jrjohnson jrjohnson commented Jan 28, 2026

Using tooling to grow the icon in the space instead of font-size and adding some padding to make it a nicer square.

Refs #9084

@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 28, 2026

Deploy Preview for ilios-frontend ready!

Name Link
🔨 Latest commit e8b452a
🔍 Latest deploy log https://app.netlify.com/projects/ilios-frontend/deploys/697d41b5974a410008c2aa92
😎 Deploy Preview https://deploy-preview-9100--ilios-frontend.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@github-actions
Copy link
Copy Markdown
Contributor

🔍 Visual Diff Results

🔍 Comparing screenshots...

Baseline: /home/runner/work/frontend/frontend/baseline-frontend
Candidate: /home/runner/work/frontend/frontend/candidate-frontend
Output: /home/runner/work/frontend/frontend/visual-diff-9100

📊 Results:
Total images: 752
With differences: 0
Identical: 752
Removed: 0
Added: 0

✅ All checks passed!

📄 Report generated: /home/runner/work/frontend/frontend/visual-diff-9100/index.html

Download the results.

@jrjohnson jrjohnson marked this pull request as ready for review January 28, 2026 20:50
Copy link
Copy Markdown
Member

@stopfstedt stopfstedt left a comment

Choose a reason for hiding this comment

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

looks slightly but still noticably too large in latest FF (147.0.1) for me now.

Image

in latest Chromium, the help button looks just fine.

Image

Using tooling to grow the icon in the space instead of font-size and
adding some padding to make it a nicer square.
@jrjohnson
Copy link
Copy Markdown
Member Author

🦅 👀 !! Pulled everything down one size and re-pushed.

@github-actions
Copy link
Copy Markdown
Contributor

🔍 Visual Diff Results

🔍 Comparing screenshots...

Baseline: /home/runner/work/frontend/frontend/baseline-frontend
Candidate: /home/runner/work/frontend/frontend/candidate-frontend
Output: /home/runner/work/frontend/frontend/visual-diff-9100

📊 Results:
Total images: 752
With differences: 2
Identical: 750
Removed: 0
Added: 0

❌ Visual differences detected.

📄 Report generated: /home/runner/work/frontend/frontend/visual-diff-9100/index.html

Download the results.

@jrjohnson jrjohnson requested a review from stopfstedt February 3, 2026 17:29
Copy link
Copy Markdown
Member

@dartajax dartajax left a comment

Choose a reason for hiding this comment

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

looks great - I approve

@dartajax dartajax assigned jrjohnson and unassigned dartajax Feb 5, 2026
@jrjohnson jrjohnson merged commit f937ccf into ilios:master Feb 5, 2026
46 of 48 checks passed
@jrjohnson jrjohnson deleted the user-guide-visual branch February 5, 2026 20:14
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.

3 participants