Skip to content

Vergici: seçili sayının rakamı okunmuyordu, zemin doygunluğu artırıldı - #110

Open
aripdcem wants to merge 1 commit into
mainfrom
claude/optimistic-brahmagupta-svlt7n
Open

aripdcem wants to merge 1 commit into
mainfrom
claude/optimistic-brahmagupta-svlt7n

Conversation

@aripdcem

Copy link
Copy Markdown
Collaborator

Kanıt

app/src/main/kotlin/com/aripd/zagames/ui/vergici/VergiciScreen.kt:301 seçili hücrenin zeminini CoinColor.copy(alpha = 0.45f) çiziyor; :328 aynı hücrenin rakamını koyu mürekkeple (Color(0xFF1C1917)) yazıyor. Saydam sarı uygulama arka planıyla (ZaColors.background, 0xFF0B0F1A) karışıyor. WCAG göreli parlaklıkla ölçüm (Python, ThemeContrastTest ile aynı formül):

selected now  : ink on coin@0.45 -> 2.83
selected alpha 0.85: ink -> 7.74
takeable: onSV on SV -> 7.44

Depo kuralı docs/oyun-testi.md E1: metin/zemin çiftleri WCAG AA eşiğini (4,5:1) tutar. ThemeContrastTest yalnızca tema çiftlerini ölçtüğünden bu ekran renkleri o testin dışında kalıyordu.

Oyuncunun gördüğü etki

Vergici'de bir sayıya dokununca sayı seçili kalır ve ikinci dokunuş onaydır (VergiciViewModel.kt:84). Yani oyuncu kararını verirken tam da en önemli hücrenin rakamı 2,8:1 kontrastla, koyu zemin üzerinde koyu zeytin sarıda duruyordu; büyük yazı eşiği olan 3:1'in bile altında.

Düzeltme neden en küçük güvenli değişiklik

  • Tek değer değişti: seçili zeminin saydamlığı 0,45 → 0,85 (SelectedFillAlpha). Toplam'daki sahipli pulla aynı doygunluk (ToplamScreen.kt:248, P0Color.copy(alpha = 0.85f) + koyu Ink); tasarım niyeti (sarı üstünde koyu mürekkep) ve altın çerçeve korunuyor, başka hiçbir hücre durumu ya da dizge değişmiyor.
  • CoinColor ve mürekkep internal yapıldı ki VergiciCellContrastTest karışımı hesaplayıp 4,5:1 eşiğini koruyabilsin; CI :app:testDebugUnitTest ile koşar.
  • Sürüm satırı, Changelog ve kelime listelerine dokunulmadı.

Doğrulama

  • Motor testleri burada koştu: 27 modül, 404 test, 0 hata (gradle --no-configuration-cache --configure-on-demand :games:*:test).
  • tools/check_strings.py ve tools/check_wordlists.py temiz.
  • Bu ortamda Android SDK yok; app derlemesi ve yeni birim testi CI'da doğrulanacak.

🤖 Generated with Claude Code

https://claude.ai/code/session_01HUx4sHGUvo1MwgrgpcMSND


Generated by Claude Code

Seçili hücre koyu mürekkebi (0xFF1C1917) %45 saydam sarının üzerine
çiziyordu; sarı uygulama arka planıyla (0xFF0B0F1A) karışınca zemin
0x775E1E'ye düşüyor ve kontrast 2,83:1 kalıyordu. Depo kuralı WCAG AA
(4,5:1); Toplam'daki sahipli pul gibi 0,85 doygunlukla 7,74:1.

Seçim onaya kadar ekranda kaldığından oyuncu kararını verirken rakamı
seçemiyordu. VergiciCellContrastTest karışımı hesaplayıp eşiği korur.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HUx4sHGUvo1MwgrgpcMSND

This branch has not been deployed

No deployments
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