bottom edge first time wizard is being incorrectly displayed on dialer-app and messaging-app

[Tiago Salem Herrmann]

A new bottom edge wizard is being displayed in the recent images, but there are cases where it is being displayed when it shouldn't.

Steps to reproduce:

1) Flash the phone with --wipe.
2) Before opening dialer-app, lock the phone and receive a call.
3) Once you accept the call, dialer-app is launched and instead of the live call view, the wizard is displayed on top of it, but there is no bottom edge on that screen, and I believe in this case we should not display any wizards, specially because the phone is locked, but even if the phone was unlocked, we should not display the wizard if the view does not have bottom edge enabled.

Another case is when launching messaging-app or dialer-app in a dual sim environment. We have a first time dialog that appears behind the wizard. Not sure if that's expected, but seems to cause a bad user experience to me.

[Michael Terry (mterry)]

I can't reproduce the live call problem. For me, the bottom edge hint waits until the phone call is over. However, there ARE about 2 seconds after the phone call ends and the hint appears before the live call view switches to the normal dialer view. For those three seconds, the bottom edge of the dialer-app isn't active.

Is that what you're seeing? Or something new?

So the two problems I know about now:
- 2 seconds after received call ends
- dialer and messaging apps with 2 SIMs

For fixing the 2-second problem, I could do a hack like "wait 2 seconds before showing bottom edge for dialer" but that creates more problems. The user can enter the settings screen in those two seconds. Which also doesn't have a bottom edge. That's why the bottom hints come up as soon as you open the app for the first time, it is the only time we know what's on the screen (at least that was the hope).

For fixing the 2-SIM problem, we could probably do a hack that skips the bottom edge completely on those two apps if you have 2 SIMs in. But that also kind of sucks, since those users would probably like the hint too. And it gets into brittle cross-domain assumptions between the app and the shell.

So I guess we very quickly came to the breaking point on the shell being able to fake knowing when to show bottom edge hints.

Long term, we wanted to bring these hints into the apps anyway. That's still the best fix for this. So we can either speed up that plan, or implement some quick-and-dirty communication mechanism between these apps and the shell for when they have an available bottom edge. Maybe as simple as exposing a bool on a DBus interface that the shell can listen to. Just as a stop-gap to moving the screens.

I'll talk with Design about where we are with that plan.

[Launchpad Janitor (janitor)]

This bug was fixed in the package unity8 - 8.12+16.10.20160617-0ubuntu1

---------------
unity8 (8.12+16.10.20160617-0ubuntu1) yakkety; urgency=medium

  [ Michael Terry ]
  * Do not create Label/Icon if we don't need it
  * Fix Z-ordering of prompt surfaces (LP: #1586219)
  * Drop bottom edge tutorials completely and small fixes for other
    edges. (LP: #1565103)
  * Redo visuals for greeter prompt box. (LP: #1435923)
  * Make tutorial qmltests more robust. (LP: #1590810)
  * Stop showing an extra lockscreen in desktop unity8 sessions. (LP:
    #1582987)

  [ Albert Astals Cid ]
  * Improve dash plugin mock
  * Stabilize PreviewExpandableTest::test_all_widgets_height
  * Workaround QTBUG-53460
  * Split artShape code for cardTool card and regular card
  * Simplify the code since fixedArtSize code is mandatory for regular
    cards
  * Desktop stage: Support rotatesWindowContents (LP: #1547481)
  * Turn if if else into if elseif else
  * Fix leaks in QInputDeviceManagerPrivate::addDevice
  * Do not call |= on uninitialized sigterm.sa_flags
  * initialize udevMonitor
  * Make notifier and notifierFd function local variables
  * Fix crash if a component that is not an Item is given to
    sectionDelegate
  * Tests: Do not use dynamic_cast if we're not going to test it anyway
  * Minor coverity fixes
  * Fix warning about anchors.fill in ProportionalShape (LP: #1590963)
  * Make wait in cleanup() wait for snapTo animation to have finished

  [ Alexandros Frantzis ]
  * Improve the way notifications interact with power management (LP:
    #1570922)

  [ Andrea Cimitan ]
  * Correctly refresh cardcreator cache on artshapestyle changes (LP:
    #1588335)
  * Same tweaks we have for click scope now for libertine
  * add a timer to pull to refresh to work on small windows (LP:
    #1589060)

  [ Daniel d'Andrada ]
  * Clean up debug leftovers
  * Ensure mouse and window movement are pixel-aligned (LP: #1510382)
  * Fix SpreadDelegate rotation animations
  * No point in version-controlling files that are automatically
    generated
  * Center cursor on screen when it's first shown
  * Custom cursor name just have to start with "custom"
  * Work around AnimatedSprite infinite loop bug (LP: #1588929)

  [ Josh Arenson ]
  * Disable showDash when it is already shown. (LP: #1506708)
  * Add a timestamp to the dash log (LP: #1580074)
  * Add timestamp to unity8 log (LP: #1498169)

  [ Lukáš Tinkl ]
  * Implement keyboard shortcuts info panel
  * Implement window controls overlay activatable with 3 fingers tap
    (LP: #1489020)
  * Select correct Chinese input method for OSK in the wizard (LP:
    #1588633)

  [ Michael Zanetti ]
  * Add support for the launcher surface pips displaying the correct
    number of surfaces
  * Add support for the persistent alert state. (LP: #1575147, #1584348,
    #1586621)
  * send a Alt-release event on alt+tab instead of delaying it
    completely and invoking it later (LP: #1565236)

  [ Stephen M. Webb ]
  * added a new upstart $SESSION for unity8-desktop-session (LP:
    #1376715)

 -- Michael Terry <email address hidden> Fri, 17 Jun 2016 01:22:57 +0000