Feature #29435
closed
Render forums list in project settings as table
Added by Bernhard Rohloff over 6 years ago.
Updated about 6 years ago.
Description
The list of discussion boards is poorly implemented as nested div containers with inline styles and extra CSS classes to look like a table.
I think if it should look like a table, it should also be one.
This also improves the consistency of the UI and reduces the list of CSS classes, theme developers have to style.
Files
Bernhard, thanks for the patches, but I think that you should see r15337 for #12909 where Jean-Philippe Lang replaced the tables with divs in Forum. Because of that, I think that reverting to a table is not a good idea. Also, I prefer to have divs instead tables and maybe is a good idea to replace on other views the tables with divs.
What we can do is to restore the alternate row colors which is missing now.
Please let me know if I can close this ticket as "Won't fix".
- Status changed from New to Needs feedback
- Assignee set to Bernhard Rohloff
Marius BALTEANU wrote:
Bernhard, thanks for the patches, but I think that you should see r15337 for #12909 where Jean-Philippe Lang replaced the tables with divs in Forum. Because of that, I think that reverting to a table is not a good idea. Also, I prefer to have divs instead tables and maybe is a good idea to replace on other views the tables with divs.
Essentially you are right. Divs would make more sense for responsive designs nowadays. I stumbled upon this particular view while creating a new theme from scratch and I found it an extremely odd and inconsistent experience that I have to apply extra styles for one single table.
What we can do is to restore the alternate row colors which is missing now.
Because it's not a huge problem for the casual user and doesn't effect the default theme that much I would let it as it is.
Please let me know if I can close this ticket as "Won't fix".
I've opened a feature request for replacing the tables in #29451 and you can close this issue in favor for this approach.
- Tracker changed from Patch to Feature
- Status changed from Needs feedback to Closed
- Assignee deleted (
Bernhard Rohloff)
- Resolution set to Wont fix
- Related to Feature #29451: Replace tables with components based on nested div containers added
Also available in: Atom
PDF
Updated by 
Updated by