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

Adds a Sized Button Test #272

Closed
wants to merge 1 commit into from
Closed

Adds a Sized Button Test #272

wants to merge 1 commit into from

Conversation

joemaller
Copy link

This demonstrates the button sizing discrepancy reported in issue #271. Anchor buttons with explicit widths (pure-input-1*) contained in forms are too wide and break out of containing boxes. This seems to be a result of left and right padding being applied on top of the defined width.

The anchor button is too wide:

screen shot 2014-01-12 at 10 45 07 am

I'm putting this in a separate pull request from my proposed solution.

This demonstrates the button sizing discrepancy reported in issue #271. Anchor buttons with explicit widths (`pure-input-1*`) contained in forms are too wide and break out of containing boxes. This seems to be a result of left and right padding being applied on top of the defined width.
@tilomitra
Copy link
Contributor

Hey, thanks for the PR!

We're actually planning on removing pure-input-* from Pure altogether. Folks wanting to control the widths of their input elements should just use the grid classnames. This is simpler and easier to understand.

We're currently doing a bunch of work on making Pure and its website mobile-first. Once this new grids work lands, we will update forms to use the new grid system and remove all pure-input-*.

I'm going to hold off on merging in this PR since these things will be getting removed from core soon.

@tilomitra tilomitra closed this Jan 14, 2014
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.

None yet

2 participants