Skip to content
Snippets Groups Projects

Alert

Merged Ryan Dee requested to merge alert into develop

Update alert format and styles

Merge request reports

Approval is optional

Merged by avatar (Mar 13, 2025 8:35pm UTC)

Merge details

  • Changes merged into develop with ad0673b7.
  • Did not delete the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • This breaks the alert styles on unl today, the internal homepage. The alert HTML is being included in unltoday, so removing the inline styles and moving it to CSS will cause a loss of styling on unltoday.

    A few options:

    1. Move alert CSS to its own file (alert.CSS) and include that file in both places.
    2. Use inline styles.
360 360 background-size: 4.499em;
361 361 }
362 362 }
363 .wdn-video-band .mejs-overlay-button {
  • Ryan Dee
    Ryan Dee @rdee2 started a thread on the diff
  • 360 360 background-size: 4.499em;
    361 361 }
    362 362 }
    363 .wdn-video-band .mejs-overlay-button {
  • 360 360 background-size: 4.499em;
    361 361 }
    362 362 }
    363 .wdn-video-band .mejs-overlay-button {
  • Boy, I was asleep at the switch on this one - I think making the styles inline makes the most sense

  • Please register or sign in to reply
    Loading