178043 - In graphical reports, provide mechanism for user to tweak presentation of report

Status: RESOLVED FIXED

(Bugzilla :: Reporting/Charting, enhancement, P4)

Description

[Joel Peshkin]

In initially using the graphical reports, feature, many of the charts are a few
tweaks away from being ready to publish.  GD::Graph has a huge set of options
available that impact the presentation that may need to be tweakable on a
per-chart basis.  Among those are....

x_labels_vertical
show_values
supress_angle

and the list goes on....

Ideally, we should be able to keep the existing behavior, but permit a user to
tweak-and-redisplay (in as generalized a manner as possible) a wide set of GD
options.  Continuing on the "ideal" wish-list, we should be able to either
bookmark the settings with the query or store the settings in a way that they
can be applied to any query.

Comment 1

[Gervase Markham [:gerv]]

Blimey... some people are never satisfied :-)

We could implement a URL tweaking mechanism with parameters of the same name as
the GD::Graph parameters. Then, sites who wished to expose particular
preferences could customise the wrapper template to have a form which allowed
tweaking, and submitted to the same URL but with the tweaks. 

But I don't think the default Bugzilla UI should have any options like that -
IMO, the charts are perfectly publishable as they are, and I have worked hard to
keep the chart UI simple :-)

Gerv

Comment 2

[Joel Peshkin]

heh.. that is why I put it as a priority 4 and did not specify a target milestone.

The main place I notice that some user-driven tweaking could help is in charts
by reporter where the names can land on top of each other.

Comment 3

[Gervase Markham [:gerv]]

> The main place I notice that some user-driven tweaking could help is in charts
> by reporter where the names can land on top of each other.

You mean if reporter is plotted along the bottom? Skipping out some names would
make the chart not make sense, so ideally we'd have them in two or more rows.
However, I can't see an option for that - is there one?

Gerv

Comment 4

[Joel Peshkin]

WRT comment 3
I think that x_labels_vertical will do just that, though I think I just
volunteered to try it :-)

Comment 5

[Joel Peshkin]

Confirmed,
            x_labels_vertical => 1,      
does make bar charts with reporter as the x-axis work.

Comment 6

[Gervase Markham [:gerv]]

Better than universal tweakability, then, would be a hard-coded conversion to
vertical x-labels for particular fields - all the email address ones, for a
start. Simple, yet solves 95% of the problem. How does that sound?

Gerv

Comment 7

[Joel Peshkin]

Vertical is likely to be popular for product and component as well.  It might be
best to provide some mechainism to specifically indicate that you want the
X-axis to use vertical labels.

I agree that having the x_labels_vertical selectable covers 80% of the need.
(Another 10% is show_values on pie charts).

If we made it a checkbox that lives clearly below the graph-selection knobs but
clearly above the query-data-selection form area, then we would have a natural
place to have a chunk of stuff that permits customization of the display
formatting options.

Comment 8

[Christian :Biesinger (don't email me, ping me on IRC)]

is it part of this bug report to add a feature to rotate the x axis labels at
http://bugzilla.mozilla.org/report.cgi?bug_file_loc_type=allwordssubstr&bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&bugidtype=include&chfieldto=Now&emailassigned_to1=1&emailassigned_to2=1&emailqa_contact2=1&emailreporter2=1&emailtype1=exact&emailtype2=exact&keywords_type=allwords&long_desc_type=substring&short_desc_type=allwordssubstr&status_whiteboard_type=allwordssubstr&type0-0-0=noop&y_axis_field=op_sys&action=wrap&format=bar&width=700&height=350
by 90 degrees so they can actually be read?

Comment 9

[Gervase Markham [:gerv]]

Yes; that is the most commonly-requested feature, and I'll try and make that
happen as soon as possible.

Gerv

Comment 10

[Gervase Markham [:gerv]]

Attachment: Patch v.1

Adds "x_labels_vertical" to the options available.

Gerv

Comment 11

[Joel Peshkin]

Comment on attachment 105888 [details] [diff] [review]
Patch v.1

r=joel

I still suspect that this will wind up with a big bloat of features later, but
this certainly helps now.

Comment 12

[Gervase Markham [:gerv]]

That worries me too, but I think this one is important. I'm going to close this
bug now, and people can request particular extra features in new bugs.

Checking in template/en/default/search/search-report-graph.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/search/search-report-graph.html.tmpl,v
 <--  search-report-graph.html.tmpl
new revision: 1.2; previous revision: 1.1
done
Checking in query.cgi;
/cvsroot/mozilla/webtools/bugzilla/query.cgi,v  <--  query.cgi
new revision: 1.112; previous revision: 1.111
done
Checking in report.cgi;
/cvsroot/mozilla/webtools/bugzilla/report.cgi,v  <--  report.cgi
new revision: 1.8; previous revision: 1.7
done
Checking in template/en/default/reports/report-bar.png.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/reports/report-bar.png.tmpl,v
 <--  report-bar.png.tmpl
new revision: 1.2; previous revision: 1.1
done
Checking in template/en/default/reports/report-line.png.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/reports/report-line.png.tmpl,v
 <--  report-line.png.tmpl
new revision: 1.2; previous revision: 1.1
done

Gerv