Bugzilla:Bug Layout Revision: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
 
(27 intermediate revisions by 13 users not shown)
Line 1: Line 1:
mconnor's proposal (which includes changes to the header) is [http://steelgryphon.com/firefox2/bmo/Bugzilla_bug4.html here].
preview of the work in progress is [https://landfill.bugzilla.org/prodpatches/show_bug.cgi?id=415 here]; note that logging in changes the appearance quite a bit.
{| style="background: lightgray; border: 1px solid blue"
| '''NOTE:''' Changes that don't have working patches by early evening PST Saturday December 23rd will not be included in the upgrade on Tuesday the 26th.
|}<br>
== Comments ==
== Comments ==
(please include your rationale!)
(please include your rationale!)


=== Jesse ===
=== gerv ===


I like the wording changes and "edit" links.  Those save space and make the page less cluttered.
Comments on the current version of the redesign (2007-01-02):


I don't like the overall layout change.  The three-column layout currently used at the top of bugs saves space, allowing me to see the CC list and the first few attachments without scrolling. Is there a good reason to switch to an entirely two-column layout?
* Priority and Severity are a long way apart; in an ideal world, the two are used together to judge the importance of a bug. Therefore, they should be placed together. I suggest moving Severity down next to Priority.
** As I understand it, the rationale for separating them is that priority is managed by the assignee (together with TM), while severity is not. It was therefore put in the "assignee" block. [[User:GavinSharp|GavinSharp]]


=== beltzner ===
* IMO, the alias in the new header should be enclosed in square rather than round brackets, and perhaps italicized to make it obvious it's not part of the summary text.  
* Agree with Jesse, 3 columns allows for more information density on a page and will actually make it easier to visually scan each chunk/grouping
* When no search results are available, just hide that row since it adds no valuable information
* Compress the header as in mconnor's mockup, the repetition of the bug name and number is distracting and confusing
* The actions section takes up too much space and is too visually prominent. There doesn't need to be a "format for printing" button, we should just use a sensible print.css file. Clone bug and "XML" are seemingly low value, too. Instead I'd propose something like:
  Bug 415 - blah blah blah blah (edit)
  [view activity] [jump to latest comment]  (<-- small text)


=== Wolf ===
* The alias is before the summary in the header but after it in the editing boxes. We should make the two consistent.
Agree with Jesse and Beltzner about the two vs three column layout stuff (and hiding the actions when searching.). Except for the header compression. I find the header appears better when not compressed. (and as far as I can see, there's no space actually being saved by those changes.) I think mirroring the actions line at the top and bottom is useful.


Removing the "Bug 415" (page title) portion might work on showbug, but on other pages, I don't think it does, since it provides useful information, which is only shown there. Plus the readdition of that value would likely end up being too wide with all the links on the same line. (See the activity log, etc). Also, why is preferences being changed to settings? Preferences seems to be more correct.
* The edit link in the header box should not be bold.
I'm not sure print view is served well by being just print.css fixed (as the page says, its a full text bug list, not really just a css change. Agree about XML, not sure what the purpose for it is, not that it bothers me being there, either. Clone Bug might be of value, though, since it didn't work in bmo before. (Just having two links to me, suggests either over pruning, or that that line should be elsewhere.)


=== Ancestor ===
* When we print names with spaces, e.g. "Dave Miller (work related)", we should use non-breaking spaces. This affects "Reported".
The dependency list tries to put as many entries in one line as it can. This becomes a problem when there are plenty of entries, because it pushes the right-hand column too far to the right. [https://landfill.bugzilla.org/prodpatches/show_bug.cgi?id=4728 see here for example].


=== LpSolit ===
* Target Milestone can be shortened to Target. Milestone is both Mozilla-specific and obsolete.
The goal of including the bug summary in the header of the page is to save some vertical space. Repeating it again in the gray line is a regression. But I can understand you want it here for visibility. Now, links to go to the prev/next/first/last bug are taking too much space and should probably be moved on the same line as other "Action" links, i.e. XML, Format for printing, Last Comment.
** Milestones are hardly Mozilla specific. That's a standard business term. Target Milestone exactly conveys what this blank should contain. [[User:Hacksaw|Hacksaw]]


Having the dependency lists on the left and having the CC list always visible is definitely the right approach (especially when you have a very long list of flags).
* Right-alignment of the field names would give a better two-column effect and direct the eye to the important information, because I think people look to the right of vertical lines (e.g. ruled margins).


Links about votes are really not important and should be moved in the "Action" list (with XML, etc...).
* I don't know about anyone else, but on my 1024x768 screen, I have a lot of whitespace in the middle, and yet the right side (Reporter, Modified etc.) is cramped and wrapping.


Note on bug 415 that I reassigned the bug to timeless, who has no real name on b.m.o, and now the assignee field is left blank. If the user has no realname, then his email address should be displayed.
* The tooltip on bug numbers truncates the summary too early. As this is a Gecko limitation, can we work with it by using abbreviations for states (e.g. RES DUPL rather than RESOLVED DUPLICATE)?


Now I strongly disagree to have the keywords, URL and status whiteboard fields at the very top. This looks wrong to me. More important is the block with the assignee + target milestone + severity, and should be moved higher in the page. The status and resolution should be combined with this block.


Ancestor, about the dependency lists: it indeed tries to use as much place as possible, but will wrap to avoid the user to scroll horizontally, so this is fine and exactly what we want. If your screen is narrower, it will wrap more often.
Comments on dolske's mockup2:


''I know that but I feel it would be better to wrap the dependency list earlier so that the columns aren't so far apart from each other with a huge white space between them. - Ancestor''
* The changes to eliminate the "knob" are non-skin changes; they should be removed in order to make it more obvious what changes he is suggesting that are actually doable in a skin revision.


* I like the drop-down styling, but it shouldn't apply to the flags - it makes it look as if the flag name can be changed.


Nit: please remove the now useless colspan=3 in the right table as the dependency lists have been moved to the left.
* The Reported and Modified dates should either both have seconds, or neither.


=== Vlad ===
* The field title color is too light. I see the desire to deemphasize, but this has gone too far. Even a small departure from black can have a noticeable effect.


This is minor, but the attachments table is pretty ugly.  At the very least I'd like to see the double-weird-90's-style-3d-borders just become simple 1-pixel line borders.  I also think that this page overall does too much "Field name: Value", and places all field names on the same importance as all others, when in reality they're not.  This is very noticeable in the header, but also in the attachments table.  Here's my take on some ideas for the attachments table: [http://people.mozilla.com/~vladimir/misc/bugz.html here].  The first table just has borders and backgrounds changed, deemphasizing obsolete attachments.  The second is what I'd really like to see, because it makes the important information (basically, patch name, type, review flags) much more visible.
* Target Milestone -> Target


=== Dolske ===
* Merging Priority and Severity into "Importance" is wrong; if these fields are used correctly, they mean two different things, and need to be labeled as such. It's not obvious from "P1... P5" that it's a priority field.


* Putting page specific info (the bug title) into the top header seems like a  unusual design, so the location would seem unexpected. Most sites have a relatively static header that one learns to ignore.
[[User:Gerv|Gerv]] 07:23, 2 January 2007 (PST)


* Status and Resolution have already been combined in the new version (now "Status: RESOLVED INVALID"). It might be worthwhile to combine a few other fields (ie, put them in the same row). For example, Hardware + OS -> Platform (System?).  And perhaps Priority + Severity -> ???.
=== gwagenknecht ===
* Over all, I like the redesign of the show bugs page.
* Having the bug status as the first field is great. But I would like it to be even more prominent (for example, try to leave the cell bellow it empty). (It might be subjective but it's one of my major complaints against JIRA. It's not easy enough to ready the bug status. Don't know why but maybe because the UI is too busy for me.)
* I like the ideas exposed by ''dolske'' in [http://people.mozilla.com/~dolske/bugzilla/mockup2.html mockup2]. Especially the right alignment of the field names make it easier to read and the colorization of the input fields makes them easier to recognize.
* Don't know if it would be interesting to have dates formatted as "x hours/days/weeks/months ago" for the modification date. (Could be a user setting though)
--[[User:Gwagenknecht|Gwagenknecht]] 07:22, 2 January 2007 (PST)


=== Peter ===
=== gekacheka ===


I very much like the 2 column layout. The old 3-column layout only works when running with the browser window maximized (horizontally). I almost never do that and so that caused a lot of very annoying horizontal scolling. The proposed 2 column layout scales much better in width.
<b>1. Conflicting Criteria for design: use style independent of grouping?</b>


Otherwise I agree with what has been said by the others.
* Group related fields by assessment task:
** Assess status:  Status, Keywords, Whiteboard, Depends on, Attachment Flags, Flags
** Assess importance: Severity, Keywords.   Priority.  Blocking.  Flags.
** Assess urgency: Severity, Target, Flags, Blocking.
** Assess interest: Votes, CC
** Assess module: Product, Component, Summary, Description/Comment
** Assess reproducibility:  Description/Comment, Attachments, URL, Product, Version, OS, Hardware [hw rare?]


=== Napolj2 ===
* Identify who may normally edit fields (e.g., avoid lay commenters editing priority, target)
** Interested party:  CC, Votes, comments.
** Reporter/Triager/Confirmer:  any except Priority, Target., Assignee, QA, Flags
** Module owners, Assignee:  any including Priority, Target, Assignee, QA, Flags


I'd move Assignee and QA Contact to just above the CC list, as they all fall under the same conceptual category as "People involved with this bug." Then I'd put Priority, Severity, and Target Milestone in with Status/Resolution, Keywords, and Whiteboard, as they fall under the category of "What state is this bug in". The remaining group of Product, Component, Version, Hardware, and OS, describes this category of "Where this bug occurs."
Ideas:
* separate assignee/QA fields into group or column associated with QA.
* identify group of reserved fields with <span style="border: 2px solid black">border</span> or <span style="background: silver">shaded background</span>. (like in paper forms)


Ideally, URL should just be considered a type of attachment and not a whole separate field (though this would require converting all the existing URL fields for existing bugs into new attachments, and perhaps patching Bugzilla).
<b>2. Attachments table: consistent location of type</b>


When you have a large amount of dependencies/blocking bugs, you can end up with one column being significantly longer that the other, as in [https://bugzilla-test.mozilla.org/show_bug.cgi?id=300030 the reflow refactoring bug].  (Maybe this won't be an issue since b.m.o has lots of flags too)Perhaps you could allow the dependency/block lists to span both columns?  This can be done by moving them to a second TR (with colspan=3) within the outer TABLE.
The type of the attachment should be in a consistent location so
it is easy to scan for attachments of a particular type.   


I'd widen the attachments table to match whatever width the comments are line-wrapped at (or the width of the comment headers), for visual consistency.
For example, sometimes I want to quickly find all the image
attachments among all the other attachments (say for UI review).  Currently attachment type is placed
after the title, so its location varies wildly from row to row
depending on the length of the title. <br>


The bug description and number are in 3 places, including the page TITLE. I Like how MConnor does the header, with no bug-specific information in it.
Example of problem:<br>
    <blockquote><b><u>Zip of files exhibiting issue</u></b> <small>(29.44KB
application/octet-stream)<br>
2007-01-01 11:23 UTC <u>Artie Boss</u></small><br>
      <u><b>Toolkit patch</b></u> <small>(21.43KB patch)<br>
2007-01-01 12:34 UTC <u>Cody Smith</u></small><br>
      <u><b>Add native Mac theme code, update windows code</b></u> <small>(67.57KB
      <br>
patch)<br>
2007-01-01 13:45 UTC <u>Cody Smith</u></small><br>
      <u><b>Possible approach on windows</b></u> <small>(2.38KB
image/gif)<br>
2007-01-01 23:45 UTC <u>Artie Boss</u></small><br>
      <u><b>Updated patch</b></u> <small>(21.44KB patch)<br>
2007-01-01 23:56 UTC <u>Cody Smith</u></small><br>
    </blockquote>
Suggestion:<br>
Place it on the second line. Right justify variable length
contributor's name before fixed-length date<br>
<blockquote>
  <table width="90%">
  <tr><td><b><u>Zip of files exhibiting issue</u></b></td></tr>
  <tr><td><table width="100%" style="margin: 0px"><tr><td><small>(29.44KB application/octet-stream)</small></td><td style="text-align:right"><small><u>Artie Boss</u> <u>2007-01-01 11:24 UTC</u></small></td></tr></table></td></tr>
  <tr><td><u><b>Toolkit patch</b></u></td></tr>
  <tr><td><table width="100%" style="margin: 0px"><tr><td><small>(21.43KB patch)</small></td><td style="text-align:right"><small><u>Archie Tek (vacationing until Sept)</u> <u>2007-01-01 12:34 UTC</u></small></td></tr></table></td></tr>
  <tr><td><u><b>Add native Mac theme code, update windows code</b></u></td></tr>
  <tr><td><table width="100%" style="margin: 0px"><tr><td><small>(67.57KB patch)</small></td><td style="text-align:right"><small><u>Cody Smith</u> <u>2007-01-01 13:45 UTC</u></small></td></tr></table></td></tr>
  <tr><td><u><b>Possible approach on windows</b></u></td></tr>
  <tr><td><table width="100%" style="margin: 0px"><tr><td><small>(2.38KB image/gif)</small></td><td style="text-align:right"><small><u>Artie Boss</u> <u>2007-01-01 23:45 UTC</u></small></td></tr></table></td></tr>
  <tr><td><u><b>Updated patch</b></u></td></tr>
  <tr><td><table width="100%" style="margin: 0px"><tr><td><small>(21.44KB patch)</small></td><td style="text-align:right"><small><u>Cody Smith</u> <u>2007-01-01 23:56 UTC</u></small></td></tr></table></td></tr>
  </table>
</blockquote>


The voting links and first/last/prev/next links should be moved into the Actions bar to save space. Actually, those navigation links are useless to me, since I always open bug links from a search into new tabs.
<b>3. "Product" link to product descriptions</b>


=== ardissone (Camino) ===
Suggest "Product" link to page with product descriptions, such as https://bugzilla.mozilla.org/enter_bug.cgi?full=1 to explain obscure product names such as 'core', 'NPSR', etc.
I concur with Jesse and beltzner about the information density benefits of the 3-column layout; specifically, it keeps (or can keep) both the product block ("what product am I in now?") and the assignee/target/prio at the top of the page, instead of having to debate over which of those two blocks are more important (and get second billing in the left column) and which gets buried. Right now I'm not happy with assignee/prio/target being at the bottom of that left column, but burying product there in earlier versions was just as annoying.


The other issue I have is that the "action" block is now totally disconnected from the fields it modifies, and this seems problematic--especially on bugs with more than about 5 comments.  The assignee/product/resolution data is at the "top" of the page and the buttons that change those data are at the very bottom.  (I'm quite fond of the comment text area moving to the bottom, though; no more scrolling up and down the page to refer to the last comment when composing a new one!)
[[User:Gekacheka|Gekacheka]] 09:50, 13 January 2007 (PST)


=== Callek ===
== Mockups ==
 
=== mconnor ===
My only comment which is not enclosed by things above, is that the individal comments text-itself, could use some more left-padding, to indent it at least a bit more than the headers for each comment, as it is those headers appear to be part of surrounding comments, rather than headers at a brief glancing.  (I thought that for brief a moment).
 
=== Silver ===


Two things really bug me with the new page:
[http://steelgryphon.com/firefox2/bmo/Bugzilla_bug4.html This mockup] was originally done prior to the redesign. Linked here for easy reference.


* The magical expanding comment box. It expands too much here, and doesn't seem to depend on browser window size, making it really, really annoying. (Yes, I know it's an option I can turn off, but it really helps to have things behave sanely by default.)
=== Dolske (mockup2) ===
* The Hide Obsolete/Show Obsolete links for the attachment table cause the browser to jump to the top of the table - really annoying. If it is to scroll anywhere, it should scroll by just enough up or down to keep the link (and bottom of the table) in the same place. That's trivial JS, too.


=== Wayne ===
I've created [http://people.mozilla.com/~dolske/bugzilla/mockup2.html another mockup], based off the redesign, to try out some more/revised ideas...
* Visual noise from form controls is again reduced (as the [http://people.mozilla.com/~dolske/bugzilla/mockup1.html previous mockup] tried), but instead of using a zillion "(edit)" links the fields are directly editable. Thanks to Jesse for suggesting styling the controls.
* The field labels were becoming too visually prominent. Changed weight and color to not draw attention away from the bug data. I also undid mconnor's left-alignment, as that was also seemed to call attention to them (ie, it almost looked like a 4-columns instead of just 2).
* Stole the header from mozilla.com, just to try out something different. The footer also changed, but there's functionality missing.
* Removed the clump of radio-button actions that were underneath the comment box. Instead, these modifications should be done direction upon the fields themselves. Need to do some more work to make sure those operations are still easy to do, but hey it's just a mockup. :-)
* Table widths are dynamic now, try changing the width of the window.
* Not shown: fields that are modified should change their background color to a light-red to indicate the modification.


There is a lot to like in this makeover which I won't go into except to say kudos to the team, and good comments above.
--[[User:Jmdesp|Jmdesp]] 07:15, 29 January 2007 (PST) <br/>
I like your mock-up, but is there a way to make it degrace more gracefully under IE6 ? The form controls changes don't seem to work under it. Also I think it's better not to remove the edit link after it's pressed


Least controversial (hopefully) aesthetics first:
== Summary of discussion prior to 26-Dec-2006 upgrade ==


* horizontal rulers IMO add nothing but space and are so 80s - begone forever (shaded text line is a more pleasing and more obvious visual cue)
(Note: this page was getting huge, and trying to rearrange and reclassify all the comments would have been a huge time sink. Instead, I've tried to fairly summarize things in a way useful for future discussions.) See the [http://wiki.mozilla.org/index.php?title=Bugzilla:Bug_Layout_Revision&oldid=45267  last verbose version] for the complete comment list.)
* shaded banner is sufficient to delineate each comment - the extra line at the end of each comment wastes space - begone (the only thing it could help IMO is writing comments on printed pages, which has got to be a rarity)
* "additional comments" - shade the line (a natural extension of the progression of the comments above), could also shorten to "add comments"
* in the change section at bottom, remove the word "bug" and associated prepositions from "accept bug", "resolve bug ...", etc - it's implied (and, as an example, not used in the text at top section of showbug)


In addition:
The redesign was largely based on a [http://steelgryphon.com/firefox2/bmo/Bugzilla_bug4.html proposal mockup] from mconnor. Changes were prototyped [https://landfill.bugzilla.org/prodpatches/show_bug.cgi?id=415 here].


* (speaking for the poor _average_ bugzilla user) whiteboard I would argue  when it has something useful is possibly the most important of all the items for communicating to the average reader something to which they need to pay attention (workaround comment #, warnings, don't comment unless, etc) - and therefore should stay in a prominent location or it is too easily missed. Respectfully disagreeing with LpSolit, I would argue at least above keyword.
=== Issues not resolved by redesign ===
* 2-column vs 3-column - 2 is easier to read and I don't see significant space gains with 3 (this coming from someone who is all about screen real estate)
* mconnor's layout below "Bug #" looks cleaner - not saying it's all better, but clean is appealing
* "last comment" link doesn't belong on a line with search functions
* a collection of actions is appealing but should not take an extra line up top - could "actions" be a drop down selection list next to "edit" on the shaded bug summary line?
* votes can perhaps go anywhere - but I submit for the average user it is lost amongst everything up top - it may not be cleaner but somewhere near "commit" makes more sense from a usability standpoint
* I very much dislike using two lines at the top, with "settings", logout and all kinds of links, where one line will do and which includes things which will be so infrequently used (unless there is a user pref to only display at bottom)


Finally, search and search navigation in multiple places seems odd. After looking at the old layout, i.e. the line with "Bug List:", mconnor's and the current proposal, how about modified version with a single line for all functions related to '''navigating''' bugzilla (not navigating the bug or prefs). For example
# When no search results are available, just hide that row since it adds no valuable information
# "Format for Printing" should be removed, replaced by appropriate media-specific CSS
## However, note that content of the current printing-formated page differs.
# Combine some fields to a single line. For example, Hardware + OS -> Platform
# Make "URL" an attachment (this is a new BZ feature)
# In the change section at bottom, remove the word "bug" and associated prepositions from "accept bug", "resolve bug ...", etc - it's implied
# Search and search navigation in multiple places seems odd. Try combing to a single line.
# Assignee needs an "(edit)" link
# Only show time estimate to logged in users.
# Some of the field names are links, others are not.
# Attachments should link to the comment describing them.
# The big radio list at the bottom of the page (right above the "Commit" button) really needs to be simplified, or even eliminated. Most of these actions could be some from the controls at the top of the page, with better design.
# The "Depends On" and "Blocks" fields should be clipped if they're really long (with a mechanism to show them all if desired). "Depends on: 99 bugs (2 open, 97 resolved)" seems more useful than a giant block of numbers.
# Bugzilla should lend a hand when you're filling out a field with a name (assigned to, qa contact, flags). "Default" is probably needed as a minimum, although a longer list of people appropriate for the task would be nice.
# The yellow header bar linking to mozilla.org seems completely useless. Maybe make it a footer, or just delete it entirely?


'''Bug:''' <u>New</u> '''Search:'''  ______ Quick  <u>Advanced</u>  '''Search Results:'''  <u>Edit</u>  ''<u><< First</u>  <u>< Prev</u>  m of n  <u>Next ></u>  <u>Last >></u>''


=== biesi ===
=== Problems/Objections with the redesign ===
Comments on https://landfill.bugzilla.org/prodpatches/show_bug.cgi?id=415


* Severity should move elsewhere, it's a property of the bug, it doesn't change over its lifetime (conceptually)
# Various comments about how the fields should be properly arranged. Everyone had their own rationale, so summarizing the ideas here isn't useful. :-)
* if I click the qa contact's "edit" link, the text field appears but the "edit" text doesn't change, even though it hides the textbox when I click it again
# 3-column layout better than 2-column (saves space, higher information density, easier to scan each chunk/grouping)
* perhaps assignee should get an "edit" link as well, just like qa contact
## However, may look worse if your browser isn't wide.
* IMO the time estimate should be shown to not logged-in users as well
## Have a single "People involved with bug" block
* adding a named tag to the currently displayed bug shouldn't require me to type in the bug#
# Old header looked better (redesign doesn't save space, consistency with other Bugzilla pages)
* IMO the dropdown list for the named tags and the textbox for creating a new tag should be on separate lines
# "Preferences" changed to "Settings"
# The "magically expanding comment box" is annoying.
# Each comment should have more left-padding to move it under the header
# Action block (at bottom of page) now disconnected from the fields it operates on (at top of page)
# Some colored regions have curved borders, others do not.
# Too much whitespace between field and label.
# Is "View All" link on attachments really useful?
# Double clicking on my mail address at the bottom of the page and pasting it into a text field prepends it with "# "




=== Jesse (again) ===
=== Other mockups ===


==== Jesse ====
I've made a [http://www.squarefree.com/bugzilla-ui/spice-3col.html proposal] that uses a "3 columns at the top, 2 columns in the middle" layout (similar to current BMO layout) but based on the page with improved grouping and "(edit)" links.  It saves a significant amount of vertical space, gives more room to the URL and whiteboard fields, and IMO looks pretty good.  See [http://www.squarefree.com/bugzilla-ui/changes.html this page] for a list of changes I made.
I've made a [http://www.squarefree.com/bugzilla-ui/spice-3col.html proposal] that uses a "3 columns at the top, 2 columns in the middle" layout (similar to current BMO layout) but based on the page with improved grouping and "(edit)" links.  It saves a significant amount of vertical space, gives more room to the URL and whiteboard fields, and IMO looks pretty good.  See [http://www.squarefree.com/bugzilla-ui/changes.html this page] for a list of changes I made.


=== Mats ===


I also prefer a 3-column layout to save vertical space.
Reply from Silver:


The navigation box at the top (with purple background) only needs to be
* Less information is available "above the fold" here (it's lost the last modified time and votes).
one line IMO, like mconnor [http://steelgryphon.com/firefox2/bmo/Bugzilla_bug4.html suggested].
* In addition to other comments against it, it actually gives me '''less''' space for the keyword/whiteboard/URL fields!
 
The "Target Milestone" should be changed to something shorter -
it makes the horizontal distance between the field names/values
above it too wide which in turn makes the left column unnecessarily
wide as a result.
 
Some of the field names are links, others are not.
I suggest adding a "title" attribute that describe their meaning
for the non-links (as you did for the individual Flags at [https://bugzilla-test.mozilla.org/show_bug.cgi?id=334136 bugzilla-test.mozilla.org])
and for the links describing what will happen if you click the link.


The summary header ("Bug# - Summary" with grey background) uses border radius,
==== Dolske (mockup1) ====
but other boxes don't, e.g. comment headers (I'd prefer no border radius).


The horizontal distance between "Summary:" and the input to the right is too short. Same for "Alias:".
I had a few more ideas on streamlining things, and did a [http://people.mozilla.com/~dolske/bugzilla/mockup1.html rough mockup] to see how well they work. It needs fine-tuning, ignore any fixable rough edges.
 
Attachments:
"Hide/Show Obsolete" should not scroll the page.
"View All" - is this really a useful feature? I have never used it that I can recall.
 
I don't like the auto-expanding "Additional Comments" when it's clicked (it scrolls
the "Commit" button out of view and I dislike non-standard behaviour for well known form elements in general - POLA).
Make the bigger size static, vertical space isn't that important here I think.
 
There is also a '''regression''': double clicking on my mail address at the bottom of the page and pasting it into a text field prepends it with "# ".
 
=== mconnor ===
 
Having spent some time flipping between the current prodpatches layout and Jesse's mockup, I find I keep skipping over the product/etc block, and the information isn't doesn't seem to scan in order.  In the two column layout, skimming one column and then the other gives you all of the information in a relatively prioritized and well-grouped way, but I don't see a sane scanning order in Jesse's mockup that's efficient to me.
 
It does have a marginal effect on vertical space (one or two lines on my MBP screen), but information density is only useful if that density does not affect readability.  Having five short columns instead of two taller ones just doesn't work for me.  It feels cluttered and misaligned now, especially at my typical browser window width of 1200 pixels or so.
 
I will echo the header complaint, but we'd need to modify all of bmo, not just the show bug page, so we may have to fix that as a second pass, but I'll agree that its really not as effective as some feel.
 
=== Silver (2) ===
 
Referring to [http://www.squarefree.com/bugzilla-ui/spice-3col.html the 3 column layout]:
 
* Less information is available "above the fold" here (it's lost the last modified time and votes).
* In addition to other comments against it, it actually gives me '''less''' space for the keyword/whiteboard/URL fields!


For these reasons, I object to it being used, and much prefer the existing 2 column proposal.
* The outlines and icons of form controls add a *lot* of visual complexity to the page. The mockup replaces almost all the HTML controls with just plain text (but you can reactivate the controls by clicking "edit"). This really improves readability, but the obvious cost is having to click "edit" to update fields is annoying. This needs improved.
* The top part of the bug is using CSS3(ish) columns, so you will see 2 or 3 columns depending on how wide the browser is. Or, uhh, 1 column for narrow windows. And browsers that don't support -moz-column-width. There's not much control over where a column starts, so tweaking a design to work with different column counts is tricky (the mockup just happens to look fairly reasonable). Perhaps there are JS/DOM tricks that could help? I consider this a failed experiment as-is.
* The CC list is hidden again (ala mconnor's mockup).
* Combined some fields

Latest revision as of 00:11, 17 April 2007

Comments

(please include your rationale!)

gerv

Comments on the current version of the redesign (2007-01-02):

  • Priority and Severity are a long way apart; in an ideal world, the two are used together to judge the importance of a bug. Therefore, they should be placed together. I suggest moving Severity down next to Priority.
    • As I understand it, the rationale for separating them is that priority is managed by the assignee (together with TM), while severity is not. It was therefore put in the "assignee" block. GavinSharp
  • IMO, the alias in the new header should be enclosed in square rather than round brackets, and perhaps italicized to make it obvious it's not part of the summary text.
  • The alias is before the summary in the header but after it in the editing boxes. We should make the two consistent.
  • The edit link in the header box should not be bold.
  • When we print names with spaces, e.g. "Dave Miller (work related)", we should use non-breaking spaces. This affects "Reported".
  • Target Milestone can be shortened to Target. Milestone is both Mozilla-specific and obsolete.
    • Milestones are hardly Mozilla specific. That's a standard business term. Target Milestone exactly conveys what this blank should contain. Hacksaw
  • Right-alignment of the field names would give a better two-column effect and direct the eye to the important information, because I think people look to the right of vertical lines (e.g. ruled margins).
  • I don't know about anyone else, but on my 1024x768 screen, I have a lot of whitespace in the middle, and yet the right side (Reporter, Modified etc.) is cramped and wrapping.
  • The tooltip on bug numbers truncates the summary too early. As this is a Gecko limitation, can we work with it by using abbreviations for states (e.g. RES DUPL rather than RESOLVED DUPLICATE)?


Comments on dolske's mockup2:

  • The changes to eliminate the "knob" are non-skin changes; they should be removed in order to make it more obvious what changes he is suggesting that are actually doable in a skin revision.
  • I like the drop-down styling, but it shouldn't apply to the flags - it makes it look as if the flag name can be changed.
  • The Reported and Modified dates should either both have seconds, or neither.
  • The field title color is too light. I see the desire to deemphasize, but this has gone too far. Even a small departure from black can have a noticeable effect.
  • Target Milestone -> Target
  • Merging Priority and Severity into "Importance" is wrong; if these fields are used correctly, they mean two different things, and need to be labeled as such. It's not obvious from "P1... P5" that it's a priority field.

Gerv 07:23, 2 January 2007 (PST)

gwagenknecht

  • Over all, I like the redesign of the show bugs page.
  • Having the bug status as the first field is great. But I would like it to be even more prominent (for example, try to leave the cell bellow it empty). (It might be subjective but it's one of my major complaints against JIRA. It's not easy enough to ready the bug status. Don't know why but maybe because the UI is too busy for me.)
  • I like the ideas exposed by dolske in mockup2. Especially the right alignment of the field names make it easier to read and the colorization of the input fields makes them easier to recognize.
  • Don't know if it would be interesting to have dates formatted as "x hours/days/weeks/months ago" for the modification date. (Could be a user setting though)

--Gwagenknecht 07:22, 2 January 2007 (PST)

gekacheka

1. Conflicting Criteria for design: use style independent of grouping?

  • Group related fields by assessment task:
    • Assess status: Status, Keywords, Whiteboard, Depends on, Attachment Flags, Flags
    • Assess importance: Severity, Keywords. Priority. Blocking. Flags.
    • Assess urgency: Severity, Target, Flags, Blocking.
    • Assess interest: Votes, CC
    • Assess module: Product, Component, Summary, Description/Comment
    • Assess reproducibility: Description/Comment, Attachments, URL, Product, Version, OS, Hardware [hw rare?]
  • Identify who may normally edit fields (e.g., avoid lay commenters editing priority, target)
    • Interested party: CC, Votes, comments.
    • Reporter/Triager/Confirmer: any except Priority, Target., Assignee, QA, Flags
    • Module owners, Assignee: any including Priority, Target, Assignee, QA, Flags

Ideas:

  • separate assignee/QA fields into group or column associated with QA.
  • identify group of reserved fields with border or shaded background. (like in paper forms)

2. Attachments table: consistent location of type

The type of the attachment should be in a consistent location so it is easy to scan for attachments of a particular type.

For example, sometimes I want to quickly find all the image attachments among all the other attachments (say for UI review). Currently attachment type is placed after the title, so its location varies wildly from row to row depending on the length of the title.

Example of problem:

Zip of files exhibiting issue (29.44KB

application/octet-stream)
2007-01-01 11:23 UTC Artie Boss

Toolkit patch (21.43KB patch)
2007-01-01 12:34 UTC Cody Smith

Add native Mac theme code, update windows code (67.57KB
patch)
2007-01-01 13:45 UTC Cody Smith

Possible approach on windows (2.38KB image/gif)
2007-01-01 23:45 UTC Artie Boss

Updated patch (21.44KB patch)
2007-01-01 23:56 UTC Cody Smith

Suggestion:
Place it on the second line. Right justify variable length contributor's name before fixed-length date

Zip of files exhibiting issue
(29.44KB application/octet-stream)Artie Boss 2007-01-01 11:24 UTC
Toolkit patch
(21.43KB patch)Archie Tek (vacationing until Sept) 2007-01-01 12:34 UTC
Add native Mac theme code, update windows code
(67.57KB patch)Cody Smith 2007-01-01 13:45 UTC
Possible approach on windows
(2.38KB image/gif)Artie Boss 2007-01-01 23:45 UTC
Updated patch
(21.44KB patch)Cody Smith 2007-01-01 23:56 UTC

3. "Product" link to product descriptions

Suggest "Product" link to page with product descriptions, such as https://bugzilla.mozilla.org/enter_bug.cgi?full=1 to explain obscure product names such as 'core', 'NPSR', etc.

Gekacheka 09:50, 13 January 2007 (PST)

Mockups

mconnor

This mockup was originally done prior to the redesign. Linked here for easy reference.

Dolske (mockup2)

I've created another mockup, based off the redesign, to try out some more/revised ideas...

  • Visual noise from form controls is again reduced (as the previous mockup tried), but instead of using a zillion "(edit)" links the fields are directly editable. Thanks to Jesse for suggesting styling the controls.
  • The field labels were becoming too visually prominent. Changed weight and color to not draw attention away from the bug data. I also undid mconnor's left-alignment, as that was also seemed to call attention to them (ie, it almost looked like a 4-columns instead of just 2).
  • Stole the header from mozilla.com, just to try out something different. The footer also changed, but there's functionality missing.
  • Removed the clump of radio-button actions that were underneath the comment box. Instead, these modifications should be done direction upon the fields themselves. Need to do some more work to make sure those operations are still easy to do, but hey it's just a mockup. :-)
  • Table widths are dynamic now, try changing the width of the window.
  • Not shown: fields that are modified should change their background color to a light-red to indicate the modification.

--Jmdesp 07:15, 29 January 2007 (PST)
I like your mock-up, but is there a way to make it degrace more gracefully under IE6 ? The form controls changes don't seem to work under it. Also I think it's better not to remove the edit link after it's pressed

Summary of discussion prior to 26-Dec-2006 upgrade

(Note: this page was getting huge, and trying to rearrange and reclassify all the comments would have been a huge time sink. Instead, I've tried to fairly summarize things in a way useful for future discussions.) See the last verbose version for the complete comment list.)

The redesign was largely based on a proposal mockup from mconnor. Changes were prototyped here.

Issues not resolved by redesign

  1. When no search results are available, just hide that row since it adds no valuable information
  2. "Format for Printing" should be removed, replaced by appropriate media-specific CSS
    1. However, note that content of the current printing-formated page differs.
  3. Combine some fields to a single line. For example, Hardware + OS -> Platform
  4. Make "URL" an attachment (this is a new BZ feature)
  5. In the change section at bottom, remove the word "bug" and associated prepositions from "accept bug", "resolve bug ...", etc - it's implied
  6. Search and search navigation in multiple places seems odd. Try combing to a single line.
  7. Assignee needs an "(edit)" link
  8. Only show time estimate to logged in users.
  9. Some of the field names are links, others are not.
  10. Attachments should link to the comment describing them.
  11. The big radio list at the bottom of the page (right above the "Commit" button) really needs to be simplified, or even eliminated. Most of these actions could be some from the controls at the top of the page, with better design.
  12. The "Depends On" and "Blocks" fields should be clipped if they're really long (with a mechanism to show them all if desired). "Depends on: 99 bugs (2 open, 97 resolved)" seems more useful than a giant block of numbers.
  13. Bugzilla should lend a hand when you're filling out a field with a name (assigned to, qa contact, flags). "Default" is probably needed as a minimum, although a longer list of people appropriate for the task would be nice.
  14. The yellow header bar linking to mozilla.org seems completely useless. Maybe make it a footer, or just delete it entirely?


Problems/Objections with the redesign

  1. Various comments about how the fields should be properly arranged. Everyone had their own rationale, so summarizing the ideas here isn't useful. :-)
  2. 3-column layout better than 2-column (saves space, higher information density, easier to scan each chunk/grouping)
    1. However, may look worse if your browser isn't wide.
    2. Have a single "People involved with bug" block
  3. Old header looked better (redesign doesn't save space, consistency with other Bugzilla pages)
  4. "Preferences" changed to "Settings"
  5. The "magically expanding comment box" is annoying.
  6. Each comment should have more left-padding to move it under the header
  7. Action block (at bottom of page) now disconnected from the fields it operates on (at top of page)
  8. Some colored regions have curved borders, others do not.
  9. Too much whitespace between field and label.
  10. Is "View All" link on attachments really useful?
  11. Double clicking on my mail address at the bottom of the page and pasting it into a text field prepends it with "# "


Other mockups

Jesse

I've made a proposal that uses a "3 columns at the top, 2 columns in the middle" layout (similar to current BMO layout) but based on the page with improved grouping and "(edit)" links. It saves a significant amount of vertical space, gives more room to the URL and whiteboard fields, and IMO looks pretty good. See this page for a list of changes I made.


Reply from Silver:

  • Less information is available "above the fold" here (it's lost the last modified time and votes).
  • In addition to other comments against it, it actually gives me less space for the keyword/whiteboard/URL fields!

Dolske (mockup1)

I had a few more ideas on streamlining things, and did a rough mockup to see how well they work. It needs fine-tuning, ignore any fixable rough edges.

  • The outlines and icons of form controls add a *lot* of visual complexity to the page. The mockup replaces almost all the HTML controls with just plain text (but you can reactivate the controls by clicking "edit"). This really improves readability, but the obvious cost is having to click "edit" to update fields is annoying. This needs improved.
  • The top part of the bug is using CSS3(ish) columns, so you will see 2 or 3 columns depending on how wide the browser is. Or, uhh, 1 column for narrow windows. And browsers that don't support -moz-column-width. There's not much control over where a column starts, so tweaking a design to work with different column counts is tricky (the mockup just happens to look fairly reasonable). Perhaps there are JS/DOM tricks that could help? I consider this a failed experiment as-is.
  • The CC list is hidden again (ala mconnor's mockup).
  • Combined some fields