jquery-archive / css-chassis

An attempt at creating an open standard for both CSS and JavaScript UI libraries.
https://css-chassis.com
Other
351 stars 67 forks source link

New Component: Inputs #164

Open nashvail opened 8 years ago

jquerybot commented 8 years ago

Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA).

:memo: Please visit http://contribute.jquery.org/CLA/ to sign.

After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know.


If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check.

cvrebert commented 8 years ago
sfrisk commented 8 years ago

I agree with @cvrebert - this is something we had discussed in the meeting about how styling default could break 3rd party items.

sfrisk commented 8 years ago

Is the demos/inputs/inputs.css file necessary? Looks empty. Also is there a reason for choosing to put input in its own folder?

cvrebert commented 8 years ago

Regarding the sizes, perhaps use maps so that users can add additional sizes just by adding entries to the maps?

sfrisk commented 8 years ago

For Reviewing: http://view.css-chassis.com/input-styling/demos/inputs.html

@kristyjy @geekman-rohit @arschmitz

sfrisk commented 7 years ago

You can view these changes at: http://view.css-chassis.com/input-styling/demos/inputs.html

sfrisk commented 7 years ago

Hey your tests are failing. "There is an extra comma following the final element of an array or object literal at scss/variables/inputs.js."

sfrisk commented 7 years ago

Normalize has been pulled in, which should fix some layout issues. Could you please rebase those changes into your develop branch.

nashvail commented 7 years ago

@sfrisk sure!

nashvail commented 7 years ago

I have rebased with the master branch. Please approve that the layout issues are fixed and I will squash and push the commits as one