Merge lp:~barry/errors/lp1543350 into lp:errors

Proposed by Barry Warsaw
Status: Merged
Merged at revision: 561
Proposed branch: lp:~barry/errors/lp1543350
Merge into: lp:errors
Diff against target: 21 lines (+2/-2)
1 file modified
errors/templates/bucket.html (+2/-2)
To merge this branch: bzr merge lp:~barry/errors/lp1543350
Reviewer Review Type Date Requested Status
Brian Murray Approve
Review via email: mp+289301@code.launchpad.net

Description of the change

I liked Occurrences for the intro to the table, but it didn't look quite right for the column heading, so I chose Incident instead.

The branch can't be run on Xenial so I just loaded a real page and hacked it using chromium developer mode!

To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

Can we have some consistency between the two different labels? I. e. pick "Incident" or "Occurrences".

Revision history for this message
Brian Murray (brian-murray) wrote :

I can see "Occurrences" as a collection of things, so don't mind the different labels since one is for the table and one is for the column.

Revision history for this message
Barry Warsaw (barry) wrote :

Concur with Brian.

Revision history for this message
Brian Murray (brian-murray) wrote :

I decided to rename the example key to make grep'ing for stuff easier but that was a whole of changes so I need to test it tomorrow.

Revision history for this message
Barry Warsaw (barry) wrote :

I thought about that, but I went for a minimal change for exactly that reason.

Revision history for this message
Brian Murray (brian-murray) wrote :

A minimal change would have certainly caused me less pain. I ran into an issue with renaming the examples div and the timestamp column ending up being very wide so I just dropped that.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'errors/templates/bucket.html'
2--- errors/templates/bucket.html 2016-03-16 13:31:52 +0000
3+++ errors/templates/bucket.html 2016-03-16 21:29:55 +0000
4@@ -210,7 +210,7 @@
5 /* Set up the examples table. */
6 var cols = [
7 {key: "timestamp", label: "Time", formatter: formatDates},
8- {key: "example", label: "Example", formatter: formatExample, allowHTML: true},
9+ {key: "example", label: "Incident", formatter: formatExample, allowHTML: true},
10 {key: "package_version", label: "Package version"},
11 {key: "ubuntu_version", label: "Ubuntu release"},
12 {key: "architecture", label: "Architecture"},
13@@ -313,7 +313,7 @@
14 {{ traceback|pygmentize:"pytb"|safe }}
15 {% endif %}
16 {% endif %}
17-<h3>Examples</h3>
18+<h3>Occurrences</h3>
19 <div id="examples_group">
20 <div id="examples"></div>
21 <div id="spinner"></div>

Subscribers

People subscribed via source and target branches

to all changes: