Code review comment for lp://qastaging/~charlesk/indicator-datetime/lp-1295799-date-menuitem-should-open-calendar
Revision history for this message
| Charles Kerr (charlesk) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
bbfa235
demo site
(Get the code!)
> * Ensure the project compiles and the test suite executes without error
Done on desktop + nexus phone.
> * Ensure that non-obvious code has comments explaining it
No new non-obvious code. The per-profile actions are actually more readable now.
> * If the change works on specific profiles, please include those in the merge description.
Changes phone behavior s.t. clicking on the date menuitem pops up the calendar app.