Update of Deposit Admin Table is pending.
Nice improvements but that diff is not landable as is (see my inline comments).
Also, you should handle the responsive layout for the tables in the admin interface and the one listing api endpoints.
you can remove that dependency
you can remove this import
I would prefer the approach of hiding the mode and size columns according to the screen size instead of duplicating some template code here.
Plus the icon for directories is not the right one and the icon colors are not the same for small screens.
Still some improvements to bring.
After reading the documentation of responsive DataTables, I think the responsive settings
In the files templates/browse/origin-save.html and templates/admin/origin-save.html, can you rename the table headers to the following:
This will make the headers look nicer on small screen devices.
Plus some inline comments to process.
You need to explicitly import the responsive CSS after the base bs4 one using:
Otherwise, webpack will not process it and the responsive stylesheets will not be part of the produced CSS bundles.
|52 ↗||(On Diff #5051)|
Can you remove the Status column here ? There is no more interest to keep it today.
We should hide the Size column too on small screens