← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

Re: [Merge] lp:~mzanetti/reminders-app/cleanup-toolbars into lp:reminders-app

 

> > I still would keep the openAccountsPage() in reminders.qml because when
> there
> > is no account set up, I want to push the AccountsPage to the stack, so the
> > user can't switch to the other tabs.
> 
> Makes sense, but then you can drop the showBackButton property. If the
> AccountsPage is used only when there isn't any account set up, then the
> backButton is always hide (and you can simply hide it, and simplify the code
> about the backButton in the AccountPage).

Thanks! fixed.
-- 
https://code.launchpad.net/~mzanetti/reminders-app/cleanup-toolbars/+merge/251347
Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app.


References