ubuntu-touch-coreapps-reviewers team mailing list archive
-
ubuntu-touch-coreapps-reviewers team
-
Mailing list archive
-
Message #00407
Re: [Merge] lp:~nik90/ubuntu-clock-app/fix-empty-state-wrap into lp:ubuntu-clock-app
> Looks fine to me. I just have one suggestion which I've added to the diff.
> That being said, I'll happily approve this if you apply that small change. :)
Thanks for the review. Originally I chose not to put wrapMode for title since it is supposed to be short and discourage 3rd party devs from making it too long. But I guess when you translate it, there is the chance it could need more space.
--
https://code.launchpad.net/~nik90/ubuntu-clock-app/fix-empty-state-wrap/+merge/251924
Your team Ubuntu Clock Developers is subscribed to branch lp:ubuntu-clock-app.
References