-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Session Objectives Having Bulleted Or Numbered Lists Need Better Formatting - Courses and Sessions #3482
Comments
Team Discussion: We should use the expanded widget so users can get at the full contents as suggested by @stopfstedt here. |
@dartajax @stopfstedt Did this get fixed at some point? Just made a new course, session, and offering, and the event detail page looks fine https://demo.iliosproject.org/events/S0120240730O248226 |
@michaelchadwick - i think you're looking in the wrong place. the problem is on the course management side, not with learning events display. i bolted some dummy text onto that objective to force truncation. this should illustrate this a bit better: see https://demo.iliosproject.org/courses/2260/sessions/83494?sessionObjectiveDetails=true |
or perhaps you are looking in the right place, and i'm misreading the issue. @dartajax please clarify, thanks. |
@stopfstedt You added the thing I was missing: enough text to force the truncation on the Course side. Without it, it looks fine, and as long as there's an expander icon, the text can be seen unprocessed. I'm guessing we can't show the markup entered as-is since this is just text going into a |
I followed the link @stopfstedt posted and that seems like exactly what I was dealing with when this was created by me. Maybe the ticket is incorrectly named - certain in the courses and sessions area, this is the case. This issue does not appear to exist in "Event Detail" - I published and tested this. I will update the title of this ticket. |
This applies to course objectives as well - program years - maybe too - that would be a new ticket though check the course objective I added using the same link @stopfstedt put in above |
@dartajax I see the issue both on Course Objectives and Session Objectives. Also, I just realized that even when the text is not truncated, it's still being put inside a @stopfstedt I'm assuming that this is the truncation algorithm, and augmenting it to handle HTML would be non-trivial. Is it worth pursuing at this point? |
@michaelchadwick that is the problem indeed. to state the obvious - arbitrary markup truncation is bad. perhaps a different approach is in order. instead of stripping markup/truncating text, overlong markup could be hidden - add a gradient fade-out and a "shore more" button or the likes. might look weird in the objectives grid view if the next cell over (parent objective or competency) is long, so this "truncation" might need to be added there too. i'm thinking along the lines of https://css-tricks.com/text-fade-read-more/ haven't thought this all the way through, but may be worth a try. |
Going to try to create a new TruncationFade component and see how it looks visually. |
@saschaben @stopfstedt @jrjohnson if you wanna check it out, I put up a PR for using the fade technique: ilios/frontend#8034 |
There will be more of these. If a bulleted list (or a numbered list) is entered as objectives(s) for a session, the advanced formatting causes the resulting output on the Event Detail screen to be difficult to read and understand.
Here is the session in Courses and Sessions.
Here it is in Event Detail. All of the items
are concatenated directly together without even a space being inserted between the values. See here ...
The text was updated successfully, but these errors were encountered: