Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(288)

Issue 9098043: TRIVIAL: Adds pluralize to search results header

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 5 months ago by j.c.sackett
Modified:
11 years, 5 months ago
Reviewers:
mp+161926
Visibility:
Public.

Description

TRIVIAL: Adds pluralize to search results header Uses the pluralize template helper to get the search results header to properly output "result" as plural or not. https://code.launchpad.net/~jcsackett/juju-gui/pluralize-search-results-header/+merge/161926 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : TRIVIAL: Adds pluralize to search results header #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -1 line) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M app/subapps/browser/templates/search.handlebars View 1 chunk +3 lines, -1 line 0 comments Download

Messages

Total messages: 3
j.c.sackett
Please take a look.
11 years, 5 months ago (2013-05-01 19:18:54 UTC) #1
j.c.sackett
LGTM - Self reviewing trivial.
11 years, 5 months ago (2013-05-01 19:20:16 UTC) #2
j.c.sackett
11 years, 5 months ago (2013-05-01 19:36:49 UTC) #3
*** Submitted:

TRIVIAL: Adds pluralize to search results header

Uses the pluralize template helper to get the search results header to
properly output "result" as plural or not.

R=
CC=
https://codereview.appspot.com/9098043
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b