Dear Maintainers,
Can you help with getting views responsive grid in drupal 8 core?
Please see this issue : http://drupal.org/node/1903746
Thanks!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

iwhitcomb’s picture

Status: Active » Fixed

Yeah, that'd be great.

derhasi’s picture

Status: Fixed » Needs work

As suggested in the core issue, I'd try to port this module to D8 and then look, what parts we can suggest for core.

derhasi’s picture

Status: Needs work » Needs review
FileSize
25.01 KB

There's a first patch that seems to work fine on my local drupal 8 dev.

@Ian, I've split up the whole patch in several commits in my local repos, so maybe (if you like) I'd open a sandbox or may get access to this module's repo, so I can directly push the history.

derhasi’s picture

btw: there currently is a core bug that prevents relying on views theme handling: #1911492: Views try to find Custom StylePlugin template in core/modules/views/templates. Therefore the hook_theme_registry_alter() had to be implemented.

mgifford’s picture

Patch applies nicely. Haven't been able to test the module though as Core is failing on me right now.

mgifford’s picture

Ok, I finally remembered to nix the /c directory that is now getting created in D8. Then went to re-install and everything went fine.

After installing the updated Views Responsive Grid in D8 though I couldn't get it to show up in more than one column. Played around with it a bit, but wasn't able to get it to do anything in 3-4 columns. I might have to read the docs.....

Anyways, also ran into this PHP warning:

Location	http://example.com/admin/structure/views/ajax/display/gridview/page_1/style_options
Referrer	http://example.com/admin/structure/views/view/gridview/edit/page_1?render=overlay
Message	Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'views_element_validate_integer' not found or invalid function name in _form_validate() (line 1469 of /DRUPAL8/core/includes/form.inc).
iwhitcomb’s picture

derhasi - I applied your patch in the 8.x-1.x branch and made you a maintainer

markhalliwell’s picture

Title: Views Responsive Grid in drupal 8 core » D8 Port
Category: support » task
Status: Needs review » Fixed

Committed to the 8.x-1.x branch: b8478cc.

This is just in-case #1903746-42: Replace the views grid table template with one using divs doesn't make the cut.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.