Code review comment for lp:~pkunal-parmar/ubuntu-calendar-app/TimeLineView

Revision history for this message
Olivier Tilloy (osomon) wrote :

232 + // FIXME: how to represent
233 + text: index+":00"

This should be localized, probably something like this (not sure it would work for all possible regional settings, but should be better than nothing):

    text: i18n.tr("%1:00").arg(index)

What exactly was the FIXME about?

review: Needs Fixing

« Back to merge proposal