Closed (fixed)
Project:
Views Bootstrap
Version:
7.x-3.0
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
6 Nov 2013 at 14:25 UTC
Updated:
5 Dec 2015 at 18:07 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
zilla commentedhas anybody done a table a different way in bootstrap using views? the default table option obviously works and displays, with just zero "bootstrap specific element/use"
Comment #2
rerooting commentedFor Radix UI I began developing something that extended the table display plugin and exposed these options to users. I could probably dig up the code and post it as a patch, it was fairly simple.
Comment #3
zilla commentedthat would be awesome - striped/bordered is quite useful, and doing it via the settings versus styling the entire custom view seems a thousand times easier and more intuitive
Comment #4
mrded commentedTable plugin has been committed to 2.x-dev & 3.x-dev branches.
Comment #6
busla commentedI would really like for the maintainer to re-open the issue since
table-responsiveis missing.Comment #7
mrded commentedComment #8
busla commented@mrred: I looked briefly over the Views Plugins API and didn´t see a simple way to wrap the whole table in a
<div></div>and add .table-responsive to theclasses_array.Perhaps, add a views plugin preproccess function and add a new varible that contains true/false and then add a condition in the template file to include the div or not?
I´m up for creating a patch but might need a tip from you so it doesn´t get messy.
Comment #9
Melissamcewen commentedAn issue I've encountered is my table header is empty
Comment #10
milos.kroulik commentedMelissamcewen: Please open a new issue for your problem. This one is for discussing new functionality, not bug reports. Also don't forget to include export of your view, so we can replicate it.
Comment #11
waverate commentedMarked #2271557: Responsive tables as duplicate of this issue.
Comment #12
alcauza commentedRegarding the responsive table style, here you can find a patch to add this feature.
Despite I am not a fan of Bootstrap solution to provide a responsive table, I though this could be a feature to include in this module in order to mimic the Bootstrap behavior at the most.
An option called 'Responsive' is added to the views bootstrap table style.
The main change is added to the views-bootstrap-table-plugin-style.tpl.php file where a
Comment #13
alcauza commentedComment #14
waverate commented#12 looks good.
Thank you.
Comment #15
burnsjeremy commented#12 works for now, would like to see that empty div completely removed in the future.
Question: How many does "looks good" does an issue need before it can be moved into the next phase? Just wondering, I realize this one only has two (One before me and the next).
Comment #16
mrded commented@burnsjeremy two is enough :)
Comment #17
iamfredrik commented#12 works, but until it gets assigned I think it's better to put the template with the wrapper div in your theme.
Comment #18
shaundychko#12 didn't apply so I re-rolled against version 7.x-3.1 and refactored a little to print the wrapping div only if responsive tables are selected.
Comment #19
waverate commented#12 worked fine against 7.x-3.1+30-dev.
Comment #20
mrded commentedLet's move 'Responsive' feature to following issue: #2271557: Responsive tables