Skip to content

Conversation

michaelchadwick
Copy link
Contributor

@michaelchadwick michaelchadwick commented Sep 4, 2025

Refs ilios/ilios#4176

Main differences:

  • Removed extra left margin so it sits flush with rest of page
  • Removed space around MeSH Manager and Timed Release Schedule
  • Don't show nothing for Timed Release Schedule when there's a start date AND it's in the past

Copy link

netlify bot commented Sep 4, 2025

Deploy Preview for ilios-frontend ready!

Name Link
🔨 Latest commit 86c9c56
🔍 Latest deploy log https://app.netlify.com/projects/ilios-frontend/deploys/68e3dd6291bb0f00081d3547
😎 Deploy Preview https://deploy-preview-8789--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.

@dartajax
Copy link
Member

dartajax commented Sep 4, 2025

I made a cpl of quick space saving and formatting suggestions - no major reformatting ... yet.

Upper part of screen ...

Currently ...
image

NOTE: slight space saving and no major re-formatting - I thought the "Show Notes to Students" should be on the same horiz area as the Instructional Notes - same with "Required" same line as Display info.

Updated ...
image

Lower part of screen ...

Currently ...
image

Updated ...
image

Blue rectangles show saved space and keep in mind the Start Date and End Date stuff gets stacked up currently making the form even longer and making it farther still be able to save or undo your work.

NOTES:
what did I do? ...

  1. Aligned "Available immediately when published" with the "Timed Release" caption - they were not aligned before.
  2. Start Date and End Date buttons are not the same size and would look better spanning the left and right sides of the screens also saving space especially when those dates are entered - currently they stack all the way down.
  3. I took one line of the search for MeSH captioning out - we'd have to ask if that's acceptable - I just found it redundant.
  4. Removed vertical space wherever possible.

Still no major overhaul. That may still be coming.

@michaelchadwick michaelchadwick force-pushed the frontend-4176-lm-form-revamp branch 3 times, most recently from 015149e to cef2c99 Compare September 11, 2025 15:31
@michaelchadwick
Copy link
Contributor Author

In order to know how best to go forward, this needs team discussion for further improvements.

@michaelchadwick
Copy link
Contributor Author

Team discussion: be free with potential revamp. Try using the Session->Offering framework with a <fieldset><legend> direction. Make related fields closer to each other, as well.

@michaelchadwick michaelchadwick force-pushed the frontend-4176-lm-form-revamp branch from cef2c99 to 86c9c56 Compare October 6, 2025 15:16
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.

2 participants