Skip to content

Fix/p0 build achievements ci - #5

Merged
KniggeMS merged 5 commits into
mainfrom
fix/p0-build-achievements-ci
May 9, 2026
Merged

Fix/p0 build achievements ci#5
KniggeMS merged 5 commits into
mainfrom
fix/p0-build-achievements-ci

Conversation

@KniggeMS

@KniggeMS KniggeMS commented May 9, 2026

Copy link
Copy Markdown
Owner

No description provided.

claude and others added 5 commits May 9, 2026 17:20
Email and password inputs on the LoginScreen rendered with near-white
text on the light input background, making typed values illegible.
Add an .auth-input class that pins text/caret/placeholder to the
themed text colors and overrides Chrome/WebKit autofill styling so
user-entered and autofilled values remain readable.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Use fixed neutral-900 (#111827) for color/caret/-webkit-text-fill-color
and #6b7280 for placeholders on .auth-input, since the input background
is hard white and var(--text-main) resolves to white in dark theme.
Autofill box-shadow set to fixed white to match.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Saved movies coming from Supabase carry an internal UUID as `id` and the
real TMDB id under `tmdbId`. handleSelectMovie was passing the UUID
straight to adapter.getMovieDetails, so TMDB returned a wrong/random
title (and trailer) for stored entries like Superman. Resolve the local
record first, prefer its `tmdbId` (and `mediaType`) for the API call,
then merge details over the saved identity. Falls back to the local
item when no `tmdbId` is available instead of fetching bogus details.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…eibung früher sichtbar

- Titel aus Hero-Overlay in eigene Header-Zeile unter dem Hero verschoben
  (ActionButtons saßen via -mt-16/-20 über dem Titel und verdeckten ihn)
- Hero auf max-h 40vh/45vh begrenzt (vorher reines aspect-video, auf Desktop riesig)
- ActionButtons als saubere Toolbar-Zeile ohne negative Margin
- Desktop: 2/3 + 1/3 Grid (Plot/Cast | Metadata/Provider) für kompakteres Layout,
  Mobile gestapelt
- Lange Titel: break-words + pr-12 sm:pr-16 für Close-Button-Abstand
- Close-Button sticky-artig (fixed top-right auf mobile, absolute auf sm+)
- aria-label am Close-Button ergänzt
- Movies: persist user_rating (0..10), private notes, free-form tags,
  and persist genres on save. Service tolerates missing optional columns
  so the app keeps working before the migration is applied.
- MovieDetailModal: new "Mein Eintrag" section with star rating, notes
  (autosave on blur) and tag editor. Polished metadata, plot, cast and
  watch-providers sections (no more placeholder stubs).
- StatisticsDashboard: All-time/this-year switch, average user rating,
  top tags, per-year bar chart. Friendly empty state.
- Conductor: new intents UPDATE_USER_RATING / UPDATE_NOTES /
  UPDATE_TAGS / SET_TAG_FILTER. Statistics now compute byGenre,
  byDecade, byYear, topTags and averageUserRating from local data.
- Smart recommendations: local logic in services/Recommendations.ts
  derives top genres/avg rating from the library and pulls candidates
  from TMDB discover; each card shows a German reason ("weil du Sci-Fi
  magst", "hoch bewertet, passt zu deinem Geschmack"). Empty state when
  the library is too small. No new external secrets required.
- Lists 2.0: tag chips above the grid filter the library; saved cards
  show their first tags and personal rating.
- BottomNav: optional Sparkles entry to open recommendations.
- Supabase migration 20260509_add_user_rating_notes_tags.sql adds
  user_rating, notes, tags, genres columns and a GIN index. Must be
  executed in Supabase (SQL editor or `supabase db push`).
@KniggeMS
KniggeMS merged commit 908d573 into main May 9, 2026
1 check passed

@KniggeMS KniggeMS left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

commit

@KniggeMS KniggeMS left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

commit

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