Skip to content

Improve UI for delete confirmation and document loading - #24

Merged
Amlaach merged 8 commits into
TypesetOK:mainfrom
YSmauas:main
Oct 30, 2026
Merged

Improve UI for delete confirmation and document loading#24
Amlaach merged 8 commits into
TypesetOK:mainfrom
YSmauas:main

Conversation

@YSmauas

@YSmauas YSmauas commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

סיכום התיקונים

שבקובץ תיקונים-חשובים-לתוכנת-עימוד.md לפי קלוד.

כבר טופל (לא נספרים): 4, 8, 9, 29, 31, 35, 37. סעיפים 32-34 תוקנו על ידי המפתח הראשי, וממתינים לאימות ידני שלך על מסמך prose. אותו אימות ידני כדאי גם לסעיף 37.

קטנים וכנראה קלים (טרם בדקתי בקוד):

  • 30: שורות בתצוגה שלא מסונכרנות עם הטקסט. ייתכן שזה כבר נפתר בתיקון ה-main, כדאי לבדוק קודם.
  • 38: בחירת תזרים אפשרית רק בייבוא ובפתיחה, ובמסמך פתוח רק גמרא.
  • 39: תמיכה בניקוד מיוחד (קמץ קטן, שווא נע).
  • 1, 2, 3: רקע תצוגת העמודים, כפתור הייצוא, ושדרוג עורך הטקסט.
  • 16, 17: מדריך השימוש ומסך האודות. סעיף 17 תלוי בהחלטה על משיכת התורמים מגיטהאב בזמן בנייה.

בינוניים:

  • 5: באג המתקין באנגלית (כיווניות LTR מול תוכנה בעברית).
  • 6: יבוא כלל סוגי הקבצים.
  • 7: המשך הנגשת הניקוי בלשון פשוטה.
  • 10: שינוי מסך הבית, עם אלגוריתם לזיהוי תבנית.
  • 11: עוד תבניות עמוד.
  • 12: קישורים להורדת פונטים בהגדרות.

גדולים ורגישים, עדיף PR נפרד לכל אחד:

  • 13, 14, 15: אבטחת תוספים ומצב סייפר, כולל נעילת הגדרות בסיסמה. אלה נושאי אבטחה, ולכן כדאי לתכנן בזהירות ולבדוק היטב.
  • 36: תזרים גמרא בצורת הדף המסורתית. הוא תלוי בעבודה הארכיטקטונית בקובץ הנפרד.

לא דחופים (18-28): מחשבון שדרה, הזזת ניקוד, ספריית עיטורים, "גביע", סוגריים חכמים, PDF להגהה, תמונות מצב, Linter לעימוד, כותרות רצות, תוכן עניינים, ומה חדש.

@YSmauas

YSmauas commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

הבדיקה שנכשלה לא קשורה לכאורה לתיקונים שביצעתי.
זאת שגיאה ביצוא PDF שככל הנראה הייתה רדומה ואני רק עוררתי אותה.
שווה בדיקה

@YSmauas

YSmauas commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

הכל עובר!!

@YSmauas

YSmauas commented Oct 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

@Amlaach הכל עובר, בקומיט האחרון תוקן גם בחירת התזרימים (תיקון 38)

@YSmauas

YSmauas commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

תוקן באג 5 שבוחרים שפת מתקין מעכשיו זה משפיע על התוכנה

@YSmauas

YSmauas commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

בקומיט הכל עובר אבל כאן זה נכשל?

@YSmauas

YSmauas commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

בכ״א הבאג היה ששינוי תיקיית הפרוייקטים במתקין לא השפיעה על התוכנה עצמה.
ובנוגע לשגיאה לטענת קלוד אין בעיה בקוד שלי אלא אולי בבדיקה עצמה יש באג...

@Amlaach

Amlaach commented Oct 30, 2026

Copy link
Copy Markdown
Member

אני אטפל בזה.

@YSmauas

YSmauas commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

@Amlaach תבדוק אם אפשר להחליף תזרים לאחר שכבר נבחר תזרים או שאפשר לבחור רק בפעם הראשונה וזהו.

@YSmauas

YSmauas commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

אני עכשיו עובד על אפשרות להחליף תבנית לאחר שכבר נבחר

@Amlaach

Amlaach commented Oct 30, 2026

Copy link
Copy Markdown
Member

@Amlaachאם אפשר להחליף תזרים לאחר שנבדקו תזרים או אפשר לבחור רק בפעם הראשונה וזהו.

בדקתי וזה אפשרי כל הזמן.
כך לפי הקוד.

@YSmauas
feat: add ability to change book layout template
209d395 
- Added a "Change book layout" option to the StructureBar.
- Implemented `changeDocumentTemplate` to allow users to switch templates on an open document.
- Added i18n support and updated `ContextualInspector` to ensure UI reactivity during template updates.
@YSmauas

YSmauas commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

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

מה שנוסף ואיך זה עובד:
כפתור עדין בלשונית התזרימים (StructureBar):
בראש פאנל התזרימים (tok-panel-head), לצד הכותרת "תזרימים", נוסף כפתור קומפקטי עם אייקון ספר ותווית נגישות: "החלפת תבנית ספר…" (Change book layout…).
לחיצה עליו פותחת את דיאלוג בחירת התבנית המוכר (ImportTemplateModal), המציג את 5 סוגי הספרים (דף גמרא, ספר רציף, ספר עם הערות, מקראות גדולות, עלון).
פקודה ישירה ב-Command Palette (Ctrl+K):
נוספה הפקודה "החלפת תבנית ספר…" תחת קטגוריית "עמודים" (Pages), המאפשרת לבצע את ההחלפה ישירות מהמקלדת.
שמירה מושלמת על כל הטקסט (No Data Loss):
הפונקציה changeDocumentTemplate מעדכנת את documentState.templateType.
כל הפסקאות והתזרימים שהיו במסמך נשמרים במלואם. אם התבנית החדשה דורשת תזרימים חדשים (כגון מעבר מספר רציף לגמרא), הם נוצרים כריקים ומוכנים לקליטת תוכן.
סרגל המבנה השמאלי מתעדכן לרשימת התזרימים הרלוונטית (structureBar.setFlows()).
עורך הטקסט נשאר מסונכרן על התזרים הנבחר.
אם המסמך נפתח מקובץ מקור (כמו Word או Markdown), הבחירה החדשה נשמרת בזיכרון (rememberImportTemplate) כדי שבפתיחה הבאה של אותו קובץ הוא ייפתח ישירות בתבנית החדשה.
מוצג טוסט מעודן: "תבנית הספר הוחלפה ל: {שם התבנית}".
אימות ובדיקות:
תרגומים מלאים בעברית ובאנגלית נוספו ב-i18n.ts.
ה-TypeScript וה-Bundle קומפלו בהצלחה מלאה (0 errors).
נוספה בדיקת יחידה ייעודית ב-test/frontend.test.mjs המאמתת את קיום הכפתור, החיבור בסרגל המבנה והפקודה ב-Command Palette.

@Amlaach
Amlaach merged commit 0ba372e into TypesetOK:main Oct 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@YSmauas @Amlaach