Note- this functionality does not exist in tef please remove a=tef+ from
your commit.


On Mon, May 6, 2013 at 3:06 PM, Kevin Grandon <notifications@github.com>wrote:

> In apps/calendar/js/templates/alarm.js:
>
> > @@ -45,7 +45,19 @@
> >
> >        // Format the display text based on a zero-offset trigger
> >        if (this.arg('layout') == 'allday') {
> > -        trigger -= MORNING;
> > +        var options = layouts['allday'];
> > +        var i = 0;
> > +        var iLen = options.length;
> > +        for (; i < iLen; i++) {
>
> @mozshiao9 <https://github.com/mozshiao9> As this seems to be a fairly
> static operation, what are your thoughts on adjusting the map here:
> https://github.com/mozshiao9/gaia/blob/b723b16d4f44ccf521e8d27d7785d715a2d8eed8/apps/calendar/js/templates/alarm.js#L9
>
> Perhaps it would be cleaner to either specify objects that have a render
> value which can differ from the actual offset, or have two different maps?
> Just thinking out loud..
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mozilla-b2g/gaia/pull/9533/files#r4102812>
> .
>