Project

General

Profile

Actions

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.

Added by Tom Clegg over 9 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Workbench
Target version:
Story points:
0.5

Subtasks 1 (0 open1 closed)

Task #3584: Review 3421-wide-project-tables-wipResolvedBrett Smith08/13/2014Actions

Related issues

Related to Arvados - Bug #3344: [Workbench] Cannot [figure out how to] rename a collection created by arv-put with no --name provided.ResolvedBrett Smith07/24/2014Actions
Actions #1

Updated by Brett Smith over 9 years ago

  • Assigned To set to Brett Smith
Actions #2

Updated by Tim Pierce over 9 years ago

  • Category set to Workbench
Actions #3

Updated by Brett Smith over 9 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.

Actions #4

Updated by Brett Smith over 9 years ago

  • Status changed from New to In Progress
Actions #5

Updated by Brett Smith over 9 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.

Actions #6

Updated by Brett Smith over 9 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.

Actions #7

Updated by Brett Smith over 9 years ago

  • Status changed from New to In Progress
Actions #8

Updated by Peter Amstutz over 9 years ago

2ad9c20601673674ad787e2457445b6c1a0e9212

I'm glad you were able to find a good solution. Looks good to me.

Actions #9

Updated by Brett Smith over 9 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:7482410350df49dbddab05448b167bf61f14d71e.

Actions

Also available in: Atom PDF