← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

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