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

Cleanup of css+js along with update to latest stable releases of libraries #531

Merged
merged 1 commit into from
Nov 7, 2013

Conversation

stloyd
Copy link
Contributor

@stloyd stloyd commented Nov 3, 2013

Refs #526.

@stloyd
Copy link
Contributor Author

stloyd commented Nov 4, 2013

@pjedrzejewski What do you think?

{{ form_row(form.options, {'attr': {'class': 'input-xlarge'}}) }}
{{ form_row(form.properties, {'attr': {'class': 'input-xlarge'}}) }}
{{ form_row(form.name, {'attr': {'class': 'input-lg'}}) }}
{{ form_row(form.options) }}
Copy link
Contributor

Choose a reason for hiding this comment

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

you forgot the input-lg's here and in the line right below? 👶

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, it's not forgot, this was removed on purpose as it's renders (<select>) ugly with that class.

@stloyd
Copy link
Contributor Author

stloyd commented Nov 7, 2013

@pjedrzejewski Any problem with this?

@pjedrzejewski
Copy link
Member

I restarted the build few mins ago. :)

@stloyd
Copy link
Contributor Author

stloyd commented Nov 7, 2013

@pjedrzejewski It will fail on specs as this was not rebase after fix.

pjedrzejewski pushed a commit that referenced this pull request Nov 7, 2013
Cleanup of css+js along with update to latest stable releases of libraries
@pjedrzejewski pjedrzejewski merged commit 0c13ab4 into Sylius:master Nov 7, 2013
@pjedrzejewski
Copy link
Member

Thanks Joseph!

@stloyd stloyd deleted the feature/css_js_cleanup branch November 7, 2013 16:22
@stloyd stloyd mentioned this pull request Nov 7, 2013
pamil pushed a commit to pamil/Sylius that referenced this pull request May 7, 2019
Cleanup of css+js along with update to latest stable releases of libraries
pamil pushed a commit to pamil/Sylius that referenced this pull request May 7, 2019
Cleanup of css+js along with update to latest stable releases of libraries
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.

3 participants