Closed Bug 964285 Opened 10 years ago Closed 10 years ago

"blank" tiles under about:start shouldn't be selectable


(Firefox for Metro Graveyard :: Browser, defect, P2)

Windows 8.1


(Not tracked)

Firefox 29


(Reporter: kjozwiak, Assigned: sfoster)



(Whiteboard: [defect] p=1)


(1 file, 1 obsolete file)

When you swipe and select a "blank" tile from the about:screen, the "Selection App Bar" will not slide into view making it very difficult to unselect them after. I tried selecting several valid tiles and then taping on "clear selection", but the selected "blank" tiles didn't clear. I had to press "F5" to refresh the entire screen and deselect everything.

- I've attached a screenshot to illustrate the issue

Steps to reproduce the issue:

1) Open Firefox Metro
2) Under the about:start screen, select several "blank" tiles by using the swipe gesture (up/downs swipe)
3) You'll notice that once the tiles are selected, it's really difficult to clear them unless you refresh the entire page

Current Behavior:

- If the user selects "blank" tiles from the about:start screen, they can't clear it through the "Selection App Bar"

Expected Behavior:

- I don't think the user's should be able to select tiles if they are completely blank. If the changes are to difficult, we should at least let the users clear the selections.

Found the issue using the following builds:
Yeah, we shouldn't allow selection of blank tiles. You can't select them with mouse, only with touch.
Summary: difficult to unselect "blank" tiles that are selected under about:start → "blank" tiles under about:start shouldn't be selectable
Assignee: nobody → sfoster
Blocks: metrov1it23
No longer blocks: metrov1backlog
Whiteboard: [triage] [defect] p=0 → [defect] p=1
Priority: -- → P2
QA Contact: jbecerra
I've replaced a few ad-hock checks with an isItem validation method on richgrid, and a similar sanity check in CrossSlide.jsm. New tests that pass - as do the existing ones. 
If this is nervous-making, we could possibly break out the CrossSlide.jsm change to make a lower-risk fix for uplift?
Attachment #8365974 - Attachment is obsolete: true
Attachment #8368799 - Flags: review?(mbrubeck)
Attachment #8368799 - Flags: review?(mbrubeck) → review+
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 29
Mozilla/5.0 (Windows NT 6.3; WOW64; rv:29.0) Gecko/20100101 Firefox/29.0

Vetified as fixed with a Surface Pro 2 device on latest Aurora (build ID: 20140206004003) using the STR from the description.
You need to log in before you can comment on or make changes to this bug.