Fix bot issue on github

[amirbiron]

מה

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

למה

• שיפור קריאות ויציבות פריסה (RTL/Mobile).
• מניעת “בריחת” תוכן וגלילה אופקית.

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

• webapp/static/css/collections.css:
  • grid-template-columns: 280px minmax(0, 1fr) למניעת חריגה.
  • .collections-content: הוספת min-width:0 ו-overflow-x:hidden.
  • .collection-item: הוספת min-width:0.
  • .collection-item .file: display:block + ellipsis יציב.
  • .collection-items: פריסה אנכית ו-gap.
• webapp/static/js/collections.js:
  • פונקציית autoFitText להקטנת טקסט דינמית (ברירת מחדל 16px→12px).
  • קריאה לאחר רינדור פריטים/סיידבר, ותזמון בעומס נמוך ב-resize.

תוכנית בדיקות

1. פתחו /collections, צרו אוסף והוסיפו פריט בשם ארוך מאוד (100–200 תווים, עם רווחים וקו תחתון).
2. בדקו:
   • שם הפריט והאוסף נכנסים בשורה אחת; אין גלילה אופקית/חיתוך.
   • במובייל (<480px) ההתנהגות נשמרת (שני האזורים: סיידבר ופריטים).
   • קליקים: פתיחת קובץ, הצמדה/ביטול, הסרה – תקינים.
   • גרירה (DnD) לא שוברת פריסה.
   • רספונסיביות בזמן שינוי גודל חלון.
3. דפדפנים: Chrome/Firefox/Safari אחרונים.

סיכונים ו-Rollback

• השפעת ביצועים זניחה: חישוב מוקטן ומבוקר ב-resize.
• Rollback: החזרת השינויים בקבצים collections.css ו-collections.js.

קישורים ותיעוד

• עיינתי ב־CodeBot – Project Docs והקפדתי על סגנון ה-UI הקיים.
• Issue: עדיין מסך נחתך #1142 – “עדיין מסך נחתך”.

צילומי מסך

לפני / אחרי (צרפו תמונות)

לפני

אחרי

צ'קליסט

• נבדק מובייל (Android/iOS)
• נבדק RTL
• אין רגרסיה בסידור כפתורים/גרירה
• הוספו צילומי מסך

 

---

Note

Prevents horizontal overflow in Collections by tightening layout CSS and introducing throttled dynamic text auto-fit for long collection and file names.

• UI (Collections):
  • Layout/Overflow:
    • Change grid-template-columns to 280px minmax(0,1fr) in webapp/static/css/collections.css.
    • Add min-width: 0 and overflow-x: hidden to .collections-content; ensure .collection-items and .collection-item can shrink without horizontal scroll.
    • Make .collection-item .file display: block with stable ellipsis.
  • Dynamic text fitting:
    • Add autoFitText with throttled resize handling in webapp/static/js/collections.js.
    • Invoke for long names in sidebar (.sidebar-item .name) and item list (.file) after render and on window resize.
  • Misc UI:
    • Minor structural hooks for DnD remain; no API changes.

^{Written by Cursor Bugbot for commit 1e141b0. This will update automatically on new commits. Configure here.}

[cursor]

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]

🧯 Dangerous deletes guard report

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

Summary:

• Flagged findings (blocking): 0
  0
• Excluded matches (not blocking): 15
• Total matches (all files): 125

Flagged findings (file:line:snippet):
(none)

Excluded matches (by path pattern)

./Dockerfile:34:    rm -rf /var/lib/apt/lists/*
./Dockerfile:76:    rm -rf /var/lib/apt/lists/*
./webapp/static/js/md_preview.bundle.js.map:4:  "sourcesContent": ["// Markdown-it plugin to render GitHub-style task lists; see\n//\n// https://github.com/blog/1375-task-lists-in-gfm-issues-pulls-comments\n// https://github.com/blog/1825-t … [truncated]
./README.md:778:find . -name "__pycache__" -exec rm -rf {} +
./docs/DOCUMENTATION_GUIDE.md:453:rm -rf _build
./docs/Makefile:24:	rm -rf $(BUILDDIR)
./node_modules/mermaid/dist/chunks/mermaid.core/chunk-KS23V3DP.mjs.map:4:  "sourcesContent": ["{\n  \"name\": \"mermaid\",\n  \"version\": \"11.12.0\",\n  \"description\": \"Markdown-ish syntax for generating flowcharts, mindmaps, sequence  … [truncated]
./node_modules/mermaid/dist/chunks/mermaid.esm.min/chunk-4HFYJGYH.mjs.map:4:  "sourcesContent": ["{\n  \"name\": \"mermaid\",\n  \"version\": \"11.12.0\",\n  \"description\": \"Markdown-ish syntax for generating flowcharts, mindmaps, sequen … [truncated]
./node_modules/mermaid/dist/chunks/mermaid.esm.min/chunk-4HFYJGYH.mjs:1:var r={name:"mermaid",version:"11.12.0",description:"Markdown-ish syntax for generating flowcharts, mindmaps, sequence diagrams, class diagrams, gantt charts, git graph … [truncated]
./node_modules/mermaid/dist/chunks/mermaid.esm/chunk-2M32CCKP.mjs.map:4:  "sourcesContent": ["{\n  \"name\": \"mermaid\",\n  \"version\": \"11.12.0\",\n  \"description\": \"Markdown-ish syntax for generating flowcharts, mindmaps, sequence d … [truncated]
./node_modules/mermaid/dist/mermaid.min.js:1524:`,"getStyles"),c1e=RQe});var h1e={};dr(h1e,{diagram:()=>NQe});var NQe,f1e=N(()=>{"use strict";$ge();a1e();l1e();u1e();NQe={parser:Fge,db:n1e,renderer:o1e,styles:c1e}});var m1e,g1e=N(()=>{"use  … [truncated]
./node_modules/mermaid/dist/mermaid.js.map:4:  "sourcesContent": ["/**\n* Default values for dimensions\n*/\nconst defaultIconDimensions = Object.freeze({\n\tleft: 0,\n\ttop: 0,\n\twidth: 16,\n\theight: 16\n});\n/**\n* Default values for tr … [truncated]
./node_modules/mermaid/dist/mermaid.min.js.map:4:  "sourcesContent": ["/**\n* Default values for dimensions\n*/\nconst defaultIconDimensions = Object.freeze({\n\tleft: 0,\n\ttop: 0,\n\twidth: 16,\n\theight: 16\n});\n/**\n* Default values fo … [truncated]
./node_modules/katex/package.json:153:    "build": "rimraf dist/ && mkdirp dist && cp README.md dist && rollup -c --failAfterWarnings && webpack && node update-sri.js package dist/README.md",
./node_modules/katex/src/fonts/Makefile:139:	rm -rf pfa ff otf ttf woff woff2

[github-actions]

⏱️ Performance report

• Run: https://github.com/amirbiron/CodeBot/actions/runs/18904140863
• Artifact: perf-durations

(No performance test durations collected. Mark tests with @pytest.mark.performance.)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!