Skip to content
This repository was archived by the owner on Oct 23, 2025. It is now read-only.

Conversation

@lyqht
Copy link
Contributor

@lyqht lyqht commented Oct 5, 2022

What kind of change does this PR introduce?

Bug fix that closes #40

What is the new behavior?

Updated container horizontal style to display grid instead of flex.

Additional context

With 2 Social Auth (fills the whole row)

image

With 5 Social Auth (fills the whole row)
image

With many social auth spanning more than a row

image

With all social auth

image

P.S. would be nice if this can get the hacktoberfest-accepted label later on

flexDirection: 'row',
margin: '4px 0',
display: 'grid',
gridTemplateColumns: 'repeat(auto-fit, minmax(48px, 1fr))',
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe we should add this to the variables so anyone can change it easily?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh where?

@lyqht lyqht closed this by deleting the head repository Nov 1, 2022
@lyqht lyqht reopened this Nov 11, 2022
@silentworks silentworks merged commit 4a269b9 into supabase-community:main Nov 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Social Auth buttons runs out of container in horizontal layout

3 participants