Closed
Bug 969335
Opened 10 years ago
Closed 7 years ago
Defect - Deleting all the Top Sites leaves bottom bar unable to close
Categories
(Firefox for Metro Graveyard :: App Bar, defect)
Tracking
(Not tracked)
RESOLVED
INCOMPLETE
People
(Reporter: firefox1, Unassigned, Mentored)
References
Details
(Keywords: polish, ux-control, Whiteboard: p=0 [lang=js])
User Agent: Mozilla/5.0 (Windows NT 6.2; WOW64; rv:28.0) Gecko/20100101 Firefox/28.0 (Beta/Release) Build ID: 20140203004003 Steps to reproduce: 1. Ater visiting several sites I went back to start page (with Top Sites on). 2. Firefox had recorded my top sites but I didnt want them so I proceeded to delete them. I selecting them with the "pull down touch" and clicked the delete from the bottom bar. 3. Additional ones appeared to replace the ones I deleted (as is normal I think) but eventually I was left with none. Actual results: All sites were deleted but the bottom bar still remained with an "Undelete" button. I could not close the bar, even by clicking somewhere else on the page. All I could do was press undelete then reselect a site and then unselect it. The bar disappeared then but I was still left with sevwral sites. Expected results: The bar should close when the screen is tpuched outside of the bar - or there should be a close button.
Reporter | ||
Updated•10 years ago
|
Severity: normal → minor
OS: All → Windows 8
Updated•10 years ago
|
Blocks: metrov1backlog
Whiteboard: [triage] p=0
Updated•10 years ago
|
Whiteboard: [triage] p=0 → p=0
Comment 1•10 years ago
|
||
We should add a close button to the bar to allow the user to explicitly commit to the action and dismiss the bar
Comment 2•10 years ago
|
||
We'll need to add a close button here: http://hg.mozilla.org/mozilla-central/file/d05c721ea1b0/browser/metro/base/content/browser.xul#l366 and some code to go with it, and decide under what conditions it should be shown. We could show it only when showing the "restore" button, e.g.: http://hg.mozilla.org/mozilla-central/file/d05c721ea1b0/browser/metro/base/content/startui/BookmarksView.js#l240
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: polish,
ux-control
Whiteboard: p=0 → p=0 [mentor=sfoster@mozilla.com][lang=js]
Comment 3•10 years ago
|
||
related to bug 877875. There's some discussion there about tapping empty space to dismiss context bar too.
Assignee | ||
Updated•10 years ago
|
Mentor: sfoster
Whiteboard: p=0 [mentor=sfoster@mozilla.com][lang=js] → p=0 [lang=js]
Comment 4•7 years ago
|
||
Mass close of bugs in obsolete product https://bugzilla.mozilla.org/show_bug.cgi?id=1350354
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•