Informatory text added - Githubissues

khannakshat7 / Elektra

⚡ML powered Electricity Outage prediction⚡

https://elecktra.uc.r.appspot.com

MIT License

41 stars 84 forks source link

Informatory text added #158

Open shonali2600 opened 3 years ago

shonali2600 commented 3 years ago

Related Issuse

There was confusion while clicking on nav bar other than home, when it ask for login.

Closes: #150

Describe the changes you've made

I've added an informatory text at the top of login page.

Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.

Screenshots

elektr

CodesbyUnnati commented 3 years ago

Please take care of the font of the website and make the text as per the font, also please change the color from red to some other color which matches with the website theme.

shonali2600 commented 3 years ago

elektra @CodesbyUnnati Please check, is it better now?, I've inherited the color to the text and uses same font style as of website

khannakshat7 commented 3 years ago

@shonali2600 This is not implemented correct way as you have to display the info line only if the user clicks on dashboard/map if a user is not logged in. If the user clicks on login/signup he/she should not see the error. And also why did u remove the ReCaptcha from the original code?

CodesbyUnnati commented 3 years ago

@shonali2600 This branch is having conflicts, Please resolve them.

arpit456jain commented 3 years ago

@CodesbyUnnati @khannakshat7 please allow me to a PR for this