ubuntu-touch-coreapps-reviewers team mailing list archive

[Riccardo Padovani <riccardo@xxxxxxxxxxxxx>]

Review: Needs Fixing

> 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).
-- 
https://code.launchpad.net/~mzanetti/reminders-app/cleanup-toolbars/+merge/251347
Your team Ubuntu Reminders app developers is subscribed to branch lp:reminders-app.