Commit Graph

25 Commits

Author SHA1 Message Date
Hermes
a765497bc9 fix(css): verhindere horizontalen scrollbalken in hero und pricing auf mobile
Some checks failed
Deploy Feature Branch to Test (haus.test.kies-media.de) / Deploy to Test Environment (push) Failing after 52s
Lint / PHP Syntax Check (push) Successful in 1m7s
PHPUnit / PHP Unit Tests (push) Failing after 37s
Lint / PHP Syntax Check (pull_request) Successful in 31s
Lint / HTML Lint (htmlhint) (push) Successful in 1m46s
Lint / CSS Lint (stylelint) (push) Successful in 1m51s
PHPUnit / PHP Unit Tests (pull_request) Failing after 45s
Lint / CSS Lint (stylelint) (pull_request) Successful in 1m20s
Lint / HTML Lint (htmlhint) (pull_request) Successful in 1m20s
Zwei weitere Overflow-Quellen identifiziert nach intro-fix:

1. .hero-content: 397.47px breit auf 320px viewport
   Ursache: Flex-item ohne width/min-width constraint, h1 min-content
   (~349px) erweitert das parent bis ueber viewport
   Fix: max-width: 100%, min-width: 0, width: 100% in @media (width <= 900px)

2. .pricing-section: ohne mobile-padding override, .rent-notes grid
   '12rem 1fr' = 192px + 1fr (min ~272px) passt nicht in 320px-24px-24px
   Fix: padding: 4rem 1.5rem + .rent-notes grid-template-columns: 1fr
   (dt/dd stapeln sich auf mobile, dt als mini-label oben)

Vorher: docScrollW=359 bei viewport=320 (3 Overflow-Quellen)
Nachher: docScrollW=320 (intro, hero, pricing alle im viewport)

Tests: 141/141 gruen.
2026-06-05 20:12:18 +00:00
Hermes
949ab201b1 fix(css): verhindere horizontalen scrollbalken im intro-bereich auf mobile
Problem: Bei viewports <375px loest die CSS-Grid '1fr'-Spalte auf die
min-content-breite von .intro-stats auf (~300px), weil grid-items
standardmaessig 'min-width: auto' haben. Das sprengt das grid und
erzeugt horizontalen overflow (docScrollW=359 bei 320px viewport).

Loesung:
- .intro-grid { min-width: 0 } erlaubt der spalte unter min-content
  zu schrumpfen und respektiert den verfuegbaren platz
- .intro-stats { flex-wrap: wrap; gap: 1.5rem 2rem } erlaubt den
  3 stat-boxen + badge auf 2 zeilen zu wrappen
- .intro-stats > .stat { flex: 1 1 auto; min-width: 0 } sauberes
  wrapping der einzelnen boxes

Diagnose: Playwright computed styles + bounding box bei 280/320/360/375px
Result: docScrollW == viewport bei 360+, 320, 280 (vorher 359/320).
Tests: 141/141 gruen.
2026-06-05 20:08:00 +00:00
Hermes
3b4c73425a fix(css): hide summary::marker for Firefox + list-style:none on li (Safari) 2026-06-05 17:24:30 +00:00
Hermes
acaea97415 fix(locale-switcher): make flag the visual anchor (32x24, no border, no lazy load)
Martin feedback round 3: dropdown still looked 'fuerchterlich' even
with the official flag-icons. Root cause: 14px vertical padding
around an 18px-tall flag meant the flag occupied only 39% of the
trigger height and was dwarfed by whitespace. Plus a 1px black
box-shadow border made flags look 'boxy', and loading='lazy' caused
empty boxes on the four menu flags the moment the <details> opened.

Changes:
- Flag size 24x18 -> 32x24 (+78% area, ~4:3 matches flag-icons)
- Trigger padding 14px 8px -> 6px (flag now 73% of trigger width,
  55% of trigger height, was 46%/39%)
- Drop the artificial 1px black box-shadow outline on flags
- Drop border-radius on flags (real flag-icons look better as
  crisp rectangles)
- Drop object-fit: cover (no longer needed for SVG)
- Drop loading='lazy' and decoding='async' (4 small SVGs, must
  be ready the moment <details> opens, not flash empty boxes)
- min-height: 44px restored on trigger for WCAG 2.5.5 touch target
- Menu border-radius 8 -> 10px, padding tightened, font-size 0.85
  -> 0.9rem for label legibility
- Two-layer box-shadow on menu for subtle elevation
2026-06-05 17:05:01 +00:00
Hermes
69e23d959a fix(flags): rename flag assets to match ADR-002 locale mapping (en->gb, uk->ua) 2026-06-04 19:45:38 +00:00
Hermes
391985cd42 fix(flags): replace hand-coded inline SVGs with official flag-icons assets
The previous inline flag SVGs were visually broken — most notably the
'en' Union Jack, which was reduced to a single X plus a cross and did
not resemble the real flag at all. The 'de' and 'ru' stripes also had
slight off-by-pixel rounding errors.

Switched to lipis/flag-icons (CC-BY 4.0) shipped as static files under
public/img/flags/. These are the canonical, professionally-designed
flag icons with correct proportions and all the details of the real
flags. Loaded via plain <img> tags (no JS, no external CDN at
runtime, no FOUC, no extra request after the page is cached).

Locale code mapping: en -> gb (per ADR-002, en = en-GB). Unknown
locales fall back to a 1x1 transparent gif so the layout stays
intact.
2026-06-04 19:43:23 +00:00
Hermes
70691ff242 fix(locale-switcher): size flag SVG in closed trigger
The original CSS scoped .flag to .locale-switcher__option only, so the
flag SVG inside the <summary> trigger rendered at 0x0 (intrinsic svg
defaults). Add a .locale-switcher .flag rule so the closed trigger
visibly shows a 24x16 flag.

Playwright recheck: trigger 52x44, flag 24x16 (matches previous inline
flags). 141/141 PHPUnit green.
2026-06-04 18:33:32 +00:00
Hermes
08235b0faf refactor(locale-switcher): single flag-sized dropdown, drop 4-inline-flag UI
The nav previously showed 4 inline flag buttons (DE/EN/UK/RU) on desktop
and a details-based dropdown on mobile. Martin asked for one dropdown with
a trigger the size of a single flag, and the 4 inline flags to go away.

- LocaleSwitcher: render a single <details class='locale-switcher'>
  everywhere; trigger is one flag + tiny caret; menu lists all 4 with labels.
- Drop the 4-inline <ul> and the locale-switcher-mobile duplicate.
- CSS: replace both blocks with one compact dropdown (flag-sized trigger,
  44px touch target via padding, scrolled/transparent-nav variants).
- Tests: assert 4 menu options, 5 flag SVGs, single <details> dropdown,
  active locale is a <span aria-current>, others are <a> with hreflang.
- 141/141 PHPUnit green.
2026-06-04 18:24:36 +00:00
Hermes
9a14803d26 fix(ui): 10 regression fixes from new design pass
- Nav: always visible glass background (no more transparent-on-top)
- Logo: remove text span, kill link underline completely
- Masonry: fix HTML class drift (gallery-grid -> masonry-grid)
- Gallery captions: rename to grid-item-label (hover-only)
- Honeypot: rename to hp-field (was rendered visible!)
- Hero: stronger gradient + text-shadow on h1/tag/meta
- LAGE features: cards with pin icon, no more bulleted list
- Map: full-viewport-width break-out from .lage-section
- Contact form: border-radius, focus glow, custom select arrow,
  working .form-submit button style
- Light text: unified --text-muted-on-dark token (replaces 4 magic
  white-XX% variants + --stone on .fact-label)
- A11y test: update honeypot class assertion
2026-06-04 18:03:50 +00:00
Hermes
7dd8023222 -m 2026-06-04 17:23:49 +00:00
Hermes
c737312ada fix(css): mobile nav overflow, hero-bg as <img>, map wrapper class
- mobile (≤900px): hide .nav-cta (was overflowing viewport by 65px)
- .nav-hamburger span: center inside button via left:50%; top:50%
- .nav-hamburger.active states: add translate(-50%,-50%) so the X
  lines stay centered after rotation
- .hero-bg: switch from background-* to object-fit/object-position
  (was an <img> but CSS targeted background-image → 1024px wide
  hero-bg broke mobile layout)
- responsive: also override nav.scrolled padding on mobile
  (specificity 0,1,1 > 0,0,1 → 3rem padding stayed in effect,
  pushing the hamburger off-screen with width:0 on scroll)
- home/index.php: rename lage-map → lage-map-wrapper so the
  existing .lage-map-wrapper CSS (border, margin-top: 3rem,
  overflow: hidden, full-width 450px iframe) actually applies
2026-06-04 16:39:05 +00:00
Hermes
d9b4c71735 fix(css): replace deprecated clip: rect() with clip-path: inset(50%) — unblocks deploy-test.yml
Some checks failed
Deploy Feature Branch to Test / PHP Syntax Check (push) Successful in 56s
Lint / PHP Syntax Check (push) Successful in 1m0s
Deploy Feature Branch to Test / CSS Lint (stylelint) (push) Successful in 1m47s
Deploy Feature Branch to Test / HTML Lint (htmlhint) (push) Successful in 1m49s
Lint / HTML Lint (htmlhint) (push) Successful in 1m16s
Lint / CSS Lint (stylelint) (push) Successful in 1m24s
Lint / PHP Syntax Check (pull_request) Successful in 37s
PHPUnit / PHP Unit Tests (push) Failing after 45s
Deploy Feature Branch to Test / Deploy to Test Environment (push) Successful in 38s
PHPUnit / PHP Unit Tests (pull_request) Failing after 56s
Lint / CSS Lint (stylelint) (pull_request) Successful in 1m23s
Lint / HTML Lint (htmlhint) (pull_request) Successful in 1m28s
2026-06-04 11:52:57 +00:00
Hermes
13a25aded2 feat(i18n): accessibility - per-field form errors, landmark aria-labels, tests (closes #76) 2026-06-04 11:04:06 +00:00
Hermes
0186de90ec feat(i18n): responsive locale-switcher with SVG flags (closes #75) 2026-06-04 09:44:40 +00:00
Hermes
ce21242308 feat(i18n): LocaleController switcher with open-redirect protection (closes #73)
- App\Controllers\LocaleController: GET /locale?set=xx&return=/path
  - Sets 1-year cookie (HttpOnly=false for SSR, SameSite=Lax, Secure on HTTPS)
  - 302 redirect to explicit return URL > Referer > /
  - Pure buildResponse() helper for unit tests (no headers/exit)
  - current() helper: resolves locale from $_GET/$_COOKIE/Accept-Language
- safeRedirect: rejects absolute URLs, protocol-relative (//evil.com),
  backslash tricks (\\evil.com), javascript:/data: schemes
- 28 PHPUnit tests (LocaleControllerTest), all green
- Total project tests now: 92
2026-06-04 08:57:33 +00:00
b774bd0363 fix: scroll-class check on load, hero null guard, JS syntax fix
Some checks failed
Lint / PHP Syntax Check (push) Has been cancelled
Lint / CSS Lint (stylelint) (push) Has been cancelled
Lint / HTML Lint (htmlhint) (push) Has been cancelled
2026-06-02 23:50:55 +02:00
ad4284733c fix: solid navbar background on scroll (closes #68)
Some checks failed
Lint / PHP Syntax Check (push) Has been cancelled
Lint / CSS Lint (stylelint) (push) Has been cancelled
Lint / HTML Lint (htmlhint) (push) Has been cancelled
2026-06-02 23:50:54 +02:00
1a72210608 Merge pull request 'feat: PHPUnit Test-Infrastruktur und Router-Tests' (#64) from feature/phpunit-tests into main
All checks were successful
Lint / PHP Syntax Check (push) Successful in 33s
Lint / CSS Lint (stylelint) (push) Successful in 1m12s
Lint / HTML Lint (htmlhint) (push) Successful in 1m9s
2026-05-22 21:33:32 +02:00
greggy
3a30abc05e fix: replace custom autoloader with composer PSR-4 autoloader
All checks were successful
Deploy Feature Branch to Test / deploy (push) Successful in 29s
Lint / PHP Syntax Check (push) Successful in 37s
Lint / CSS Lint (stylelint) (push) Successful in 1m17s
Lint / HTML Lint (htmlhint) (push) Successful in 1m15s
Lint / PHP Syntax Check (pull_request) Successful in 32s
Lint / CSS Lint (stylelint) (pull_request) Successful in 1m12s
Lint / HTML Lint (htmlhint) (pull_request) Successful in 1m8s
The custom autoloader used lowercase directory names which broke
after renaming app/core/ → app/Core/ and app/controllers/ → app/Controllers/
for PSR-4 compliance.
2026-05-22 19:19:28 +00:00
3db7dc8971 fix(#58): remove broken getElementById('contactForm') stub
All checks were successful
Deploy Feature Branch to Test / deploy (push) Successful in 26s
Lint / PHP Syntax Check (push) Successful in 35s
Lint / CSS Lint (stylelint) (push) Successful in 1m15s
Lint / HTML Lint (htmlhint) (push) Successful in 1m11s
Lint / PHP Syntax Check (pull_request) Successful in 39s
Lint / CSS Lint (stylelint) (pull_request) Successful in 1m13s
Lint / HTML Lint (htmlhint) (pull_request) Successful in 1m12s
The JS had a dangling document.getElementById('contactForm') call that
did nothing - no event handler attached, no return value used.

Form handling is entirely server-side (PHP POST → redirect to #form-result).
No JS form intervention needed.

Closes #58
2026-05-22 14:14:23 +00:00
e30bc5704b Merge PR #59: remove old haus-schleusingen.html references (fixes #56)
All checks were successful
Lint / PHP Syntax Check (push) Successful in 36s
Lint / CSS Lint (stylelint) (push) Successful in 1m25s
Lint / HTML Lint (htmlhint) (push) Successful in 1m12s
2026-05-22 14:05:29 +00:00
a170afa7c0 fix: remove mailto handler and fix formSuccess element ID (refs #57, #58)
All checks were successful
Lint / PHP Syntax Check (pull_request) Manual approval
Lint / CSS Lint (stylelint) (pull_request) Manual approval
Lint / HTML Lint (htmlhint) (pull_request) Manual approval
Deploy Feature Branch to Test / deploy (push) Manual approval
Lint / PHP Syntax Check (push) Manual approval
Lint / CSS Lint (stylelint) (push) Manual approval
Lint / HTML Lint (htmlhint) (push) Successful in 1m12s
2026-05-22 13:57:21 +00:00
148b4849fd fix: remove all references to old haus-schleusingen.html (refs #56)
All checks were successful
Deploy Feature Branch to Test / deploy (push) Successful in 25s
Lint / PHP Syntax Check (push) Successful in 32s
Lint / CSS Lint (stylelint) (push) Successful in 1m15s
Lint / HTML Lint (htmlhint) (push) Successful in 1m11s
Lint / PHP Syntax Check (pull_request) Successful in 34s
Lint / CSS Lint (stylelint) (pull_request) Successful in 1m15s
Lint / HTML Lint (htmlhint) (pull_request) Successful in 1m11s
2026-05-22 13:38:33 +00:00
2d9f1838b6 fix(security): add CSP and security headers via .htaccess (#41)
All checks were successful
Deploy Feature Branch to Test / deploy (push) Successful in 25s
Lint / PHP Syntax Check (push) Successful in 39s
Lint / CSS Lint (stylelint) (push) Successful in 1m25s
Lint / HTML Lint (htmlhint) (push) Successful in 1m10s
Lint / PHP Syntax Check (pull_request) Successful in 34s
Lint / CSS Lint (stylelint) (pull_request) Successful in 1m12s
Lint / HTML Lint (htmlhint) (pull_request) Successful in 1m8s
- Content-Security-Policy: strict CSP for static landingpage
- X-Content-Type-Options: nosniff
- X-Frame-Options: SAMEORIGIN
- Referrer-Policy: strict-origin-when-cross-origin

Fix #41
2026-05-21 23:04:52 +00:00
1aedcaf314 refactor: Umstellung auf Mini-MVC-Architektur (Issue #46)
All checks were successful
Deploy Feature Branch to Test / deploy (push) Successful in 24s
- Front Controller Pattern mit public/index.php als Einstiegspunkt
- Eigenes Routing (App\Core\Router) ohne externes Framework
- Controller: HomeController, ImpressumController, DatenschutzController
- Views mit gemeinsamem Layout (app/views/layouts/main.php)
- PSR-4 Autoloading
- Statische Assets nach public/ verschoben
- Alte Dateien (index.php, impressum.html, datenschutz.html) geloescht
- 301-Redirects fuer alte URLs
- PHP 8.5 kompatibel
- Apache DocumentRoot auf public/ gesetzt
2026-05-19 14:38:38 +00:00