Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Google icon added to signup view #2861

Merged
merged 1 commit into from
Jun 25, 2018
Merged

Conversation

SidharthBansal
Copy link
Member

Fixes #2852
@jywarren I am trying to make other things as modular as I can. You can approve these views if they seem good to you else suggest changes. You can merge these views once the functionality of #2771 is checked by Soc people. Till then they will be kept under approved.
image

@ghost ghost assigned SidharthBansal Jun 19, 2018
@ghost ghost added the in progress label Jun 19, 2018
@SidharthBansal SidharthBansal changed the title signup view is created Google icon added to signup view Jun 19, 2018
@SidharthBansal SidharthBansal requested review from a team June 19, 2018 18:11
@plotsbot
Copy link
Collaborator

plotsbot commented Jun 19, 2018

1 Message
📖 @SidharthBansal Thank you for your pull request! I’m here to help with some tips and recommendations. Please take a look at the list provided and help us review and accept your contribution! And don’t be discouraged if you see errors – we’re here to help.

Generated by 🚫 Danger

Copy link
Member

@grvsachdeva grvsachdeva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking great @SidharthBansal 🎉

@grvsachdeva
Copy link
Member

Just wondering, how it will look after centering the block? 😅

@SidharthBansal
Copy link
Member Author

We will also have 3 more icons for other 3 providers. Once, we are done with all 4 provider I will centre it. I know left justified looks a little bit odd.

@SidharthBansal SidharthBansal added this to the OAUTH LOGIN milestone Jun 20, 2018
@jywarren jywarren merged commit df7b7ce into publiclab:master Jun 25, 2018
@ghost ghost removed the review-me label Jun 25, 2018
@jywarren
Copy link
Member

Awesome!!!

@SidharthBansal SidharthBansal modified the milestone: OAuth Login Jul 9, 2018
SrinandanPai pushed a commit to SrinandanPai/plots2 that referenced this pull request May 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants