Skip to content

Fix button and scroll layout issues - #846

Merged
amirbiron merged 1 commit into
mainfrom
cursor/fix-button-and-scroll-layout-issues-cb78
Oct 17, 2025
Merged

amirbiron merged 1 commit into
mainfrom
cursor/fix-button-and-scroll-layout-issues-cb78

Conversation

@amirbiron

@amirbiron amirbiron commented Oct 17, 2025 •

Copy link
Copy Markdown
Owner

✨ תיאור קצר

תיקנתי גלישת תוכן שמאלה במובייל, שכללה את רקע ה"גל", כפתור "בחירה מרובה" ותיבת השלמות החיפוש. השינויים נועדו למנוע גלילה אופקית ולשפר את חווית המשתמש במסכים קטנים.

📦 שינויים עיקריים

  • קוד (Backend)
  • בוט טלגרם
  • מסד נתונים/מיגרציות
  • תיעוד (docs/)
  • DevOps/CI/CD

פירוט נקודות (רשימת תבליטים):

  • מרכוז רקע ה"גל" ב-base.html ומניעת גלישה אופקית על ידי הגדרת background-position: bottom center ו-overflow: hidden לרכיבים רלוונטיים.
  • טיפול בחריגה שמאלה של כפתור "בחירה מרובה" ב-multi-select.css וב-files.html באמצעות max-width, overflow: hidden, text-overflow: ellipsis ו-flex-wrap לשורת הכפתורים.
  • עיגון תיבת ההשלמות של החיפוש (#searchSuggestions) בתוך המכולה ב-files.html על ידי הגדרת left: 0; right: 0; max-width: 100vw;.

🧪 בדיקות

  • איך בדקתם? מה עבר? מה נשאר?
    • בדיקה ידנית על דפדפן במצב מובייל (Chrome DevTools) לוודא שהרקע, הכפתורים ותיבת החיפוש אינם חורגים מהמסך ואינם יוצרים גלילה אופקית.
  • Unit
  • Integration
  • Manual

🧪 בדיקות נדרשות ב‑PR

  • 🔍 Code Quality & Security
  • 🧪 Unit Tests (3.11)
  • 🧪 Unit Tests (3.12)

📝 סוג שינוי

  • feat: פיצ'ר חדש
  • fix: תיקון באג
  • docs: שינוי תיעוד בלבד
  • refactor: שינוי קוד ללא שינוי התנהגות
  • perf: שיפור ביצועים
  • chore/ci: תשתית/CI
  • breaking change: שינוי שובר תאימות

✅ צ'קליסט

  • הקוד עוקב אחרי הסגנון (Black/isort/flake8/mypy)
  • בדיקות רצות ועוברות (בדיקה ידנית)
  • תיעוד עודכן (README/Docs)
  • אין סודות/מפתחות בקוד
  • אין מחיקות מסוכנות/פעולות על root (ראו .cursorrules)
  • הודעת הקומיט תואמת Conventional Commits (ע"פ הטבלה)
  • CHANGELOG עודכן אם נדרש
  • כל ה‑Required Checks לעיל ירוקים
  • צילום/וידאו UI מצורף אם רלוונטי

🧩 השפעות/סיכונים

  • השפעה אפשרית על פרודקשן, ביצועים, או אבטחה: שינויים ויזואליים בלבד, סיכון נמוך.

🔗 קישורים

  • Issues קשורים: #
  • Docs Preview:
  • מסמכים/מפרטים רלוונטיים:

🧯 סיכון / החזרה לאחור (Rollback)

  • תוכנית חזרה לאחור במקרה תקלה: ניתן לבצע Rollback לשינויי ה-CSS וה-HTML בקבצים base.html, multi-select.css ו-files.html.

Open in Cursor Open in Web


Note

Prevents horizontal scrolling on mobile by centering/resizing the background, constraining toolbars/buttons, and anchoring search suggestions within the container.

  • UI/Responsive:
    • Base layout (templates/base.html):
      • Center and resize body background wave (body::before) and make it fixed; add mobile tuning to avoid white edges.
      • Prevent horizontal scroll via overflow-x: hidden and max-width: 100% on .main-content and .container.
    • Files page (templates/files.html):
      • Anchor #searchSuggestions within its container (left: 0; right: 0; max-width: 100vw).
      • Wrap action buttons in .results-info; add min-width: 0 and overflow guards to prevent spill.
      • Apply min-width: 0 to #multiSelectToggleBtn in the toolbar.
    • Multi-select styles (static/css/multi-select.css):
      • Limit .bulk-actions-toolbar width on mobile.
      • Constrain #multiSelectToggleBtn (ellipsis, overflow handling, responsive flex behavior).
      • Allow wrapping/gap for the buttons container inside .results-info.

Written by Cursor Bugbot for commit 31cd933. This will update automatically on new commits. Configure here.

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

cursor Bot commented Oct 17, 2025

Copy link
Copy Markdown
Contributor

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@github-actions

github-actions Bot commented Oct 17, 2025 •

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): 15

Flagged findings (file:line:snippet):

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

@codecov

codecov Bot commented Oct 17, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@amirbiron
amirbiron marked this pull request as ready for review October 17, 2025 11:21
@amirbiron
amirbiron merged commit 0b00245 into main Oct 17, 2025
32 checks passed
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