Skip to content

Fix: Prevent horizontal scrollbar - #551

Merged
amirbiron merged 1 commit into
mainfrom
fix/tablet-code-view-overflow-hidden-20250917
Sep 17, 2025
Merged

amirbiron merged 1 commit into
mainfrom
fix/tablet-code-view-overflow-hidden-20250917

Conversation

@amirbiron

Copy link
Copy Markdown
Owner

What

  • Fix: מניעת גלילה אופקית ו״שטח מת״ בצפייה בקוד במיני ווב־אפ בטאבלט.
  • הוספת overflow-x: hidden; לכלל html, body ב־webapp/templates/base.html.

Why

במכשירי טאבלט הופיעה גלילה אופקית ושטח ריק בצד בעת צפייה בקוד. הגורם הוא overflow לרוחב שנגרם מאלמנט(ים) נדירים. חסימת גלילה אופקית ברמת הדף מונעת את התופעה בצורה בטוחה מבלי לשבור פריסות קיימות.

How

html, body { height: 100%; overflow-x: hidden; }

שאר הפריסה נשארה ללא שינוי. אין פגיעה ב־desktop.

Tests

  • בדיקה ידנית בטאבלט/אמולטור: מעבר לצפייה בקובץ → אין יותר גלילה אופקית.
  • בדיקה רוחבית: עמודי files, view_file, html_preview נטענים ללא תופעות לוואי.

Risks & Rollback

  • סיכון נמוך: ייתכן חיתוך מינורי של overflow אופקי חריג. במקרה כזה נטפל נקודתית באלמנט.
  • Rollback: הסרת כלל ה־CSS בקובץ base.html תחזיר התנהגות קודמת.

Reference

  • Gist ההצעה: c1ee47e6d439414040d909295b4756ab
  • מדיניות פרויקט: קוד־בוט – Android & AI Style Rules

Co-authored-by: amirbiron <amirbiron@gmail.com>
@github-actions

Copy link
Copy Markdown
Contributor

🧯 Dangerous deletes guard report

Policy: see .cursorrules — dangerous deletions are blocked unless wrapped safely.

Summary:

  • Flagged findings (blocking): 0
    0
  • Excluded matches (not blocking): 5
  • Total matches (all files): 12

Flagged findings (file:line:snippet):

Excluded matches (by path pattern):
./README.md:696:find . -name "pycache" -exec rm -rf {} +
./docs/DOCUMENTATION_GUIDE.md:450:rm -rf _build
./docs/Makefile:24: rm -rf $(BUILDDIR)
./Dockerfile:31: rm -rf /var/lib/apt/lists/*
./Dockerfile:66: rm -rf /var/lib/apt/lists/*

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