codeuino / social-platform-donut-frontend

This is an Open Source social Platform where people can interact with Open Source expertise around the globe and work on different projects
207 stars 220 forks source link

[BUG FIX] Update the navigation panel in proposal route #638

Open varunks99 opened 4 years ago

varunks99 commented 4 years ago

This fixes #636.

Bug description

The navigation bar on the left-hand side of the proposal related pages is not properly updated.

Root cause

The 'active' class for the navigation panel is defined by the respective props. The props passed for the proposal component was incorrect/incomplete.

Solution description

I added the state {org:true} in the ProposalEditor component and passed it as props to the navigation panel. Now when clicking on the "propose an idea button", the "Organization" tab is highlighted and active as should be.

Screenshots

Screenshot

@AuraOfDivinity @jaskiratsingh2000

jaskiratsingh2000 commented 4 years ago

hey @AuraOfDivinity and @ksraj123 can we review this up?

varunks99 commented 4 years ago

@AuraOfDivinity

  1. The background color of the navbar is not consistent. We still have the white color appearing over some portions. Please find the cause of this and fix it.

Are you talking about the Jitsi Meet and Logout buttons?

  1. The nav bar should take up the entire width of the navigation panel(Check and compare the implementation of this with the navbar implementation of the dashboard.)

Could you please elaborate on this a little? As I was not able to reproduce this issue.

AuraOfDivinity commented 4 years ago

image

@varunks99 Please have a look at the parts I have highlighted in red and compare it with the nav panel on the following image. They should basically look similar.

image

Notice that the nav panel takes up the entire width and the background color of the navbar is consistent.

varunks99 commented 4 years ago

@AuraOfDivinity, I fixed it now.

Screenshot (238)