Give a new look to sidebar

[Palakkgoyal]

This PR closes #1141
Updated the UI of sidebar and made it responsive.

[github-actions]

Thank you Palakkgoyal! for creating this pull request and contributing to Dummygram! 💗

The maintainers will review this Pull Request and provide feedback as soon as possible! 😇
We appreciate your patience and contribution, Keep up the great work! 😀

[narayan954]

Hi @Palakkgoyal , is this the expected behaviour of the sidebar??

it seems a bit off, like the circle around profile card,3 dots on top misaligned... Can you please take a look into that?

[Palakkgoyal]

Hi @Palakkgoyal , is this the expected behaviour of the sidebar?? it seems a bit off, like the circle around profile card,3 dots on top misaligned... Can you please take a look into that?

Its definitely not the desired behavior. I will remove that active state. And 3 dots misaligned? You mean that I should maintain a fix gap between those 3 dots and the other items?

[narayan954]

Hi @Palakkgoyal , is this the expected behaviour of the sidebar?? it seems a bit off, like the circle around profile card,3 dots on top misaligned... Can you please take a look into that?

Its definitely not the desired behavior. I will remove that active state. And 3 dots misaligned? You mean that I should maintain a fix gap between those 3 dots and the other items?

alright, by misaligned I meant, there's a bigger left margin in other sidebar icons that those 3 dots, so they're not vertically aligned. Also, can we move that profile card to the top similar to this image, perhaps if it looks nice

[Palakkgoyal]

Hey, I did some changes. Can you please check.

[narayan954]

Hey, I did some changes. Can you please check.

sure, I am thinking to merge the refactoring pr first so there're very less merge conflicts

[narayan954]

Hi @Palakkgoyal , can you please fix these conflicts?

[Palakkgoyal]

Yes, sure.

[Palakkgoyal]

I have solved the conflict. Can you please check.

[narayan954]

I have solved the conflict. Can you please check.

Hi @Palakkgoyal I just found this css conflicts with comment box css as well, can you please fix?

[Palakkgoyal]

I have solved the conflict. Can you please check.

Hi @Palakkgoyal I just found this css conflicts with comment box css as well, can you please fix?

Fixed

[narayan954]

I have solved the conflict. Can you please check.

Hi @Palakkgoyal I just found this css conflicts with comment box css as well, can you please fix?

Fixed

when we click on view profile, some one else's profile opens... also, that box is overtaken by sidebar in terms of z index.
Also,profile picture is square now and the box opens very lower than expected. Can you check if you're facing these as well now? I think earlier I didn't face that.

[Palakkgoyal]

I will fix others. But, I checked for this Aditya Roy's profile. I don't know why everytime we are getting there. I thought its happening with me only. I checked for user.uid and I think I checked for everything but still going to this profile. Even I am working on other PR. There also I am getting navigated(sometimes) to this one

[Palakkgoyal]

Hey I checked just now that even on the deployed website we are reaching that profile only.

[Palakkgoyal]

Hey hey. I get it there is problem with the way we are fetching data.
This function:
const docRef = db.collection("users", user?.uid);
const docSnap = await docRef.get();

---

You will think that this will return only one doc but, guess what it is returning an array of 87 docs(Perhaps all user docs). Let me check what is wrong.

[Palakkgoyal]

Hey I fixed that in my centralize data PR. Check it out.
const docRef = doc(db, "users", user?.uid);
const docSnap = await getDoc(docRef);

[Palakkgoyal]

Hey I have fixed the issues. The dropdown alignment issue was due to routing PR

[narayan954]

Hey hey. I get it there is problem with the way we are fetching data. This function: const docRef = db.collection("users", user?.uid); const docSnap = await docRef.get();

You will think that this will return only one doc but, guess what it is returning an array of 87 docs(Perhaps all user docs). Let me check what is wrong.

thanks a ton for finding that error!

[narayan954]

Hey I checked just now that even on the deployed website we are reaching that profile only.

I didn't check that, sorry

[narayan954]

looks good to me!

[narayan954]

Thanks for your contribution @Palakkgoyal :)

[Palakkgoyal]

Hey I checked just now that even on the deployed website we are reaching that profile only.

I didn't check that, sorry

No worries.