`)]
+ title := strings.Index(head, `
`)
+ tags := strings.Index(head, `
`)
+ if title < 0 || tags < 0 || tags < title {
+ t.Errorf("%s card %d: want the title row, then a .stack-tags row (title %d, tags %d)", lang, i, title, tags)
+ continue
+ }
+ if first := strings.Index(head, `class="tag `); first < tags {
+ t.Errorf("%s card %d: a tag outside the tags row (at %d, row at %d)", lang, i, first, tags)
+ }
+ if !strings.Contains(head[title:tags], `
`) {
+ t.Errorf("%s card %d: the name block has no .stack-title-text (it carries min-width: 0)", lang, i)
+ }
+ if strings.Contains(head, "subdomain-link") {
+ m := addrRe.FindStringSubmatch(head)
+ if m == nil || m[1] != m[2] || m[2] != m[3] {
+ t.Errorf("%s card %d: the address must be one ellipsis span with the full address in title: %q", lang, i, m)
+ }
+ addresses++
+ }
+ }
+ if addresses != 2 {
+ t.Errorf("%s: %d addresses, want 2 (filebrowser, privatebin)", lang, addresses)
+ }
+ }
+
+ data, err := StyleCSS()
+ if err != nil {
+ t.Fatal(err)
+ }
+ css := string(data)
+ if h := cssRule(t, css, ".stack-detail-header"); strings.Contains(h, "space-between") || !strings.Contains(h, "flex-direction: column") {
+ t.Errorf(".stack-detail-header must stack the title row over the tags, not spread them on one row: %s", h)
+ }
+ if r := cssRule(t, css, ".stack-title-row"); !strings.Contains(r, "align-items: flex-start") {
+ t.Errorf(".stack-title-row must align the logo to the top of the name: %s", r)
+ }
+ if r := cssRule(t, css, ".stack-title-text"); !strings.Contains(r, "min-width: 0") {
+ t.Errorf(".stack-title-text needs min-width: 0, or a long address cannot shrink: %s", r)
+ }
+ if r := cssRule(t, css, ".stack-tags"); !strings.Contains(r, "flex-wrap: wrap") {
+ t.Errorf(".stack-tags must wrap (English tags are longer): %s", r)
+ }
+ if r := cssRule(t, css, ".subdomain-text"); !strings.Contains(r, "text-overflow: ellipsis") || !strings.Contains(r, "overflow: hidden") {
+ t.Errorf(".subdomain-text must cut a long address with an ellipsis: %s", r)
+ }
+ if r := cssRule(t, css, ".subdomain-link"); !strings.Contains(r, "white-space: nowrap") {
+ t.Errorf(".subdomain-link must keep the arrow on the address line: %s", r)
+ }
+}
+
+// R-907 — on a phone the launcher's title, domain chip and share button sat on one row that does not wrap; the
+// button ran off the right edge (390 px, demo-hp 9201, 2026-10-08).
+//
+// RED-PROOF (REPORT): against 05e12921's style.css this fails — the 768px block has no .page-header rule.
+func TestR907_PageHeaderWrapsOnAPhone(t *testing.T) {
+ data, err := StyleCSS()
+ if err != nil {
+ t.Fatal(err)
+ }
+ block := mediaBlock768(t, string(data))
+ if !regexp.MustCompile(`\.page-header \{[^}]*flex-wrap: wrap`).MatchString(block) {
+ t.Error("the phone block must let .page-header wrap")
+ }
+ if !regexp.MustCompile(`\.share-open-btn \{[^}]*white-space: nowrap`).MatchString(block) {
+ t.Error("the share button's text must stay whole when it drops to its own line")
+ }
+ // The button is still in the header it wraps inside, in both languages.
+ s := i18nTestServer(t)
+ for _, c := range i18nCases() {
+ if c.name != "launcher_full" {
+ continue
+ }
+ for _, lang := range []string{"hu", "en"} {
+ page := renderI18nCase(t, s, lang, c)
+ i := strings.Index(page, `
-
{{if $im}}
diff --git a/controller/internal/web/templates/style.css b/controller/internal/web/templates/style.css
index 4fbbe8a..c9f71cf 100644
--- a/controller/internal/web/templates/style.css
+++ b/controller/internal/web/templates/style.css
@@ -447,24 +447,48 @@ h3 {
.stack-detail-card.stack-state-run::before,
.stack-detail-card.stack-state-progress::before { background: var(--blue); }
.stack-detail-card.stack-state-warn::before { background: var(--warn); }
+/* Card header, the same on every card (2026-10-08): row 1 = logo + name (+ the address under it), row 2 =
+ the tags, left-aligned and wrapping. The tags used to share one space-between row with the name and
+ covered it once an installed app had two of them; the logo was centred on the name block, so a card
+ with an address carried it lower. Pinned by TestStackCardHeaderLayout. */
.stack-detail-header {
display: flex;
- justify-content: space-between;
- align-items: flex-start;
+ flex-direction: column;
+ gap: .6rem;
margin-bottom: .75rem;
}
.stack-title-row {
display: flex;
- align-items: center;
+ align-items: flex-start;
gap: .75rem;
+ min-width: 0;
+}
+.stack-title-text {
flex: 1;
min-width: 0;
}
+.stack-title-text h3 { margin-bottom: .25rem; overflow-wrap: anywhere; }
+.stack-title-text .route-unpublished { white-space: normal; }
+.stack-tags {
+ display: flex;
+ flex-wrap: wrap;
+ gap: .4rem;
+}
.subdomain-link {
+ display: flex;
+ align-items: baseline;
+ gap: .3rem;
+ max-width: 100%;
font-size: .8rem;
color: var(--blue-bright);
text-decoration: none;
transition: color 0.2s ease;
+ white-space: nowrap;
+}
+.subdomain-text {
+ min-width: 0;
+ overflow: hidden;
+ text-overflow: ellipsis;
}
.subdomain-link:hover { text-decoration: underline; }
/* Tag — square 2px state chip with a 6px dot. Color only on deviation:
@@ -2675,6 +2699,10 @@ a.stat-card:hover {
.app-row { flex-wrap: wrap; }
.app-row-actions { width: 100%; justify-content: flex-end; }
.stack-grid { grid-template-columns: 1fr; }
+ /* R-907: a page header (title, domain chip, buttons) wraps instead of running off the screen; on the
+ launcher the share button drops to its own line. Pinned by TestR907_PageHeaderWrapsOnAPhone. */
+ .page-header { flex-wrap: wrap; row-gap: .6rem; }
+ .share-open-btn { margin-left: 0; white-space: nowrap; }
.stats-grid { grid-template-columns: repeat(3, 1fr); }
.deploy-info { flex-direction: column; }
.system-info-items { flex-direction: column; gap: 1rem; }
`)
+ if i < 0 {
+ t.Fatalf("%s: launcher has no .page-header", lang)
+ }
+ head := page[i : i+strings.Index(page[i:], "
")]
+ if !strings.Contains(head, "share-open-btn") {
+ t.Errorf("%s: the share button left the launcher header", lang)
+ }
+ }
+ return
+ }
+ t.Fatal("no launcher_full parity case")
+}
diff --git a/controller/internal/web/handlers.go b/controller/internal/web/handlers.go
index c545ccd..ccfa37e 100644
--- a/controller/internal/web/handlers.go
+++ b/controller/internal/web/handlers.go
@@ -139,7 +139,7 @@ func (s *Server) baseData(page, title string) map[string]interface{} {
if s.alertManager != nil {
// Hungarian here; addLanguageData re-renders the set in the request's language, because
// baseData has no request and every page goes through executeTemplate (v0.252.0, R-557).
- data["Alerts"] = s.alertManager.GetAlerts(i18n.Default)
+ data["Alerts"] = s.alertManager.GetBannerAlerts(page, i18n.Default)
}
return data
}
@@ -917,7 +917,7 @@ func (s *Server) monitoringHandler(w http.ResponseWriter, r *http.Request) {
data["StorageBars"] = s.buildStorageBars(s.langFor(r))
if s.alertManager != nil {
- data["Alerts"] = s.alertManager.GetAlerts(s.langFor(r))
+ data["Alerts"] = s.alertManager.GetBannerAlerts("monitoring", s.langFor(r))
data["DiskWarnings"] = s.alertManager.GetInlineAlerts("monitoring", s.langFor(r))
}
diff --git a/controller/internal/web/r906_overflow_test.go b/controller/internal/web/r906_overflow_test.go
new file mode 100644
index 0000000..1d5f41f
--- /dev/null
+++ b/controller/internal/web/r906_overflow_test.go
@@ -0,0 +1,80 @@
+package web
+
+import (
+ "fmt"
+ "io"
+ "log"
+ "strings"
+ "testing"
+ "time"
+
+ "gitea.dooplex.hu/admin/felhom-controller/internal/config"
+ "gitea.dooplex.hu/admin/felhom-controller/internal/monitor"
+)
+
+// R-906 — the banner showed „+ 5 további figyelmeztetés" with no warning above it. 9202 (2026-10-08) carried nine
+// „not on a separate drive" warnings (Inline, PageOnly dashboard/monitoring) and the hub-off warning: the cap took the
+// first five of all ten, counted five into the overflow line, and the layout then dropped all five visible ones,
+// because they belong under the storage bars, not in the banner. Every page showed only the overflow line, and the
+// hub-off warning — the one that DID belong in the banner — was cut.
+//
+// RED-PROOF (REPORT-dashboard-layout.md): point baseData and the monitoring handler back at GetAlerts and every page fails
+// (overflow line present, hub-off banner missing).
+func TestR906_OverflowCountsOnlyThisPagesBannerAlerts(t *testing.T) {
+ s := newDashboardServer(t, time.Time{})
+ am := NewAlertManager(log.New(io.Discard, "", 0))
+ var warnings, kinds []string
+ for i := 0; i < 9; i++ {
+ warnings = append(warnings, fmt.Sprintf("Az adattároló (/mnt/hdd/app%d) nem külön meghajtón van", i))
+ kinds = append(kinds, monitor.WarnKindStorageNotSeparate)
+ }
+ cfg := &config.Config{}
+ cfg.Hub.Enabled = false // the hub-off warning: the one banner alert this box has
+ cfg.Backup.Enabled = true
+ am.Refresh(&monitor.HealthReport{Status: "warn", Warnings: warnings, WarningKinds: kinds}, cfg, nil, false, "")
+ s.alertManager = am
+
+ if n := len(am.alerts); n != 10 {
+ t.Fatalf("setup: %d alerts, want 10 (nine inline disk warnings + hub off)", n)
+ }
+
+ for _, path := range []string{"/launcher", "/stacks", "/backups", "/monitoring", "/launcher?lang=en", "/stacks?lang=en"} {
+ rec := getPage(t, s, path)
+ if rec.Code != 200 {
+ t.Fatalf("GET %s = %d", path, rec.Code)
+ }
+ body := rec.Body.String()
+ if strings.Contains(body, "alert-banner-info") {
+ t.Errorf("%s: an overflow line is shown, but this page shows no more than one banner alert", path)
+ }
+ if got := strings.Count(body, "alert-banner-warning"); got != 1 {
+ t.Errorf("%s: %d warning banners, want 1 (the hub-off warning)", path, got)
+ }
+ }
+}
+
+// The cap still works: six alerts that ALL belong in the banner show five and „+ 1".
+func TestR906_CapStillCountsBannerAlerts(t *testing.T) {
+ am := NewAlertManager(log.New(io.Discard, "", 0))
+ var warnings []string
+ for i := 0; i < 5; i++ {
+ warnings = append(warnings, fmt.Sprintf("CPU warning %d", i))
+ }
+ cfg := &config.Config{}
+ cfg.Hub.Enabled = false
+ cfg.Backup.Enabled = true
+ am.Refresh(&monitor.HealthReport{Status: "warn", Warnings: warnings, WarningKinds: make([]string, 5)}, cfg, nil, false, "")
+
+ got := am.GetBannerAlerts("launcher", "hu")
+ if len(got) != 6 || got[5].ID != "overflow" || !strings.Contains(got[5].Message, "1") {
+ t.Fatalf("six banner alerts: got %d, last %+v — want five and an overflow of 1", len(got), got[len(got)-1])
+ }
+ // An Inline/PageOnly alert never enters the banner list, on any page.
+ for _, page := range []string{"dashboard", "monitoring", "launcher"} {
+ for _, a := range am.GetBannerAlerts(page, "hu") {
+ if !bannerShows(a, page) && a.ID != "overflow" {
+ t.Errorf("%s: %q is not a banner alert on this page", page, a.ID)
+ }
+ }
+ }
+}
diff --git a/controller/internal/web/templates/stacks.html b/controller/internal/web/templates/stacks.html
index 2e35061..570ed1d 100644
--- a/controller/internal/web/templates/stacks.html
+++ b/controller/internal/web/templates/stacks.html
@@ -24,12 +24,12 @@
+
+