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

Moved image thumbnail style rules to Framework #1250

Merged
merged 1 commit into from Aug 2, 2015

Conversation

jonom
Copy link
Contributor

@jonom jonom commented Jul 29, 2015

for consistent styling of image thumbnails in GridFields. This is a companion fix for silverstripe/silverstripe-framework#4441

for consistent styling of image thumbnails in GridFields
@jonom jonom force-pushed the choose-file-thumbnails-3.2 branch from f540da6 to 7041a43 Compare July 30, 2015 23:34
@jonom
Copy link
Contributor Author

jonom commented Jul 31, 2015

@tractorcow this is the CMS part of silverstripe/silverstripe-framework#4441 - would you be able to help me understand why some tests are failing? I had a look but it doesn't seem to be related to what I did in this PR.

@tractorcow
Copy link
Contributor

Hi @jonom sorry something in framework must have broken the CMS behat tests. I think it's probably something outside of this PR. I'll investigate.

@tractorcow
Copy link
Contributor

I should have merged this when I merged your CMS changes, sorry. I'll do this now and investigate the test failures.

tractorcow pushed a commit that referenced this pull request Aug 2, 2015
Moved image thumbnail style rules to Framework
@tractorcow tractorcow merged commit e45ef93 into silverstripe:3.2 Aug 2, 2015
@tractorcow
Copy link
Contributor

Actually this is likely fixed by silverstripe/silverstripe-travis-support#19 :D

@jonom
Copy link
Contributor Author

jonom commented Aug 2, 2015

Thanks mate 👍

@jonom jonom deleted the choose-file-thumbnails-3.2 branch August 2, 2015 20:56
@tractorcow
Copy link
Contributor

Rest of the issues seem to be caused by a framework PR missing a CMS pr at #1241

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