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
=== modified file 'errors/templates/bucket.html'
--- errors/templates/bucket.html 2016-03-16 13:31:52 +0000
+++ errors/templates/bucket.html 2016-03-16 21:29:55 +0000
@@ -210,7 +210,7 @@
210 /* Set up the examples table. */210 /* Set up the examples table. */
211 var cols = [211 var cols = [
212 {key: "timestamp", label: "Time", formatter: formatDates},212 {key: "timestamp", label: "Time", formatter: formatDates},
213 {key: "example", label: "Example", formatter: formatExample, allowHTML: true},213 {key: "example", label: "Incident", formatter: formatExample, allowHTML: true},
214 {key: "package_version", label: "Package version"},214 {key: "package_version", label: "Package version"},
215 {key: "ubuntu_version", label: "Ubuntu release"},215 {key: "ubuntu_version", label: "Ubuntu release"},
216 {key: "architecture", label: "Architecture"},216 {key: "architecture", label: "Architecture"},
@@ -313,7 +313,7 @@
313 {{ traceback|pygmentize:"pytb"|safe }}313 {{ traceback|pygmentize:"pytb"|safe }}
314 {% endif %}314 {% endif %}
315{% endif %}315{% endif %}
316<h3>Examples</h3>316<h3>Occurrences</h3>
317<div id="examples_group">317<div id="examples_group">
318 <div id="examples"></div>318 <div id="examples"></div>
319 <div id="spinner"></div>319 <div id="spinner"></div>

Subscribers

People subscribed via source and target branches

to all changes: