[#13835] - [RFC] Responsive Tables
- Closed
- 12 Feb 2017
- Medium
- Build: staging
- # 13835
- Diff
- ciar4n:com-content-table
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
Creates a responsive tables class allowing more data to be displayed on smaller screens. Class applied to article list.
My intent here is to give an alternative table display on smaller screens where you are not limited to the amount of data that is displayed.
Testing Instructions
Navigate to Content -> Article and resize browser window.
Before patch
After patch
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_content Templates (admin) |
Associations don't have a title when shrunk. It's also missing the clickable table headers to sort the fields.
Personally, I don't like the new small view, I prefer the current phone view.
| Labels |
Added:
?
|
||
Really my intent here is to give an alternative table display on smaller screens where you are not limited to the amount of data that is displayed by converting the table from a column view to a row view. The current table, although it looks neater, is very limited on how much data can be displayed. Maybe com_contents is a bad example to demonstrate however the idea may be useful elsewhere.
I used a similar approach on #13769 where it was suggested that the same technique could be used on other tables.
For the repeatable I think this makes sense, because hiding elements doesn't make sense there.
However in regular manager tables, hiding columns is perfectly fine, and those tables already are responsive. No need to fix something that isn't broken imho.
I really like to have the possibility to make tables correctly responsive and not just by hiding elements. Would be nice to have that possibility also on the front end for templates available as less file to include.
I'd suggest removing all the columns apart from the title and checkbox columns. For mobile, this would be enough
I have tested this item
I have tested the request and it works fine!!
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13835.
I'm not in favor of this PR due to the display feeling more cluttered. I attached a screen shot showing with the patch and the other without. Both are responsive, this patch shows more meta data per row which ends up making the article view feel not as clean. Normally I like more meta data, but in this case it doesn't seem to add much value on small display. I suppose maybe the author, but not the permission column.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13835.
| Title |
|
||||||
Conflicts fixed.
I tend to agree that for the article list and for want of as simple a view as possible on mobile devices, this PR might be a little to much. The question is, is there any table view that this layout would be needed? Are we missing any important data on any of the tables on mobile devices that should be displayed?
If so then this PR allows a workable alternative. If not then there is no need for it.
I normally am in favour of all your PR's Ciaran, but on this one I think it over complicates the visible data
I'm gonna close this. If a need for it arises it can be re-opened.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-02-12 08:20:40 |
| Closed_By | ⇒ | ciar4n |
I have tested this item✅ successfully on aa771fc
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/13835.