104340 - showdependencytree.cgi: toolbar UI design

Status: RESOLVED FIXED

(Bugzilla :: User Interface, enhancement)

Description

[Andreas Franke (gone)]

I'd prefer the following UI:

[Hide Resolved] Max depth: [1] [<] [_3_] [>] [Unlimited]

You don't need a submit button for the number field because you can submit it
with just pressing return. Maybe you can even completely leave out the text
input field:

[Hide Resolved] Max depth: [1] [<] [>] [Unlimited]

Let the discussion begin.

Comment 1

[Christian Reis]

Hmmm. I like that design. A lot, even.

There is one problem: Lynx won't handle the input without a submit button. That
problem can be worked around with a <noscript> tag around the submit.

I've changed it at 

http://bugs.async.com.br/showdependencytree.cgi?id=172&maxdepth=2&hide_resolved=1

And I must confess it looks _much_ better.

Comment 2

[Christian Reis]

Attachment: kiko_v1: implement proposed UI

Comment 3

[Dave Miller [:justdave]]

Comment on attachment 53747 [details] [diff] [review]
kiko_v1: implement proposed UI

r= justdave

verified it works in Mozilla, iCab, and Lynx.

Comment 4

[Gervase Markham [:gerv]]

Comment on attachment 53747 [details] [diff] [review]
kiko_v1: implement proposed UI

Seems to work. r=gerv.

Gerv

Comment 5

[Jacob Steenhagen]

/cvsroot/mozilla/webtools/bugzilla/showdependencytree.cgi,v  <-- 
showdependencytree.cgi
new revision: 1.11; previous revision: 1.10
done

Comment 6

[Marcus Pallinger]

I updated bugzilla recently (it has version 1.11 of showdependencytree.cgi)

When I click on a button, I end up at www.bugzilla.com.
Looking at the source code I see:

 <form method="get" action="//bugzilla/showdependencytree.cgi" 
        style="display: inline; margin: 0px;">

Is this related to these changes or should I file a new bug? (I havn't used the
depencency tree since Bugzilla 1.12, so I'm nto sure if this change was the one
that broke it) 

bugzilla on OpenBSD 2.8

Comment 7

[Christian Reis]

That's quite wierd. It means, basically, that $scriptname is getting
set to //bugzilla/showdependencytree.cgi - and that information itself
comes from the CGI %ENV environment var. So it's not a problem with
showdependencytree, afaict.

I wonder if this happens because you are specifying a bugzilla root of
/bugzilla which is being compounded to a default "/" inserted
somewhere? What did you change to get the /bugzilla prefix working?

At any rate, this doesn't have much to do with this bug, I think, so
feel free to file another one, or research more. If you like you can CC
me on this.