Skip to content

103 flash message - #121

Open
conwelld wants to merge 11 commits into
developmentfrom
103_flash_message
Open

conwelld wants to merge 11 commits into
developmentfrom
103_flash_message

Conversation

@conwelld

Copy link
Copy Markdown
image starting on the home page, click on the top right on "administration" image

Then click on "add course"
image

Fill out the information, with whatever dummy data you want, then the flash message will pop up, and disappear after 5 seconds

Issue Description

Fixes #<103>

  • Flash messages stayed on screen until the next page load, so they piled up and cluttered the view.

Changes

  • Added a timeout in app.js that fades out flash messages after a few seconds and removes them from the DOM.
  • Updated the HTML so flash messages have the class/wrapper that the script targets.
  • Screenshot or short screen recording of the message fading out.

Testing

  • Run the app locally and trigger an action that produces a flash message.
  • Confirm the message appears, then fades and disappears on its own without a page reload.
  • Trigger two messages back-to-back and confirm both clear independently.
  • Confirm nothing shifts or breaks on the page after the message is removed.

@conwelld conwelld self-assigned this Sep 15, 2026
@conwelld
conwelld requested a review from MImran2002 September 15, 2026 15:47
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