Good idea, I'll fix that tonight.


On Tue, Mar 18, 2014 at 6:29 PM, Zaahir Moolla <notifications@github.com>wrote:

> In share/spice/bbc/bbc.js:
>
> > @@ -67,14 +67,17 @@ Handlebars.registerHelper("time", function() {
> >          end = new Date(this.end);
> >
> >      function standard_time(time) {
> > -        var hour = time.getHours() % 12;
> > -        if (hour == 0) {
> > -          hour = 1;
> > +        var hour = time.getHours(), ampm = time.getHours() < 12 ? "AM" : "PM";
>
> Actually looking at the old code the only problem was line 72, hour = 1,
> that just needs to be hour = 12 and then you can still add in your ampmvariable.
>
> var hour = time.getHours() %12,
>     ampm = hour < 12 ? "AM" : "PM";if (hour === 0) {
>     hour = 12;}
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/duckduckgo/zeroclickinfo-spice/pull/465/files#r10718457>
> .
>