Skip to content

Conversation

@michaelchadwick
Copy link
Contributor

Fixes ilios/ilios#6629
Refs ilios/ilios#6608

This replaces the mixin with a class and removes several now-unused component styles that were only using the mixin.

@netlify
Copy link

netlify bot commented Nov 7, 2025

Deploy Preview for ilios-frontend ready!

Name Link
🔨 Latest commit 9bd0e0d
🔍 Latest deploy log https://app.netlify.com/projects/ilios-frontend/deploys/69122ec8d56d02000877b1a1
😎 Deploy Preview https://deploy-preview-8938--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.

@michaelchadwick michaelchadwick force-pushed the 6629-replace-data-visualization-sass-mixin branch from 68095f7 to f27d5e9 Compare November 7, 2025 17:39
@stopfstedt stopfstedt removed the request for review from jrjohnson November 10, 2025 16:22
@stopfstedt stopfstedt enabled auto-merge November 10, 2025 16:57
Copy link
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.

LGTM.

@stopfstedt stopfstedt assigned dartajax and unassigned dartajax Nov 10, 2025
@stopfstedt stopfstedt added the run ui tests Run the expensive UI tests label Nov 10, 2025
@michaelchadwick michaelchadwick force-pushed the 6629-replace-data-visualization-sass-mixin branch from f27d5e9 to b595613 Compare November 10, 2025 18:09
@dartajax dartajax self-assigned this Nov 10, 2025
Copy link
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.

click tested visualizations - looks good

@stopfstedt stopfstedt merged commit 3647b36 into ilios:master Nov 10, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run ui tests Run the expensive UI tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

replace data-visualization sass mixin with equivalent class

3 participants