date & time dialog not modal, can't be closed in some cases

Bug #213364 reported by Kyle Nitzsche
6
Affects Status Importance Assigned to Milestone
Moblin UI Framework
Fix Released
High
Bob Spencer

Bug Description

Date and time launched from the marquee should be a modal dialog.

If not, then the user can find themselves in a case where they cannot easily close date and time.

This occurs in the acton project because the marquee only displays when the home flash plugin does not have focus and because you close the date and time dialog from the marquee, as follows:

 * user launches an app, marquee displays
 * user clicks date and time on marquee, it launches
 * Since date and time is not modal, user clicks the marquee close application button
 * Focus returns to home flash plugin, marquee disappears, but the date and time dialog remains displayed, but with no way to close it

Possible fixes include:
 * make date and time modal
 * add a close button to date and time

Revision history for this message
cathyli (cathy-li) wrote :

The issue only happened on Ubuntu Hardy, right? If so, pls. file bug against LP Ubuntu as following steps:
1. Go to https://launchpad.net/mobileprocessing/trunk
2. Click “Link to Ubuntu package” on the left sidebar
3. Link a package you want to file bug.
   The package list is under:
   https://launchpad.net/~ubuntu-mobile/+archive
4. Go to https://launchpad.net/mobileprocessing
5. Click button [Report a bug] and go to
   https://launchpad.net/mobileprocessing/+filebug
6. Click project link to file bug as usual
7. After file a new one, you need add tag “ume” and subscribe “ubuntu-mobile” to the bug

Changed in moblin-applets:
status: New → Invalid
Jing Wang (jing-j-wang)
Changed in moblin-applets:
assignee: nobody → todd-e-brandt
importance: Undecided → Low
status: Invalid → New
Revision history for this message
ToddBrandt (todd-e-brandt) wrote :

Switching owner to Bob since this is the marquee clock, not the moblin applet dialog.

Changed in moblin-applets:
assignee: todd-e-brandt → bob-spencer
Horace (horace-li)
Changed in moblin-applets:
status: New → Triaged
Changed in moblin-applets:
importance: Low → High
milestone: none → m11
Revision history for this message
Tony Espy (awe) wrote :

Changed status and importance based on yesterday's triage meeting.

Changed in acton:
importance: Undecided → High
status: New → Triaged
Revision history for this message
Tony Espy (awe) wrote :

Bob, can you add an estimate for when you think this bug might be fixed?

Horace (horace-li)
Changed in moblin-applets:
milestone: m11 → none
Changed in moblin-ui-framework:
milestone: none → m11
Revision history for this message
Horace (horace-li) wrote :

Hi, All,

I adopted the second fix suggestion that adding a 'Close' button on date & tiem dialog, considering setting dialog to modal might have side-effect that it will then never be closed once it is open.

The patch is attach here, generated based on latest 0.22-0ubuntu1.

Thanks,
Horace

Revision history for this message
Tony Espy (awe) wrote : Dialog still needs to be modal...

Horace, I don't think just adding a "Close" button is the right fix. The problem with non-modal dialogs is that they can be hidden by other dialogs and/or applications ( eg. the user clicks the home button ). Once this happens, it leaves the user very confused.

After talking to Kyle, we both think it's not a bad idea to add a "Close" button, but the dialog still needs to be made "modal".

Revision history for this message
Horace (horace-li) wrote :

Hi, Tony,

Date & Time dialog is now set to modal, and new marquee-plugins 0.22-0ubuntu3 is put to hardy PPA.

Thanks,
Horace

Revision history for this message
Tony Espy (awe) wrote :

We'll pick this up next week as part of sync for our Beta2 delivery.

Revision history for this message
Horace (horace-li) wrote :

Following two suggestions, setting dialog modal and add a 'Close' button on the dialog as well. new changes have been put to Hardy PPA.

Changed in moblin-ui-framework:
status: Triaged → Fix Committed
Revision history for this message
cathyli (cathy-li) wrote :

Verified in mobile-basic-flash 0.44-0ubuntu2, the issue is fixed, close it.

Changed in moblin-ui-framework:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.