Major Update to the Project - Fully functional !

[develad]

19

Hey everyone :) I've made some changes in all of the project structure

The major updates were:

1. Integrating a Context to replace the utils file -> No more prop-drilling
2. Creating some logic structure between folders and files
3. Lots of styling
4. Adding routing system with 'react-router-dom'
5. Much more that I can't recall at the moment...

Known issues

1. There is NO ADD MORE RESULTS functionality
2. There is bug when trying to ADD the English results to the Hebrew results after the user clicked
3. Type annotations are not the best (Typescript isn't my best skill 😂)
4. Continuing working on the style and adding new features

[develad]

25

There is NO ADD MORE RESULTS functionality --> Fixed! There is bug when trying to ADD the English results to the Hebrew results after the user clicked --> Fixed!

[noam-honig]

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

בגדול אני מאמין שפחות כפתורים זה יותר טוב - יש מאמר מעולה של ג'ואל על זה: indo

[develad]

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

בגדול אני מאמין שפחות כפתורים זה יותר טוב - יש מאמר מעולה של ג'ואל על זה: indo

Totally agree on that one... I think I'll combine the 2 buttons into minimalistic one

[develad]

@noam-honig

Thoughts?

[noam-honig]

פחות טקסט :)

מספיק עוד תוצאות.

אל תשכח שזה בטלפון ושזה מסתיר מידע שאני רוצה לראות.

On Tue, Mar 28, 2023 at 11:55 Elad Bar @.***> wrote:

@noam-honig https://github.com/noam-honig

[image: localhost_5173_wikiloc_list(Samsung Galaxy A51_71)] https://user-images.githubusercontent.com/70572668/228183671-262be2a7-1c34-4302-b057-693f6f8af019.png

Thoughts?

— Reply to this email directly, view it on GitHub https://github.com/noam-honig/wikiloc/pull/25#issuecomment-1486470320, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD65PU7SGVGBL5OGNDRKUVDW6KRQTANCNFSM6AAAAAAWIEYY4M . You are receiving this because you were mentioned.Message ID: @.***>

[noam-honig]

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

On Tue, Mar 28, 2023 at 12:57 noam honig @.***> wrote:

פחות טקסט :)

מספיק עוד תוצאות.

אל תשכח שזה בטלפון ושזה מסתיר מידע שאני רוצה לראות.

On Tue, Mar 28, 2023 at 11:55 Elad Bar @.***> wrote:

@noam-honig https://github.com/noam-honig

[image: localhost_5173_wikiloc_list(Samsung Galaxy A51_71)] https://user-images.githubusercontent.com/70572668/228183671-262be2a7-1c34-4302-b057-693f6f8af019.png

Thoughts?

— Reply to this email directly, view it on GitHub https://github.com/noam-honig/wikiloc/pull/25#issuecomment-1486470320, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD65PU7SGVGBL5OGNDRKUVDW6KRQTANCNFSM6AAAAAAWIEYY4M . You are receiving this because you were mentioned.Message ID: @.***>

[develad]

@noam-honig

1. No problem. I'll shorten the text
2. What icon would you think is a good fit? Or maybe remove the icon altogether? (You can choose the icon from here)

[develad]

New design for the buttons...

Note that on mobile devices the buttons looks much more smaller

[noam-honig]

@develad קודם כל, כל הכבוד על העבודה. יש לי כמה הערות:

1. הBOTTOM BAR עם הקוד, הלייקים וכו... לא מופיע בדף הראשון - אלא כאשר נכנסים. הייתי רוצה שיופיע גם שם.

2. הBOTTOM BAR עם המפה והכל - במחשב, חורג מהרוחב הצר של התמונות -הייתי מעדיף שהוא ישאר בתוך התחום הצר הזה - ובמסך רחב הם גם גדולים מידי:

3. אם אתה כבר שם, בא לך להחליף את הFAVICON ולבחור ICON שיתאים למערכת?

4. ואם ממש בא לך, הרבה אנשים ביקשו שאפשר יהיה להוסיף את זה למסך הבית בטלפון. באנדרואיד אפשר לעשות את זה מהקוד - ראה מאמר: https://developer.mozilla.org/en-US/docs/Web/Progressive_web_apps/Add_to_home_screen

נראה לי שאנשים יעופו על זה.