בירור | שרשור תגובת לתוסף להצגת ספר בשני טורים

[חד צורבא]

היום נזדמן לידי מר קלוד וניסיתי לשפר ולהוסיף פונקציות על הגירסא של @ u88 , הנה התוצאות והסבר שלו

מהדורה ראשונה משיחה אחרת טרומית
תצוגת_טורים.otzplugin

Spoiler

עשיתי את כל 7 השינויים בקובץ. הנה מה שהשתנה:

זום גדול יותר – הגבול המקסימלי הוגדל מ-1.5 ל-3.5, כך שבתצוגת ספר אפשר להגדיל עד שהעמודים ימלאו את כל המסך.
חיצי ניווט שקופים – ברירת המחדל עכשיו כמעט שקופה (opacity 0.12), ובריחוף הם מופיעים במלואם. בנוסף הוספתי ניווט מקלדת: חץ שמאלה = דף הבא, חץ ימינה = דף קודם (פעיל רק בתצוגת ספר).
הבעיה בטעינה בתצוגת ספר – מצאתי את הבאג: בכל אצווה של טעינה (כל ~70 צמתים) הפונקציה שמוסיפה עמודים לתצוגת הספר הייתה סורקת מחדש את כל העלים בספר ומעדכנת את הסגנון של כולם. בספר ארוך כמו כריתות (עם כל הפרשנים) זו התנהגות ריבועית (O(n²)) שהולכת ומאטה ככל שנטענים יותר עמודים, ונראית כמו תקיעה/לולאה אינסופית. תיקנתי את זה כך שכל עלה חדש מתעדכן פעם אחת בלבד, בלי לסרוק את כל מה שכבר נטען. חשוב לציין: לא הצלחתי להריץ את התוסף בפועל בתוך אוצריא כדי לוודא ב-100% שזו הייתה הסיבה היחידה, אז אשמח לשמוע אם זה פתר את הבעיה או שנשאר עוד משהו.
הודעת ההתקדמות האדומה – עכשיו כבויה כברירת מחדל, עם מתג חדש בהגדרות ("הצג הודעת התקדמות בעת טעינה/סידור עמודים") למי שכן רוצה אותה בחזרה.
שמירת מיקום יחסי – שינוי הגדרות (גופן, שוליים, מרווחים, טורים) שומר עכשיו את המיקום היחסי בספר (אחוז גלילה, או העלה היחסי בתצוגת ספר) ומשחזר אותו אחרי הסידור מחדש, במקום לקפוץ להתחלה.
גופנים – שיניתי את שרשרת הגיבוי כך שאם הגופן המבוקש (למשל Frank Ruhl Libre / Shofar) לא נטען, הדפדפן יפול ל-Rubik ואז ל-Segoe UI לפני ברירת המחדל הכללית. חשוב להבהיר: המשיכה בפועל של קבצי הגופנים מתבצעת ע"י אוצריא עצמו (מחוץ לקובץ הזה), כך שאם הבעיה היא שהגופן לא נמשך בכלל מהאתר, זה לא משהו שאני יכול לתקן מכאן – אבל עכשיו לפחות הנפילה תהיה לגופן קריא ולא ל-serif גנרי.
מצב קריאה במסך מלא – F11 מסתיר גם את סרגל הטאבים וגם את הכלים העליונים (ומנסה להיכנס למסך מלא אמיתי של הדפדפן), ו-ESC (או F11 שוב) יוצא ממנו.

מהדורה ראשונה
תצוגת-טורים.otzplugin
מה התבצע ?

Spoiler

עדכנתי את הקובץ וטיפלתי בכל 14 הנקודות:

1. זום – הגבלת המקסימום הועלתה ל-4.0 (במקום 1.5) כך שאפשר למלא את המסך, ונוספו קיצורי Ctrl+‎/‎Ctrl-‎.
2. חצי ניווט שקופים – opacity ברירת מחדל 0, ובריחוף עולה ל-0.85 כמו קודם.
3. ניווט מקלדת – חץ ימינה = לכיוון תחילת הספר, שמאלה = לכיוון סופו, גם בתצוגת ספר וגם בגלילה רגילה.
4. Ctrl+W / Ctrl+Shift+W – סוגר טאב פעיל / כל הטאבים.
5. תיקון הלולאה האינסופית – נוסף זיהוי "צ'אנק חלקי" (קצר מה-limit) כסימן לסוף הספר, ובלם ביטחון נוסף למקרה שה-offset נתקע.
6. תגיות שבורות – פונקציית sanitizeFragment חדשה מזהה רק תגיות HTML שלמות ותקינות; כל שבר תג (בלי סוגר/פותח) מוסר לגמרי ולא מוצג כטקסט.
7. הודעת הפרוגרס האדומה – מוסתרת לצמיתות.
8. שימור מיקום בהגדרות – נוספה מנגנון capture/restore של מיקום (עמוד/עלה) סביב כל סידור-מחדש; אין משיכה חוזרת מהתוכנה כי תמיד נעשה שימוש ב-rawContent השמור.
9. גופן – ברירת המחדל שונתה ל-Segoe UI, והגופן המדויק מהתוכנה (theme.typography) ממשיך לחול כרגיל.
10. תיקון "הגופן גדל מאליו" – זו הייתה תקלת חוסר-התאמת יחידות (px מול pt) בין ברירת המחדל הישנה (25px) לשאר המערכת שעבדה ב-pt. אוחדו כל היחידות ל-pt, כולל המרה נכונה כשמגיע ערך מהתוכנה.
11. ניקוד/טעמים/פיסוק – נוספו שני מתגים בהגדרות (הסתרת ניקוד וטעמים, הסתרת ‎?!:-_*‎) שפועלים על התוכן בזמן הפיצוול בלי לפגוע ב-rawContent המקורי. (סנכרון אוטומטי לפי הגדרת התוכנה עצמה תלוי בחשיפת ה-API הזה מ-Otzaria - כרגע לא קיים, אז המתגים הידניים הם הפתרון המעשי).
12. מרווח שורות אחיד – נוספה התאמת CSS ל-sup/sub (מספרי הערות) כך שלא ינפחו את גובה השורה.
13. מסך מלא F11/Esc – נוסף כפתור וקיצור מקלדת.
14. התאמה לרוחב/גובה – נוספו שני כפתורים (fitToScreen) שמחשבים זום אוטומטי לפי גודל האזור הזמין.

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

מהדורה שניה
תצוגת - טורים.otzplugin
מה התבצע ?

Spoiler

מצאתי ותיקנתי עוד כמה באגים אמיתיים:

באג חמור בסגירת טאבים – כשסוגרים טאב שנמצא לפני הטאב הפעיל (למשל סוגרים טאב 1 כשטאב 3 פעיל), האינדקס לא היה מתעדכן נכון והמערכת הייתה קופצת לטאב הלא-נכון. תיקנתי את הלוגיקה כך שהיא מבחינה בין שלושה מקרים: הטאב שנסגר הוא עצמו הפעיל / הוא לפני הפעיל / הוא אחריו.

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

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

עמידות בפני שגיאות רשת במשיכת התוכן – קריאה בודדת שנכשלת (טיימאאוט, שגיאת API) הייתה מפילה את כל התהליך ומאבדת את כל מה שכבר נטען. עכשיו יש ניסיון חוזר (עד 3 ניסיונות) לכל צ'אנק בנפרד, כך שכשל חד-פעמי לא יקטע את טעינת הספר.

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

וזאת למודעי

לצערי הוא לא באמת הצליח לתקן את כל מה שהוא הצהיר ואשר על כן צריך מישהו שימשיך מכאן (יש לי עוד 5 באגים לפחות חוץ ממה שאמרתי לו והוא לא תיקן באמת אבל נגמר לי לכמה שעות ואני לא יודע אם זה יחזור)

יאכלו ענוים  וישבעו יהללו ה' דורשיו יחי לבבכם לעד

[חד צורבא]

שמועה טובה תדשן עצם

התוסף הזה ישפר את חויית הלימוד שלכם אלף רמות קדימה !!!

בסוף התברר לי שחשבוני לא בוטל ועל כן עשיתי עוד כמה שיפורים ועכשיו התוסף מוצלח ביותר (בתצוגת עמוד יש קצת בעיה עם המעבר בין דפים)
@יאיר-דניאל עכשיו כבר אפשר להעלות את זה לחנות (זה עדיין מוגדר כגירסא של @u88 ואותו מספר גירסא (אני רק מודיע, אם חייב לשנות את זה כי אין לי מושג מה צריך לעשות)

מהדורה שלישית

תצוגת ־ טורים.otzplugin
ההסבר של מר קלוד

Spoiler

1. מסך מלא – הסרתי לגמרי את התחבולה של position: fixed שהחזיקה הכל "מתוח" על המסך כל הזמן. עכשיו זה פשוט: ה-Fullscreen API האמיתי (F11) עושה את המילוי בפועל של המסך כרגיל, וה"תכונה" היחידה שהתוסף מוסיף היא הסתרת שורת הטאבים בזמן מסך מלא, כדי לפנות עוד קצת מקום. כשיוצאים (F11/Esc) שורת הטאבים חוזרת.

2. חיצים תמיד קדימה/אחורה בתצוגת עמוד בודד – זה היה באג אמיתי: שינוי זום (כולל התאמה לרוחב/לגובה) לא שימר את העמוד הנוכחי. כשה-scrollTop (במספרי פיקסלים) נשאר זהה אבל גובה כל עמוד השתנה בעקבות הזום, אותו מיקום גלילה "הצביע" פתאום על עמוד אחר לגמרי - ולכן זה תמיד נראה כמו קפיצה לכיוון קבוע. עכשיו נשמר ומשוחזר העמוד המדויק סביב כל שינוי זום.

3. הסתרת פיסוק – מקף (-) מוחלף עכשיו ברווח (לא נמחק) כדי שמילים לא ידבקו; שאר הסימנים (?!:_*) נמחקים כרגיל. בכותרות (h1-h6) הנקודתיים : נשארות תמיד, ורק שאר הסימנים מוסתרים שם. הלוגיקה עכשיו מבוססת DOM (לא regex גולמי על מחרוזת), כדי להבחין נכון בין כותרת לגוף טקסט.

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

5. סרגל כלים מצומצם – נוסף מתג חדש בהגדרות (בקטגוריית "הגדרות פריסה") בשם "סרגל כלים מצומצם". כשמפעילים אותו, ה-padding של הסרגל העליון, גובה שורת הטאבים, וגודל האייקונים מוקטנים במעט - משפיע מיידית בלי צורך בסידור מחדש של הדפים.

6. תגיות שבורות/לא סגורות – זה היה השיפור המהותי ביותר: קודם הניקוי היה פר-שורה, כך שתגית לא סגורה יכלה "לדלוף" על פני הרבה משפטים/פסקאות אם התוכן לא היה מפוצל בול לפי שורות. עכשיו יש מאזן תגיות אמיתי שרץ על כל תוכן הספר בבת אחת: כל תג פתיחה (למשל <b>) שאין לו סגירה תואמת בשום מקום, וכל תג סגירה יתום (</b> בלי פתיחה), מוסרים לגמרי - לא מוצגים כטקסט וגם לא משפיעים על הטקסט שאחריהם. בדקתי את זה בנפרד עם כמה תרחישי קיצון (תגית לא סגורה על פני כמה שורות, קינון שבור כמו <i><b>טקסט</i></b>, תגית קטומה בסוף) וכולם עובדים נכון.

בנוסף, בעקבות התיקון של #6, גם שיפרתי את איחוד הצ'אנקים שמגיעים מהרשת - עכשיו הם מחוברים ברצף לפני כל עיבוד, כך שתגית שנחתכה בול על גבול בקשת רשת (5000 תווים) מתאחדת מחדש לתגית שלמה במקום להיחשב "שבורה" רק בגלל איפה שהחיתוך הטכני קרה.

אשמח לשמוע הערות!!!

(לא מחייב שאמשיך לטפל)

טעימות:

Spoiler

[eli8565]

@חד-צורבא אלוףףףףףף!!!!
זה כ"כ טוב שצריך לשכנע את @י.-פל. להכניס את זה לתוכנה עצמה, אבל נראה לי שנתן לו לסיים את 996 בשקט וזה ייעלץ לחכות ל997 [אהבתי גם את האופציות של התאמה לאורך/רוחב העמוד]

[חד צורבא]

@י.-פל. אני רוצה שיוכלו לפתוח ספרים מתוך התוסף בשורת החיפוש מה אני צריך להראות למר קלוד ? איזה API ודומיו איך ווייס (אנא עארף כלשונו של אחד האחראים)
גם רציתי שידע למשוך את הגופן שהתוכנה מציגה באותו רגע איך אני עושה את זה ?

[י. פל.]

@חד-צורבא כתב:

@י.-פל. אני רוצה שיוכלו לפתוח ספרים מתוך התוסף בשורת החיפוש מה אני צריך להראות למר קלוד ? איזה API ודומיו איך ווייס (אנא עארף כלשונו של אחד האחראים)
גם רציתי שידע למשוך את הגופן שהתוכנה מציגה באותו רגע איך אני עושה את זה ?

הכל פה:
https://github.com/Otzaria/otzaria/tree/dev/docs/plugin-sdk

[חד צורבא]

@י.-פל. אני צריך להביא לו את API_REFERENCE.md ?
חוץ מזה עוד משהו ?

[נתנאל_26]

@חד-צורבא
תן לAI אחר וזול לחלץ לך את API הרלוונטיים חובל לגמור את הטוקנים על משהו כזה פשוט

[חד צורבא]

@נתנאל_26 מה זאת אומרת לחלץ מה אני אמור לומר לג'מיני נניח ?

[נתנאל_26]

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

[חד צורבא]

@נתנאל_26 מצויין יש הכל, איך אני מגיש את זה לקלוד ? עושה העתק הדבק של כולם לתוך קובץ טקסט ומעלה לו בתוך הצ'אט או בריבוע מצד ימין ?

[נתנאל_26]

@חד-צורבא תעתיק לצאט למה בקובץ טקסט?

[יאיר דניאל]

@חד-צורבא כתב:

זה עדיין מוגדר כגירסא של @u88 ואותו מספר גירסא

חייב לעלות מספר גירסה - תשנה, ואני אעלה לחנות.
נראה ממש יפה!

[חד צורבא]

@יאיר-דניאל

חד-צורבא כתב:

(אני רק מודיע, אם חייב לשנות את זה כי אין לי מושג מה צריך לעשות)

[יאיר דניאל]

@חד-צורבא מה הפירוש?
תגיד לקלוד לארוז מחדש/תארוז אתה מחדש ותשנה את מספר הגירסה בעוד 0.0.1

[חד צורבא]

איך משנים ? איפה משנים ? מה זה נקרא לארוז? להפוך לאוצריאפלאגין ?

[נתנאל_26]

@חד-צורבא

פה

[חד צורבא]

@נתנאל_26 הוא אמר שיש אפשרות לבד
נגמר לי קלוד להיום

[יאיר דניאל]

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

[חד צורבא]

@יאיר-דניאל להחליף שם מספר וזהו ?

[יאיר דניאל]

@חד-צורבא כן, לעלות את המספר - לדוגמא היה כתוב 0.4.3 = תכתוב 0.4.4