[Bug] Navbar disappearance

[yash-parsana]

[Bug] Navbar disappearance

[Yash-Parsana]

This issue is assigned toy you @adarsh1001111

Acceptance criteria:

• Nav bar should be visible to in chat section.
• chat screen should be rendered without any UI issue after login.
• Leaderboard and chat option should be visible to unauthenticated user as well

Testing Scope:

• Create two testing accounts and test chat section by dummy chat. provide screen recording with PR.

Ask doubts on this thread if you have any.

[MishraTanishq619]

'Footer' and 'Navbar' , both dissaperar at "/chat" route.
Becouse this is how code is written.

If no PRs yet, then i would like to contribute to this.
Also, Please state specefically which would be needed at "/chat" route i.e

• Navbar
• Footer
• or Both

[adarsh1001111]

'Footer' and 'Navbar' , both dissaperar at "/chat" route. Becouse this is how code is written.

If no PRs yet, then i would like to contribute to this. Also, Please state specefically which would be needed at "/chat" route i.e

• Navbar
• Footer
• or Both

Thanks for volunteering , since there were no clear instructions in the readme file about setting the project up using vite+react it took a little time but m working on this one ! Thank You

[adarsh1001111]

@Yash-Parsana
I fixed the bug but ,
Can i ask why there is an if block for /chat route specifically in "App.jsx" without headers and footers (in the pic i added them and the page is fixed), maybe some bug/feature which was fixed by doing that.
Also can u specify how to test the app using chat elaborately with setup and files .

[MishraTanishq619]

@Yash-Parsana I fixed the bug but , Can i ask why there is an if block for /chat route specifically in "App.jsx" without headers and footers (in the pic i added them and the page is fixed), maybe some bug/feature which was fixed by doing that. Also can u specify how to test the app using chat elaborately with setup and files .

It would be better if you delete the if-else block , And simply do return statement once.

[adarsh1001111]

@Yash-Parsana I fixed the bug but , Can i ask why there is an if block for /chat route specifically in "App.jsx" without headers and footers (in the pic i added them and the page is fixed), maybe some bug/feature which was fixed by doing that. Also can u specify how to test the app using chat elaborately with setup and files .

It would be better if you delete the if-else block , And simply do return statement once.

I already wrote in my PR that i kept the if else block for the ease to debug any issue created by my edit(cause i was not able to test it) please go through the PR @MishraTanishq619

[Yash-Parsana]

@adarsh1001111 keeping header and footer in chat page same as other pages was causing ui related issues in main chat screen so as temporary fix i had removed them by if condition but it is not good practice so we should fix it now. you have to solve that ui bug in chat screen.
i have provided firbase config file now you can use it to test chat screen. Create real chat scenario by using two dummy users.
and yes we want both header and footer in chat page.
And i am adding steps to run project in readme file.

cc: @MishraTanishq619