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

Remove customize spacing-horizontal to use default #59

Merged
merged 1 commit into from
Apr 12, 2022

Conversation

tnir
Copy link
Contributor

@tnir tnir commented Apr 10, 2022

Defaults --ifm-spacing-horizontal: 1rem instead of 3rem.

Closes #58

Screenshots after changes

Padding in the main container would be reset as default of Docusaurus/Infima (1rem).

- Mobile Small-screen PC
to-be mobile-1rem pc-1rem

Resources

Signed-off-by: Takuya Noguchi takninnovationresearch@gmail.com

Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
@tnir
Copy link
Contributor Author

tnir commented Apr 12, 2022

@lisasy What do you think?

Copy link
Contributor

@Zhao-Andy Zhao-Andy left a comment

Choose a reason for hiding this comment

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

@tnir Thanks for the contribution! I think this looks better than before since we have a large header in the intro.

@Zhao-Andy Zhao-Andy merged commit b22753e into forem:main Apr 12, 2022
@tnir tnir deleted the 58-default-ifm-spacing-horizontal branch April 12, 2022 23:22
@tnir
Copy link
Contributor Author

tnir commented Apr 12, 2022

Thanks!

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.

Too wide padding in the main container
2 participants