Skip to content

feat: add demo shutdown notice banner across all pages#229

Merged
josecelano merged 2 commits into
torrust:developfrom
josecelano:228-add-demo-shutdown-banner
Jun 26, 2026
Merged

feat: add demo shutdown notice banner across all pages#229
josecelano merged 2 commits into
torrust:developfrom
josecelano:228-add-demo-shutdown-banner

Conversation

@josecelano

Copy link
Copy Markdown
Member

Closes #228

Adds a prominent full-width banner at the very top of every page (above the header) notifying visitors that all Torrust public demo services are currently shut down.

Changes

  1. New banner in layout (src/routes/+layout.svelte) — Dark-themed banner with red left accent bar and red border, positioned above the navigation header so it's visible on every page.

  2. Removed old subtle notice (src/lib/components/organisms/Hero.svelte) — The tiny .demo-notice paragraph from the hero section has been removed (was hard to read and easy to miss).

  3. Demo links redirected to issue Add banner on homepage informing that all Torrust public demos are currently down #228 — All "Live demo" buttons and demo URLs now point to the issue instead of broken demo endpoints:

    • HeroCard live demo buttons (Index & Tracker)
    • TorrustIndexPost.svelte demo site link
    • constants.ts demo fields in both index and tracker comparison tables

Revert

See the comment on issue #228 for a full revert checklist to undo these changes when the demos are back online.

Copilot AI review requested due to automatic review settings June 26, 2026 16:48

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a prominent, site-wide notice that Torrust public demo services are shut down, and redirects several demo URLs to the tracking issue (#228) to avoid sending users to broken endpoints.

Changes:

  • Added a full-width shutdown banner above the header in the root layout.
  • Repointed “demo/live demo” URLs in key homepage components and comparison-table data to issue #228.
  • Minor homepage styling change (currently introduces a duplicated selector / likely unintended styling regression).

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/routes/+layout.svelte Adds the global demo shutdown banner and its styling.
src/routes/(home)/+page.svelte Updates homepage styling (currently duplicates .container selector, likely meant to style h2).
src/lib/constants/constants.ts Redirects index/tracker table demo URLs to issue #228.
src/lib/components/singletons/TorrustIndexPost.svelte Redirects the “demo site” link to issue #228 (copy still claims the demo is running).
src/lib/components/organisms/Hero.svelte Redirects HeroCard “Live demo” targets to issue #228 (button label remains “Live demo”).
Comments suppressed due to low confidence (1)

src/routes/(home)/+page.svelte:63

  • The styles define .container twice; the second block looks like it was intended for the h2 but now applies font-size/font-weight to the whole container, likely affecting all nested text and overriding earlier container padding.
	.container {
		text-align: center;
		color: rgba(245, 245, 245, 0.96);
		padding-top: 4rem;
		font-size: 1.8rem;

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/routes/+layout.svelte Outdated
Comment thread src/routes/+layout.svelte
Comment thread src/lib/components/singletons/TorrustIndexPost.svelte Outdated
Comment thread src/lib/components/organisms/Hero.svelte
@josecelano josecelano self-assigned this Jun 26, 2026
@josecelano

Copy link
Copy Markdown
Member Author

Fixed by adding rel="noopener noreferrer" to the banner link in src/routes/+layout.svelte.

@josecelano

Copy link
Copy Markdown
Member Author

Updated the following blog posts to replace or note dead demo links:

Historical code examples (like wget commands) were left as-is since they document the actual commands that were run at the time.

@josecelano

Copy link
Copy Markdown
Member Author

Fixed. Updated the text in src/lib/components/singletons/TorrustIndexPost.svelte to say "We previously ran a demo site on Digital Ocean... though all public demo services are currently shut down."

@josecelano

Copy link
Copy Markdown
Member Author

Fixed. Updated HeroCard.svelte to detect when the demo link points to the shutdown issue and display "Unavailable" instead of "Live demo", with a dimmed/greyed-out appearance (reduced opacity, grey background, no pointer-events).

- Add rel="noopener noreferrer" to banner link
- Update TorrustIndexPost text to reflect demo shutdown
- Change HeroCard button to show "Unavailable" when demo is down
- Fix duplicated .container CSS selector in homepage
- Replace broken demo links in blog posts with issue reference
@josecelano

Copy link
Copy Markdown
Member Author

ACK 26c69f7

@josecelano josecelano merged commit fd0101e into torrust:develop Jun 26, 2026
1 check passed
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.

Add banner on homepage informing that all Torrust public demos are currently down

2 participants