Merge lp:~beuno/launchpad/sprite-issues-never-die into lp:launchpad/db-devel

Proposed by Martin Albisetti
Status: Merged
Approved by: Martin Albisetti
Approved revision: not available
Merged at revision: not available
Proposed branch: lp:~beuno/launchpad/sprite-issues-never-die
Merge into: lp:launchpad/db-devel
Diff against target: 38 lines (+3/-3)
3 files modified
lib/canonical/launchpad/icing/style-3-0.css (+1/-1)
lib/lp/registry/templates/products-index.pt (+1/-1)
lib/lp/registry/templates/projects-index.pt (+1/-1)
To merge this branch: bzr merge lp:~beuno/launchpad/sprite-issues-never-die
Reviewer Review Type Date Requested Status
Brad Crittenden (community) code Approve
Review via email: mp+17229@code.launchpad.net

Commit message

Fix extraneous sprite icons in error messages.

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

This is a simple fix to avoid the problems with error messages (bug 487142), I just renamed the sprite class, and changed it in the only place that it currently seems in use.

Revision history for this message
Brad Crittenden (bac) wrote :

As we discussed on IRC, please change the use in projects-index.pt.

Thanks,
Brad

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/canonical/launchpad/icing/style-3-0.css'
--- lib/canonical/launchpad/icing/style-3-0.css 2010-01-06 02:02:18 +0000
+++ lib/canonical/launchpad/icing/style-3-0.css 2010-01-12 14:39:18 +0000
@@ -893,7 +893,7 @@
893.person-inactive-badge {background:url(icon-sprites-2) 0 -64px no-repeat;}893.person-inactive-badge {background:url(icon-sprites-2) 0 -64px no-repeat;}
894.mentoring {background:url(icon-sprites-2) 0 -96px no-repeat;}894.mentoring {background:url(icon-sprites-2) 0 -96px no-repeat;}
895.undecided {background:url(icon-sprites-2) 0 -128px no-repeat;}895.undecided {background:url(icon-sprites-2) 0 -128px no-repeat;}
896.error {background:url(icon-sprites-2) 0 -160px no-repeat;}896.error-icon {background:url(icon-sprites-2) 0 -160px no-repeat;}
897.cancel {background:url(icon-sprites-2) 0 -192px no-repeat;}897.cancel {background:url(icon-sprites-2) 0 -192px no-repeat;}
898.confirm {background:url(icon-sprites-2) 0 -224px no-repeat;}898.confirm {background:url(icon-sprites-2) 0 -224px no-repeat;}
899.flame {background:url(icon-sprites-2) 0 -256px no-repeat;}899.flame {background:url(icon-sprites-2) 0 -256px no-repeat;}
900900
=== modified file 'lib/lp/registry/templates/products-index.pt'
--- lib/lp/registry/templates/products-index.pt 2009-11-10 03:54:59 +0000
+++ lib/lp/registry/templates/products-index.pt 2010-01-12 14:39:18 +0000
@@ -93,7 +93,7 @@
93 </table>93 </table>
9494
95 <div id="application-summary">95 <div id="application-summary">
96 <p class="sprite error"96 <p class="sprite error-icon"
97 tal:condition="not: view/search_results">97 tal:condition="not: view/search_results">
98 No projects matching98 No projects matching
99 &#8220;<span tal:replace="view/search_string">moz</span>&#8221;99 &#8220;<span tal:replace="view/search_string">moz</span>&#8221;
100100
=== modified file 'lib/lp/registry/templates/projects-index.pt'
--- lib/lp/registry/templates/projects-index.pt 2009-11-10 05:05:49 +0000
+++ lib/lp/registry/templates/projects-index.pt 2010-01-12 14:39:18 +0000
@@ -57,7 +57,7 @@
57 </tbody>57 </tbody>
58 </table>58 </table>
5959
60 <p class="sprite error application-summary"60 <p class="sprite error-icon application-summary"
61 tal:condition="not: view/search_results">61 tal:condition="not: view/search_results">
62 No project groups matching62 No project groups matching
63 &#8220;<span tal:replace="view/search_string">moz</span>&#8221;63 &#8220;<span tal:replace="view/search_string">moz</span>&#8221;

Subscribers

People subscribed via source and target branches

to status/vote changes: