Skip to content

fix(ui): Show the Stackable Cockpit name and drop placeholder wording - #469

Merged
Bohreromir merged 2 commits into
mainfrom
fix/branding-cockpit-name
Nov 12, 2026
Merged

Bohreromir merged 2 commits into
mainfrom
fix/branding-cockpit-name

Conversation

@Bohreromir

Copy link
Copy Markdown
Member

Description

The UI never named Cockpit, and the dashboard still had the initial "Stackable Unified Data Platform overview" heading.

  • Sidebar brand, logo alt text and page title suffix: "Stackable Cockpit"
  • Remove the dashboard heading (the header already says "Dashboard") and the bookmarks subtitle; the Bookmarks heading becomes an h2

Docs

Release note

Checklist

Author

  • AI review done, findings addressed
  • Tests added or updated
  • Changelog updated (if the repo keeps one)

Reviewer

  • Useful comments and logging
  • Docs added or updated per the style guide

Acceptance

  • Check for release notes and docs

The UI never named Cockpit, and the dashboard still had the initial
"Stackable Unified Data Platform overview" heading.

- Sidebar brand, logo alt text and page title suffix: "Stackable Cockpit"
- Remove the dashboard heading (the header already says "Dashboard") and
  the bookmarks subtitle; the Bookmarks heading becomes an h2
- Describe Cockpit as "the UI" rather than "web UI" in README, AGENTS.md,
  docs, Helm chart, image labels and meta description
@Bohreromir Bohreromir self-assigned this Nov 12, 2026
@Bohreromir Bohreromir moved this to Development: Waiting for Review in Stackable Engineering Nov 12, 2026
@Bohreromir
Bohreromir requested a review from lfrancke November 12, 2026 14:00
Comment thread messages/de.json Resolved
Comment thread messages/en.json Resolved
@lfrancke lfrancke moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Nov 12, 2026
@github-actions

github-actions Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 67.57% 9253 / 13693
🔵 Statements 66.49% 12429 / 18691
🔵 Functions 68.33% 2980 / 4361
🔵 Branches 57.49% 5214 / 9069
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/lib/components/dashboard/DashboardBookmarks.svelte 98.86% 85.71% 100% 100% 42
src/lib/components/layout/sidebar/Sidebar.svelte 0% 0% 0% 0% 116-288
src/routes/(app)/+layout.svelte 0% 0% 0% 0% 11-106
src/routes/(app)/+page.svelte 80% 83.33% 100% 85.71% 4-14
src/routes/auth/login/+page.svelte 0% 0% 0% 0% 7-63
Generated in workflow #1300 for commit ea90030 by the Vitest Coverage Report Action

Apply review suggestions and align the README wording.
@Bohreromir
Bohreromir requested a review from lfrancke November 12, 2026 14:26
@Bohreromir
Bohreromir enabled auto-merge November 12, 2026 14:28
@Bohreromir
Bohreromir added this pull request to the merge queue Nov 12, 2026
Merged via the queue into main with commit b60c256 Nov 12, 2026
12 checks passed
@Bohreromir
Bohreromir deleted the fix/branding-cockpit-name branch November 12, 2026 15:02
@Bohreromir Bohreromir moved this from Development: In Review to Development: Done in Stackable Engineering Nov 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Done

Development

Successfully merging this pull request may close these issues.

2 participants