Bug #3421
closed[Workbench] "Rename collection" button on project page is hidden/inaccessible if the current name is too long and the browser window is too narrow.
Updated by Brett Smith over 10 years ago
There isn't one obviously right way to fix this bug. The project contents tables have fixed layouts, meaning each td has white-space: nowrap; overflow: hidden;
set. And these tables just use render_editable_attribute
to render this name, meaning we can't change the layout of the name+pencil pair without far-reaching consequences. I have not found any way to say "If the contents of this box overflow, render this content at the end" in CSS.
Does this table need to have a fixed layout (i.e., each row is the same height)? Removing that constraint would make for the easiest, and most flexible fix. Judiciously adding s to the row contents would help prevent awkward wraps between buttons, etc.
If the table should have a fixed layout, then I think our next best option is to stop using the application-wide table-fixedlayout
class, and write more specialized CSS for these tables.
We could pass render_editable_attribute
the option to render the pencil on the left of the name instead of the right, but while that's very simple code-wise I think the UI inconsistency would be really jarring to users.
Updated by Brett Smith over 10 years ago
At sprint planning nobody wanted to get too deep into this, so I was given unilateral authority to solve it however I think best.
Updated by Brett Smith over 10 years ago
- Status changed from In Progress to New
Setting the status back to New for the time being because while I've thought about this, no code has been changed yet, and it's going to be a bit before I get to it again.
Updated by Peter Amstutz over 10 years ago
2ad9c20601673674ad787e2457445b6c1a0e9212
I'm glad you were able to find a good solution. Looks good to me.
Updated by Brett Smith over 10 years ago
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Applied in changeset arvados|commit:7482410350df49dbddab05448b167bf61f14d71e.