UI-Politur: Kontrast, Abstaende, Typografie, Bewegung
Kein Redesign — Schwarz/Rot, 4 Tabs und Lieder/Kategorie bleiben. Grundlage sind die Design-Skills: ui-ux-pro-max (Barrierefreiheit, Touch, Typografie, Bewegung), ui-design/mobile-android (Material 3) und die Token-Disziplin aus Hue. Behobene Maengel (messbar, nicht Geschmack): - Text in Colors.white38 an 17 Stellen = 3,4:1 Kontrast, WCAG verlangt 4,5:1. Ersetzt durch drei benannte Stufen (text1 15,9:1 / text2 8,8:1 / text3 4,9:1). theme_kontrast_test.dart rechnet die Verhaeltnisse bei jedem Lauf nach, statt sie zu behaupten. - SubTabs waren ~38 dp hoch (Material 3 verlangt 48), ohne Ripple und ohne Uebergang. Jetzt 48 dp, InkWell, AnimatedContainer, und die Auswahl wird Vorlesehilfen als Zustand gemeldet (Semantics.selected). - mini_player.dart: Expanded UM eine feste Hoehe herum — zwei widerspruechliche Angaben. Entschaerft. - Emoji in Bedienelementen der Einstellungen (jeweils neben einem echten Icon) entfernt. Neu in shared/theme.dart: MeloSpace (8er-Raster), MeloRadius, MeloMotion, Farbrollen text1/2/3 + border + hairline, minTouchTarget, vollstaendiges textTheme (7 Stufen) und Themes fuer Listen, Sheets, Snackbars, Fortschritt, Trenner. 318 Tests gruen (23 neue, 1 uebersprungen), flutter analyze ohne Befund. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FpPu4nuKjKKeX1RpdDeX81
This commit is contained in:
co-authored by
Claude Opus 5
parent
2df85e650e
commit
bc861f6087
@@ -91,11 +91,11 @@ void main() {
|
||||
|
||||
// Die Sync-Einträge liegen weiter unten in der Liste und müssen erst
|
||||
// in den sichtbaren Bereich gescrollt werden.
|
||||
await tester.scrollUntilVisible(find.text('🌟 Favoriten-Sync'), 200);
|
||||
await tester.scrollUntilVisible(find.text('Favoriten-Sync'), 200);
|
||||
await tester.pumpAndSettle();
|
||||
|
||||
expect(find.text('🌟 Favoriten-Sync'), findsOneWidget);
|
||||
expect(find.text('📋 Playlisten-Sync'), findsOneWidget);
|
||||
expect(find.text('Favoriten-Sync'), findsOneWidget);
|
||||
expect(find.text('Playlisten-Sync'), findsOneWidget);
|
||||
|
||||
await db.close();
|
||||
});
|
||||
|
||||
@@ -0,0 +1,99 @@
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:melo/shared/sub_tabs.dart';
|
||||
import 'package:melo/shared/theme.dart';
|
||||
|
||||
Widget _wrap(Widget child) => MaterialApp(
|
||||
theme: MeloTheme.dark,
|
||||
home: Scaffold(body: child),
|
||||
);
|
||||
|
||||
void main() {
|
||||
testWidgets('jeder Reiter ist mindestens 48 dp hoch — sonst trifft man ihn '
|
||||
'im Gehen nicht', (tester) async {
|
||||
await tester.pumpWidget(_wrap(SubTabs(
|
||||
labels: const ['Lieder', 'Kategorie', 'Künstler'],
|
||||
index: 0,
|
||||
onChanged: (_) {},
|
||||
)));
|
||||
await tester.pumpAndSettle();
|
||||
|
||||
for (final label in ['Lieder', 'Kategorie', 'Künstler']) {
|
||||
final flaeche = tester.getSize(find.ancestor(
|
||||
of: find.text(label),
|
||||
matching: find.byType(AnimatedContainer),
|
||||
));
|
||||
expect(flaeche.height, greaterThanOrEqualTo(MeloTheme.minTouchTarget),
|
||||
reason: 'Reiter "$label" ist nur ${flaeche.height} dp hoch');
|
||||
expect(flaeche.width, greaterThanOrEqualTo(MeloTheme.minTouchTarget));
|
||||
}
|
||||
});
|
||||
|
||||
testWidgets('ein Tipp meldet den angetippten Reiter', (tester) async {
|
||||
int? gewaehlt;
|
||||
await tester.pumpWidget(_wrap(SubTabs(
|
||||
labels: const ['Lieder', 'Kategorie'],
|
||||
index: 0,
|
||||
onChanged: (i) => gewaehlt = i,
|
||||
)));
|
||||
|
||||
await tester.tap(find.text('Kategorie'));
|
||||
await tester.pumpAndSettle();
|
||||
|
||||
expect(gewaehlt, 1);
|
||||
});
|
||||
|
||||
testWidgets('der aktive Reiter ist auch ohne Farbe erkennbar', (tester) async {
|
||||
final semantik = tester.ensureSemantics();
|
||||
|
||||
await tester.pumpWidget(_wrap(SubTabs(
|
||||
labels: const ['Lieder', 'Kategorie'],
|
||||
index: 1,
|
||||
onChanged: (_) {},
|
||||
)));
|
||||
await tester.pumpAndSettle();
|
||||
|
||||
// Vorlesehilfen brauchen die Auswahl als Zustand, nicht als Farbe —
|
||||
// wer die weiße Pille nicht sieht, erfährt sonst nie, wo er ist.
|
||||
expect(
|
||||
tester.getSemantics(find.text('Kategorie')),
|
||||
isSemantics(label: 'Kategorie', isSelected: true, isButton: true),
|
||||
);
|
||||
expect(
|
||||
tester.getSemantics(find.text('Lieder')),
|
||||
isSemantics(label: 'Lieder', isSelected: false, isButton: true),
|
||||
);
|
||||
|
||||
// Muss vor Testende passieren — addTearDown wäre zu spät.
|
||||
semantik.dispose();
|
||||
});
|
||||
|
||||
testWidgets('jeder Reiter gibt beim Antippen Rückmeldung (Ripple)',
|
||||
(tester) async {
|
||||
await tester.pumpWidget(_wrap(SubTabs(
|
||||
labels: const ['Lieder', 'Kategorie'],
|
||||
index: 0,
|
||||
onChanged: (_) {},
|
||||
)));
|
||||
|
||||
// Ohne InkWell bliebe der Tipp unbestätigt — der Nutzer weiß nicht,
|
||||
// ob er getroffen hat.
|
||||
expect(find.byType(InkWell), findsNWidgets(2));
|
||||
});
|
||||
|
||||
testWidgets('der Wechsel wird animiert, nicht gesprungen', (tester) async {
|
||||
await tester.pumpWidget(_wrap(SubTabs(
|
||||
labels: const ['Lieder', 'Kategorie'],
|
||||
index: 0,
|
||||
onChanged: (_) {},
|
||||
)));
|
||||
await tester.pumpAndSettle();
|
||||
|
||||
final container = tester.widget<AnimatedContainer>(find.ancestor(
|
||||
of: find.text('Lieder'),
|
||||
matching: find.byType(AnimatedContainer),
|
||||
));
|
||||
expect(container.duration, MeloMotion.fast);
|
||||
expect(container.curve, MeloMotion.curve);
|
||||
});
|
||||
}
|
||||
@@ -0,0 +1,152 @@
|
||||
import 'dart:math' as math;
|
||||
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:flutter_test/flutter_test.dart';
|
||||
import 'package:melo/shared/theme.dart';
|
||||
|
||||
/// Relative Helligkeit nach WCAG 2.1, 1.4.3.
|
||||
double _luminanz(Color farbe) {
|
||||
double kanal(double wert) =>
|
||||
wert <= 0.03928 ? wert / 12.92 : math.pow((wert + 0.055) / 1.055, 2.4) as double;
|
||||
return 0.2126 * kanal(farbe.r) +
|
||||
0.7152 * kanal(farbe.g) +
|
||||
0.0722 * kanal(farbe.b);
|
||||
}
|
||||
|
||||
/// Legt [vordergrund] (ggf. halbdurchsichtig) auf [hintergrund].
|
||||
Color _ueber(Color vordergrund, Color hintergrund) {
|
||||
final a = vordergrund.a;
|
||||
return Color.from(
|
||||
alpha: 1,
|
||||
red: vordergrund.r * a + hintergrund.r * (1 - a),
|
||||
green: vordergrund.g * a + hintergrund.g * (1 - a),
|
||||
blue: vordergrund.b * a + hintergrund.b * (1 - a),
|
||||
);
|
||||
}
|
||||
|
||||
/// Kontrastverhältnis nach WCAG 2.1 — 1 (unlesbar) bis 21 (Schwarz auf Weiß).
|
||||
double kontrast(Color vordergrund, Color hintergrund) {
|
||||
final a = _luminanz(_ueber(vordergrund, hintergrund));
|
||||
final b = _luminanz(hintergrund);
|
||||
final hell = math.max(a, b);
|
||||
final dunkel = math.min(a, b);
|
||||
return (hell + 0.05) / (dunkel + 0.05);
|
||||
}
|
||||
|
||||
void main() {
|
||||
// Die hellste Fläche ist der ungünstigste Fall: helle Schrift hat dort
|
||||
// den geringsten Abstand zum Untergrund. Wer hier besteht, besteht überall.
|
||||
const flaechen = {
|
||||
'Hintergrund': MeloTheme.black,
|
||||
'Fläche': MeloTheme.surface,
|
||||
'Karte': MeloTheme.surfaceHigh,
|
||||
};
|
||||
|
||||
group('Textfarben erfüllen WCAG 4,5:1 auf jeder Fläche', () {
|
||||
const textfarben = {
|
||||
'text1': MeloTheme.text1,
|
||||
'text2': MeloTheme.text2,
|
||||
'text3': MeloTheme.text3,
|
||||
};
|
||||
|
||||
for (final flaeche in flaechen.entries) {
|
||||
for (final text in textfarben.entries) {
|
||||
test('${text.key} auf ${flaeche.key}', () {
|
||||
final wert = kontrast(text.value, flaeche.value);
|
||||
expect(wert, greaterThanOrEqualTo(4.5),
|
||||
reason: '${text.key} auf ${flaeche.key} nur '
|
||||
'${wert.toStringAsFixed(2)}:1');
|
||||
});
|
||||
}
|
||||
}
|
||||
});
|
||||
|
||||
test('die Rot-Markierung ist auf dem Hintergrund erkennbar', () {
|
||||
// 3:1 genügt: Rot trägt keinen Fließtext, es markiert (aktiver Reiter,
|
||||
// Fortschritt). Für Text wäre 4,5:1 nötig — deshalb steht auf Rot Weiß.
|
||||
expect(kontrast(MeloTheme.red, MeloTheme.black),
|
||||
greaterThanOrEqualTo(3.0));
|
||||
});
|
||||
|
||||
test('Colors.white38 wäre zu blass — genau deshalb gibt es text3', () {
|
||||
// Festhalten, warum die alte Schreibweise ersetzt wurde: sie verfehlt
|
||||
// die Grenze. Fällt dieser Test, ist die Rechnung kaputt, nicht das Theme.
|
||||
expect(kontrast(Colors.white38, MeloTheme.surfaceHigh), lessThan(4.5));
|
||||
expect(kontrast(MeloTheme.text3, MeloTheme.surfaceHigh),
|
||||
greaterThanOrEqualTo(4.5));
|
||||
});
|
||||
|
||||
test('die Rechnung selbst stimmt: Schwarz auf Weiß ergibt 21:1', () {
|
||||
expect(kontrast(const Color(0xFF000000), const Color(0xFFFFFFFF)),
|
||||
closeTo(21.0, 0.01));
|
||||
});
|
||||
|
||||
group('Abstände rasten auf 8 (4 nur für Feinheiten)', () {
|
||||
test('alle Stufen sind Vielfache von 4', () {
|
||||
for (final wert in [
|
||||
MeloSpace.xs,
|
||||
MeloSpace.sm,
|
||||
MeloSpace.md,
|
||||
MeloSpace.lg,
|
||||
MeloSpace.xl,
|
||||
]) {
|
||||
expect(wert % 4, 0, reason: '$wert passt nicht ins Raster');
|
||||
}
|
||||
});
|
||||
|
||||
test('ab der Standardstufe sind es Vielfache von 8', () {
|
||||
for (final wert in [MeloSpace.sm, MeloSpace.md, MeloSpace.lg, MeloSpace.xl]) {
|
||||
expect(wert % 8, 0, reason: '$wert passt nicht ins 8er-Raster');
|
||||
}
|
||||
});
|
||||
|
||||
test('die Stufen steigen streng', () {
|
||||
const stufen = [
|
||||
MeloSpace.xs,
|
||||
MeloSpace.sm,
|
||||
MeloSpace.md,
|
||||
MeloSpace.lg,
|
||||
MeloSpace.xl,
|
||||
];
|
||||
for (var i = 1; i < stufen.length; i++) {
|
||||
expect(stufen[i], greaterThan(stufen[i - 1]));
|
||||
}
|
||||
});
|
||||
});
|
||||
|
||||
group('Typo-Skala', () {
|
||||
final skala = MeloTheme.dark.textTheme;
|
||||
|
||||
test('kein Text ist kleiner als 12 — darunter wird es unlesbar', () {
|
||||
for (final stil in [
|
||||
skala.displaySmall,
|
||||
skala.headlineSmall,
|
||||
skala.titleMedium,
|
||||
skala.bodyLarge,
|
||||
skala.bodyMedium,
|
||||
skala.bodySmall,
|
||||
skala.labelLarge,
|
||||
]) {
|
||||
expect(stil!.fontSize, greaterThanOrEqualTo(12));
|
||||
}
|
||||
});
|
||||
|
||||
test('Fließtext hat mindestens 1,35 Zeilenhöhe', () {
|
||||
expect(skala.bodyLarge!.height, greaterThanOrEqualTo(1.35));
|
||||
expect(skala.bodyMedium!.height, greaterThanOrEqualTo(1.35));
|
||||
});
|
||||
|
||||
test('die Größen nehmen von Überschrift zu Beiwerk ab', () {
|
||||
final groessen = [
|
||||
skala.displaySmall!.fontSize!,
|
||||
skala.headlineSmall!.fontSize!,
|
||||
skala.titleMedium!.fontSize!,
|
||||
skala.bodyLarge!.fontSize!,
|
||||
skala.bodyMedium!.fontSize!,
|
||||
];
|
||||
for (var i = 1; i < groessen.length; i++) {
|
||||
expect(groessen[i], lessThan(groessen[i - 1]));
|
||||
}
|
||||
});
|
||||
});
|
||||
}
|
||||
Reference in New Issue
Block a user