a) Who cares?
b) Who uses this information?
c) It makes the table wider than it needs to (which tends to cause things to wrap unnecessarily, and it makes the table not look as nice on an iphone).
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | pi_my_projects_UI_fixes_2.patch | 5.67 KB | dww |
| #3 | pi_my_projects_UI_fixes_1.patch | 5.68 KB | dww |
| #1 | pi_my_projects_UI_fixes.patch | 4.93 KB | dww |
Comments
Comment #1
dwwWhile I was at it, I noticed some other wonky cases in this UI, so I'm bumping the prio back to normal. Since I had to change the same stuff anyway to deal with removing this column, I figured it was easier to just fix it all in one place:
A) The 'add release' link always appeared, even if the project had disabled releases.
B) The code from http://drupal.org/node/104583 was firing and printing an "Enable" link even if the user couldn't edit the project.
C) If issues were disabled, the usual "Edit | Add release" disappeared, even though those were potentially still valid.
D) The wording on that message was too big to fit in the columns at hand (especially now that we're removing one).
Attached patch solves all of the above, removes the wasteful column, and fixes the .css to give (it seems) better spacing w/o wrapping lines. Installed on p.d.o, for example: http://project.drupal.org/project/user/dww
Looking much better on an iphone now, and in a regular browser, too.
Any objections?
Comment #2
dwwHere's a better example, since it includes a project w/o an issue tracker: http://project.drupal.org/project/user/Eaton
Comment #3
dwwNew patched based on some input from aclight, and more tweaking:
- we add "white-space:nowrap" to the links in the .project-project-links column, so that "Add release" is never wrapped/split.
- removed "open" from each row, and put it in the column header as "Open issues"
- centered the text in the last 3 columns -- looks better.
- tweaked the spacing so that things don't wrap funny either on an iphone, narrow browser window, or wide.
Comment #4
dwwErr, fixed the wording back to just "Issue tracking is disabled." -- works better.
Comment #5
dwwaclight thought this was RTBC, so I committed to HEAD and deployed on d.o. Refresh your browser caches, folks. ;)
Comment #6
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.