noam-honig / wikiloc

11 stars 8 forks source link

Major Update to the Project - Fully functional ! #27

Open develad opened 1 year ago

develad commented 1 year ago

https://github.com/noam-honig/wikiloc/pull/25#issuecomment-1492848056

@noam-honig סעיפים 1 ו-2 טופלו. עכשיו רואים Bottom bar וגם הכפתורים יושבים בתוך המסגרת עם התמונות והוקטנו. לגבי ה Favicon לא מצאתי ממש משהו מתאים. ולגבי סעיף 4 אני חושב שכדאי לפתוח על זה Issue

noam-honig commented 1 year ago

סבבה. אבדוק בקרוב

On Sat, Apr 1, 2023 at 21:23 Elad Bar @.***> wrote:

25 (comment)

https://github.com/noam-honig/wikiloc/pull/25#issuecomment-1492848056

סעיפים 1 ו-2 טופלו. עכשיו רואים Bottom bar וגם הכפתורים יושבים בתוך המסגרת עם התמונות והוקטנו. לגבי ה Favicon לא מצאתי ממש משהו מתאים. ולגבי סעיף 4 אני חושב שכדאי לפתוח על זה Issue

— Reply to this email directly, view it on GitHub https://github.com/noam-honig/wikiloc/pull/27#issuecomment-1493067109, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD65PUYKL2ZH35EO6XN534DW7BXARANCNFSM6AAAAAAWPY22MI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

noam-honig commented 1 year ago

בוקר טוב - אוהב את העבודה שאתה עושה. כמה הערות: אוהב מאד את:

  1. הויקיפדיה המסתובב
  2. את העיצוב של הכפתורים.
  3. הראוטים זה נחמד

תקלות ושיפורים:

  1. פחות אוהב את השינוי של הפונט קודם: image עכשיו: image

מעדיף את הקודם.

  1. פחות אוהב את הרווח בין תמונה לתמונה, ואת התצוגה המעוגלת מסביב - העדפתי איך שזה היה קודם (ראה תמונות למעלה) (יכול להיות שזה בגלל שאני ברקע לבן ואז יש לבן בין לבין - אבל עדיין - מעדיף את זה מחובר. (יכול להיות שזה הגיוני בPOPUP במפה אבל פחות ברשימה)
  2. פחות אוהב את המספר "עוד תוצאות" - אני מבין שזה מעין ספירה לאחור לכמה עוד פעמים אפשר ללחוץ על זה, אבל אני חושב שזה מיותר ויכול לבלבל (משתמע שיש עוד 8 תוצאות)
  3. בIPHONE - החץ של לעשות SCROLL למעלה - לא מוצג - היה את זה פעם וסידרתי את זה (כבר לא זוכר איך - אבל עכשיו שכל הCSS זז, זה חזר :) image
  4. כאשר אני במפה ולוחץ על REFRESH - המערכת נתקעת.
  5. כאשר אני עובר למפה - הזום רחוק מידי וצריך לעשות PINCH כדי להכנס יותר טוב.
  6. כאשר יש תקלה בשליפת המיקום בכניסה - (נגיד המשתמש לא הסכים לשתף מיקום) המערכת מציע ללחוץ על עוד תוצאות - עדיף להציג את התקלה, אם יש תקלה. (למרות שממש אהבתי את ההצגה של הכפתור עם הBOUNCE)
  7. כאשר זה לא מוצא תוצאות - בגלל ויקיפדיה, הייתי מפעיל לבד את העוד תוצאות, אין סיבה שהמשתמש לצטרך לעשות את זה :) נסה עם הכתובת הזו: http://localhost:5173/wikiloc/?32.11034511052293,34.685953328449465
  8. כאשר משתמשים במיקום בכתובת - ואז בROUTES זה מאבד את המיקום בכתובת - כך שREFRESH וכו לא עובד.
  9. נראה שהתוצאות נשלפות לפני שאני לוחץ על לחצו כדי לראות מה קורה סביבכם - פחות עדיף, גם מבחינת חוקי UX גוגל מעדיפים ממש שתלחץ על כפתור לפני שמבקשים ממך מיקום - וגם בגלל שהטיפול בשגיאות והעיגול של הטעינה נראים מוזר.
  10. לדאוג שכאשר אנחנו במפה/ברשימה - אם המשתמש עושה REFRESH, נשלוף את הנתונים מחדש (כי יכול להיות שהוא זז) - קודם אם המשתמש היה עושה REFRESH זה היה מביא אותו למסך הבית והוא יכל ללחוץ על הכפתור לקבל מידע עדכני.