Skip to content

תיקוני נגישות ועיצוב: ניגודיות, דיאלוגים בטלפון, מקלדת - #64

Open
yt314 wants to merge 1 commit into
mainfrom
claude/project-thread-okf5vd
Open

yt314 wants to merge 1 commit into
mainfrom
claude/project-thread-okf5vd

Conversation

@yt314

@yt314 yt314 commented Oct 20, 2026 •

Copy link
Copy Markdown
Owner

תיקון סעיפי P1 מבדיקת העיצוב.

מה השתנה

  • צבע ראשי (מצב בהיר): primary.700 במקום 600. טקסט לבן על הכפתור עולה מ-3.7:1 ל-5.5:1 (תקן AA).
  • טקסט משני (מצב כהה): surface.300 במקום 400. היה מתחת ל-4:1 על כרטיסים.
  • רצועת הפתיחה בספרייה: גרדיאנט כהה יותר בקצוות, והוסרה השקיפות מהטקסט, כך שטקסט לבן קריא לכל הרוחב.
  • הודעות שגיאה בטפסים: מחלקה field-error על טוקן חדש dangerText (red-700 בבהיר, red-300 בכהה) במקום text-red-500 (3.8:1), ועם role="alert" כדי שקוראי מסך יקריאו אותן. 89 מקומות.
  • דיאלוגים בטלפון: max-width גלובלי ל-.p-dialog, כך שאף דיאלוג ברוחב קבוע לא גולש מהמסך.
  • מקלדת: שם התלמיד ברשימה הוא קישור אמיתי (לחיצה על כל השורה נשארת לעכבר), ופריטי ההתראות הם כפתורים.
  • תוויות: aria-label לשדות החיפוש והסינון בספרייה, בחיפוש המהיר ובמסכי הניהול, ולכפתור החיפוש בתקלות.

בדיקה

  • ng build עובר, ו-40 הבדיקות עוברות.
  • צילום של הספרייה בדסקטופ ובטלפון (375px): הטקסט ברצועת הפתיחה קריא.

- Primary color teal.700 in light mode so white button text meets AA (was 3.7:1)
- Dark-mode muted text surface.300 (was under 4:1 on cards)
- Darker light-mode hero gradient so white hero text stays readable
- Form errors use a dangerText token and role=alert instead of red-500
- Dialogs never exceed the viewport width on phones
- Student name is a real link; notifications are buttons (keyboard access)
- Labels for placeholder-only search fields
@yt314 yt314 self-assigned this Oct 20, 2026
@vercel

vercel Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
talmidon Ready Ready Preview Oct 20, 2026 9:34am UTC

This branch was successfully deployed

1 active deployment
Preview — dcc84923 Deployed Oct 20, 2026 by vercel[bot]
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.

1 participant