Skip to content

Comments

Informatory text added#158

Open
shonali2600 wants to merge 6 commits intokhannakshat7:masterfrom
shonali2600:infoText
Open

Informatory text added#158
shonali2600 wants to merge 6 commits intokhannakshat7:masterfrom
shonali2600:infoText

Conversation

@shonali2600
Copy link
Contributor

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:

  • My code follows the style guidelines of this project.
  • 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.
  • My changes generate no new warnings.

Screenshots

elektr

@CodesbyUnnati
Copy link
Collaborator

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
Copy link
Contributor Author

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

@shonali2600 shonali2600 mentioned this pull request Apr 18, 2021
5 tasks
@khannakshat7
Copy link
Owner

@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
Copy link
Collaborator

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

@arpit456jain
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding informatory text at the top of some sections

4 participants