Code review comment for lp:~gary-wzl77/ubuntu-calendar-app/fix-1428036

Revision history for this message
Gary.Wang (gary-wzl77) wrote :

> In en_GB this is pretty tight on a low resolution display like the bq e4.5.
>
> Here's the current calendar app:-
> http://people.canonical.com/~alan/screenshots/device-2015-06-02-120425.png
>
> This is what it looks like with your change:-
> http://people.canonical.com/~alan/screenshots/device-2015-06-02-120540.png
>
> Could be even more tight in other languages.

Quick review on zh_CN on bq e4.5
Yeah, same case for zh_CN, pretty tight.Almost no space for each items on the header.
hmm.......
I think it gonna be quite bad to check on this according to different locale.
However current implementation was broken in several locale.
Any ideas?

« Back to merge proposal