Skip to content

דפדפן וחיפוש שיפורים - #2920

Merged
amirbiron merged 4 commits into
mainfrom
cursor/-bc-609dcc9e-b192-41af-8f29-e68f26aec82a-425d
Jan 23, 2026
Merged

amirbiron merged 4 commits into
mainfrom
cursor/-bc-609dcc9e-b192-41af-8f29-e68f26aec82a-425d

Conversation

@amirbiron

@amirbiron amirbiron commented Jan 23, 2026 •

Copy link
Copy Markdown
Owner

✨ תיאור קצר

  • תיקוני UI/UX בדפדפן הריפו ובחיפוש הגלובלי: הסרת כפילויות בנתיב הקובץ, מיקום כפתור "היסטוריה" מחדש, החזרת עיצוב כפתור "העתק" ושינוי התנהגות החיפוש הגלובלי ללחיצת Enter בלבד, יחד עם תיקון מיקום תוצאות החיפוש.

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

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

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

  • דפדפן ריפו:
    • הוסתר ה-breadcrumb הכפול והוסר שם הקובץ הכפול מכותרת הקובץ.
    • כפתור "היסטוריה" וכפתור "העתק" הועברו לאותה שורה עם נתיב הקובץ.
    • עיצוב כפתור "העתק" הוחזר לכפתור אייקון קומפקטי.
  • חיפוש גלובלי:
    • החיפוש מופעל כעת רק בלחיצה על מקש Enter, במקום חיפוש אוטומטי בכל הקלדה.
    • מיקום תוצאות החיפוש (הדרופדאון) שונה כך שלא יסתיר את שורת החיפוש.

🧪 בדיקות

  • כל הסעיפים נבדקו ידנית בדפדפן ועובדים כמצופה.
  • Unit
  • Integration
  • Manual

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

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

📝 סוג שינוי

  • fix: תיקון באג
  • docs: שינוי תיעוד בלבד
  • refactor: שינוי קוד ללא שינוי התנהגות

✅ צ'קליסט

  • הקוד עוקב אחרי הסגנון (Black/isort/flake8/mypy)
  • בדיקות רצות ועוברות
  • תיעוד עודכן (README/Docs)
  • אם נוספו ג'ובים חדשים (Background Jobs) – וודא שהם רשומים ב-services/register_jobs.py (כולל Callback/Trigger להפעלה ידנית — למשל callback_name/trigger_func לפי המבנה) כדי שיופיעו בדשבורד
  • אם נוספו/שונו משתני סביבה – עודכן docs/environment-variables.rst וגם services/config_inspector_service.py
  • אם נוספו/השתנו טוקנים – עודכן גם docs/webapp/theming_and_css.rst + FEATURE_SUGGESTIONS/theme_matrix.md
  • אין סודות/מפתחות בקוד
  • אין מחיקות מסוכנות/פעולות על root (ראו .cursorrules)
  • הודעת הקומיט תואמת Conventional Commits (ע"פ הטבלה)
  • CHANGELOG עודכן אם נדרש
  • כל ה‑Required Checks לעיל ירוקים
  • צילום/וידאו UI מצורף אם רלוונטי

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

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

🔗 קישורים

  • Issues קשורים: # (התבסס על פידבק משתמש)
  • Docs Preview:
  • מסמכים/מפרטים רלוונטיים:
  • Branch Protection & PR Rules

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

  • תוכנית חזרה לאחור במקרה תקלה: Revert PR זה.

Open in Cursor Open in Web


Note

Improves repo browser UI and makes global search explicit (Enter to search) with safer async handling.

  • File header UI: Replaces duplicate filename with a single file-path, adds .file-header/.file-header-actions, aligns "history" and compact "copy" icon button in the header, and hides duplicate file-breadcrumb in the header area.
  • Search UX/logic: Adds performRepoSearch with AbortController to prevent races, tracks searchLastQuery, and triggers search on Enter (not on every keystroke); Escape hides dropdown; legacy hash search now calls performRepoSearch directly.
  • Dropdown/positioning: Adjusts .search-results-dropdown position and raises search bar z-index; container set to position: relative to avoid overlap.

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

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

cursor Bot commented Jan 23, 2026

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 Jan 23, 2026 •

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): 15
  • Total matches (all files): 125

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

Excluded matches (by path pattern)
./Dockerfile:42:    rm -rf /var/lib/apt/lists/*
./Dockerfile:121:    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:842: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

github-actions Bot commented Jan 23, 2026 •

Copy link
Copy Markdown
Contributor

⏱️ Performance report

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

@amirbiron
amirbiron marked this pull request as ready for review January 23, 2026 22:37
@sentry

sentry Bot commented Jan 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Comment thread webapp/static/js/repo-browser.js
Co-authored-by: amirbiron <amirbiron@gmail.com>
Comment thread webapp/static/js/repo-browser.js Outdated
Comment thread webapp/static/js/repo-browser.js
Co-authored-by: amirbiron <amirbiron@gmail.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment thread webapp/static/js/repo-browser.js
Co-authored-by: amirbiron <amirbiron@gmail.com>
@amirbiron
amirbiron merged commit 9b9fa5d into main Jan 23, 2026
22 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