Skip to content

Conversation

@michaelchadwick
Copy link
Contributor

@michaelchadwick michaelchadwick commented Jan 26, 2026

Fixes ilios/ilios#6721
Refs ilios/ilios#6574
Refs ilios/ilios#6671

Issues fixed:

  • Event tooltip now has instructors
  • Event tooltip now has session type
  • Event and tooltip now has room/location
  • Event now has recently updated icon if valid
  • Template class and style attributes are more compact and spacing is improved

Issues not fixed (future PR)

  • Event color

@netlify
Copy link

netlify bot commented Jan 26, 2026

Deploy Preview for ilios-frontend ready!

Name Link
🔨 Latest commit 96b4d21
🔍 Latest deploy log https://app.netlify.com/projects/ilios-frontend/deploys/6977bc030add8b000860b9d9
😎 Deploy Preview https://deploy-preview-9086--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.

@netlify
Copy link

netlify bot commented Jan 26, 2026

Deploy Preview for ilios-frontend ready!

Name Link
🔨 Latest commit e1a23f6
🔍 Latest deploy log https://app.netlify.com/projects/ilios-frontend/deploys/697a9bdd0c35df0008ea09de
😎 Deploy Preview https://deploy-preview-9086--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 marked this pull request as ready for review January 26, 2026 21:48
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

@dartajax dartajax added run ui tests Run the expensive UI tests and removed run ui tests Run the expensive UI tests labels Jan 27, 2026
@michaelchadwick michaelchadwick force-pushed the frontend-6574-lg-calendar-improvements branch from 6e07fad to 92249c9 Compare January 28, 2026 18:09
@jrjohnson jrjohnson removed the run ui tests Run the expensive UI tests label Jan 28, 2026
@jrjohnson
Copy link
Member

Some legit test failures here, pulled off the UI label as those won't ever pass if the CI ones are failing.

@michaelchadwick
Copy link
Contributor Author

Checking the tests now...

@michaelchadwick michaelchadwick force-pushed the frontend-6574-lg-calendar-improvements branch from 92249c9 to e1a23f6 Compare January 28, 2026 23:29
@github-actions
Copy link
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-9086

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

❌ Visual differences detected.

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

Download the results.

@jrjohnson jrjohnson added the run ui tests Run the expensive UI tests label Jan 29, 2026
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.

I like this.

@dartajax dartajax merged commit 4e7016a into ilios:master Jan 29, 2026
44 of 45 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.

Recently Updated Icon - Missing From Learner Group Calendar

4 participants