apluslms / a-plus

A+ frontend portal - A+ LMS documentation:
https://apluslms.github.io/
Other
65 stars 72 forks source link

'Course has ended' sticky alert padding causes UI bug with exercise anchor links #1311

Closed ihalaij1 closed 7 months ago

ihalaij1 commented 8 months ago

Padding was added to the sticky alerts in PR https://github.com/apluslms/a-plus/pull/1219.

This is so that the alert does not hide the information that the anchor link points to. This should probably use margin-top instead of padding-top, since links to exercises now result in a weird exercise frame UI bug after scrolling up:

image

Example link: https://plus.cs.aalto.fi/y2/2022/materials_k04/k04_softwaretesting_testing/#chapter-exercise-2