Closed Bug 92253 Opened 23 years ago Closed 22 years ago

Boolean chart makes very wide web pages from query.cgi

Categories

(Bugzilla :: Query/Bug List, defect)

2.13
defect
Not set
trivial

Tracking

()

RESOLVED FIXED
Bugzilla 2.18

People

(Reporter: bugzilla-mozilla, Assigned: endico)

References

Details

Attachments

(2 files, 3 obsolete files)

It is difficult to quickly review what is currently in a boolean chart with 
lots of OR operators, as the page generated is too wide to fit in a window. It 
would be more readable if each OR term was on a different row, but more tightly 
packed than the AND terms.

Something roughly along the lines of the way it is shown in queryhelp.cgi.

I patched query.cgi our local bugzilla, and the results seem to look ok to me.
Keywords: patch
in theory this is good because it makes it fit on the page, however, making this 
change blows away the "obvious order of precidence" and makes the precidence not 
quite so obvious anymore...  (you notice that AND already went below and not next 
to, this is because OR has a higher precidence - with both AND and OR being 
below, it'll be hard to tell which takes higher precidence.
hmm, maybe this will...  AND starts two rows, OR only starts one...   let my 
throw it up somewhere and look at it :)
targetting all bugs with patches so they show up on the radar.
Target Milestone: --- → Bugzilla 2.16
Mass moving to new product Bugzilla...
Assignee: justdave → endico
Component: Bugzilla → Query/Bug List
Product: Webtools → Bugzilla
Version: Bugzilla 2.12 → 2.13
I think we don't want to do it the way this patch does it; removing patch
keyword until Dave thinks of a better solution.

Gerv
Keywords: patch
Just to remove ambiguity, this is how the boolean chart actually looks with
this patch applied. Note that the precedence of AND and OR is still clear, but
the page is all viewable at once.
We are currently trying to wrap up Bugzilla 2.16.  We are now close enough to
release time that anything that wasn't already ranked at P1 isn't going to make
the cut.  Thus this is being retargetted at 2.18.  If you strongly disagree with
this retargetting, please comment, however, be aware that we only have about 2
weeks left to review and test anything at this point, and we intend to devote
this time to the remaining bugs that were designated as release blockers.
Target Milestone: Bugzilla 2.16 → Bugzilla 2.18
Comment on attachment 43520 [details] [diff] [review]
Patch to make complex boolean charts less wide (standard width)

this won't apply due to templatisation.

justdave: You never followed up your comment #3 about whether or not you wanted
this.
Attachment #43520 - Flags: review-
What I said was "let's throw this up somewhere and see what it looks like."  Now
we don't have a patch that applies so I can't throw it up somewhere.  However,
Stephen was kind enough to post an HTML mockup, and I really like the way it
works.  Now we just need to get a patch that applies to the tip.
Glancing at the current code, I'd suggest that this would be done in the
template as in the attached hand-written patch.

I checked the patch applies, but since I don't have 2.16 up and running yet, I
can't verify the functionality. Seems straightforward enough that there can't
possibly be any errors though........
Attachment #43520 - Attachment is obsolete: true
Keywords: patch, review
Found this didn't work when first applied...

... then noticed that I was getting lots of [Or] buttons rather than the Or
text, because the conditional test in the template is wrong.

This patch fixes that fault (and also the same thing with the [And] buttons) as
well as modifying the layout. Modified layout is essentially the same as in
attachment 57575 [details].
Attachment #99049 - Attachment is obsolete: true
Depends on: 164003
Attached patch Patch v.4Splinter Review
Never has such a small tweak produced such a big UI improvement.

This applies to the tip.

Gerv
Attachment #104479 - Attachment is obsolete: true
myk, dave: is this worth checking in now? It really improves the UI, and it's a
six-character change :-)

Gerv
Comment on attachment 105648 [details] [diff] [review]
Patch v.4

I had my doubts at first, but the positioning of the word "And" is enough
different that it still makes it clear that the entire group of ORs is lumped
together before the AND.

r= justdave
a= justdave
Attachment #105648 - Flags: review+
Fixed.

Checking in template/en/default/search/boolean-charts.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/search/boolean-charts.html.tmpl,v
 <--  boolean-charts.html.tmpl
new revision: 1.3; previous revision: 1.2
done

Gerv
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.