ContriHUB / Sheher

Django project to get info regarding tourist places.
1 stars 19 forks source link

Adding gender #72

Closed eleensmathew closed 10 months ago

eleensmathew commented 10 months ago

Added Gender drop down option in register form

Added gender option as dragdown in front end, and saved it to backend. Unfortunately trying to use {% load index %} in profile.html was not getting reflected, so i copy pasted relevant css from index.html and changed the background image.

Profile looks like this right now Screenshot 2023-10-12 002226

Related Issue

Fixes # (https://github.com/ContriHUB/Sheher/issues/61)

Type of change

eleensmathew commented 10 months ago

Also the commit time is shown wrong? Any reasons for this?

kunal2812 commented 10 months ago

Not sure. Probably depends upon your system clock

kunal2812 commented 10 months ago

Also add navbar to profile page. No means to navigate from there

eleensmathew commented 10 months ago

Any suggestions for fixing css of nav bar? Maybe it has something to do with location of scripts file?

kunal2812 commented 10 months ago

Yes, it might be due to css and script files not properly linked for that page

eleensmathew commented 10 months ago

I tried a few ways to fix this but no changes reflected on the page. Do you have any suggestions?

kunal2812 commented 10 months ago

Will check and let you know

kunal2812 commented 10 months ago

@eleensmathew You can remove the navbar and add Home, Report, Log Out Buttons just like on the Report page

eleensmathew commented 10 months ago

I added some extra functionality in navbar_compliants.html. The css was still not getting loaded so I basically tried to hard code the buttons and now it looks like this. Also added an extra status number =3 so the navbar won't get shown twice Screenshot 2023-10-12 233808

eleensmathew commented 10 months ago

Let me know if I should I remove this and do something else.

kunal2812 commented 10 months ago

image

Getting this error when I click on Profile

kunal2812 commented 10 months ago

image

Also mail is not getting displayed properly

kunal2812 commented 10 months ago

@eleensmathew You can remove the navbar and add Home, Report, Log Out Buttons just like on the Report page

Buttons for Home, Report are still not present in the navbar

eleensmathew commented 10 months ago

Okay I'll add it now

eleensmathew commented 10 months ago

Screenshot 2023-10-13 001636 It looks like this now but I'm not sure how to fix the text problem. I'm not good in front end😅. I have fixed the profile link and added home and complaint link.

kunal2812 commented 10 months ago

When you click on those buttons it should not open a new tab. Expected behaviour is to refresh the same page. Also try to fix the text issue. Will add bonus.

eleensmathew commented 10 months ago

Sure will try to fix it.

On Fri, 13 Oct 2023 at 01:54 Kunal Katiyar @.***> wrote:

When you click on those buttons it should not open a new tab. Expected behaviour is to refresh the same page. Also try to fix the text issue. Will add bonus.

— Reply to this email directly, view it on GitHub https://github.com/ContriHUB/Sheher/pull/72#issuecomment-1760317329, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWAG2HAOEBUI3GD64OKYAWDX7BGWXANCNFSM6AAAAAA54OVIYY . You are receiving this because you were mentioned.Message ID: @.***>

-- Eleena

eleensmathew commented 10 months ago

HI @kunal2812 I fixed the links to open on the same page and got the long email to be broken into the next line. As a bonus i also changed the background color of edit_profile.html because nav bar text and background colour was same and also centered the hedaing. It currently looks like this Screenshot 2023-10-13 150145 Screenshot 2023-10-13 150212