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

Files: 

Comments

Status:Active» Fixed

Yeah, that'd be great.

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.

Status:Needs work» Needs review
StatusFileSize
new25.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.

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.

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

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).

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

Title:Views Responsive Grid in drupal 8 coreD8 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.