1000097 - UI issues when tiles are dragged out of the tab then resize (tiles locked at old size/position)

Status: RESOLVED WONTFIX

(Firefox :: New Tab Page, defect)

Description

[Cornel Ionce [:noni]]

User Agents:
Mozilla/5.0 (Windows NT 6.3; rv:31.0) Gecko/20100101 Firefox/31.0
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Firefox/31.0
Mozilla/5.0 (X11; Linux i686; rv:31.0) Gecko/20100101 Firefox/31.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:31.0) Gecko/20100101 Firefox/31.0

Build ID: 20140422030204

While verifying bug 994185 I've encountered some issues for the Newtab page tiles. I've attached a video showing them.

STR:
1. Open Firefox.
2. Open a new tab or go to about:newtab.
3. Resize the window until one row/column disappear.
4. Drag the last downwards, under the last row.

Results:
- several issues can be seen; tiles will overlap into the last position.
- scrollbars are shown.
- after maximizing the window, the tiles are misplaced.

Notes:
This started to reproduce on 2014-03-26 Nightly, when the fix for 980014 landed.

Comment 1

[Cornel Ionce [:noni]]

Added the video to the URL, could not attach it due to it's size.

Comment 2

[u279076]

> This started to reproduce on 2014-03-26 Nightly, when the fix for 980014 landed.

Marking this is a regression from bug 980014. Based on what I've seen this is a bit of an edge-case so I don't think this should block the feature from shipping. However we should address this in Firefox 32 Beta if possible. We should also pay attention to user feedback in release in case we need to pref this off if it becomes explosively negative.

Comment 3

[Sylvestre Ledru [:Sylvestre]]

Wontfix for 31 but it is low risk, we could take a patch for ESR31.

Comment 4

[Ed Lee :Mardak]

Attachment: v1 backout

Comment 5

[Lawrence Mandel [:lmandel] (use needinfo)]

Ed - I think we want a separate backout bug as this bug will still need to be fixed on 32.

Comment 6

[Ed Lee :Mardak]

Comment on attachment 8456446 [details] [diff] [review]
v1 backout

Created bug 1038997 for the beta backout.

Comment 7

[Sylvestre Ledru [:Sylvestre]]

Not a regression: 30 was affected too...

Comment 8

[u279076]

The overlapping tiles here was caused by something other than bug 980014 since I can reproduce this in Firefox 30. Cornel, I think this needs further testing and each issue should be broken out into their own bug. We need to figure out when each individual issue regressed (if they are indeed regressions).

FWIW, I think this investigation should occur in parallel to releasing Firefox 31 as is.

Comment 9

[Ed Lee :Mardak]

ashughes helped explain how to reproduce this on 31RC2 and 30; and because it reproduces on 30, it's not a regression from bug 980014.

The simplest STR:
1) make sure there's at least 2 tiles on the newtab page
2) drag a tile out of the window to another window (console, text, browser, etc.)
3) resize the window and see some tiles locked at the before-resize size

My best guess right now is that step 2 initiates a drag but doesn't trigger the end of the drag, so that newtab page still marks the tiles as "locked" (in position and size).

Comment 10

[Sylvestre Ledru [:Sylvestre]]

Too late for 31.

Comment 11

[:Gavin Sharp [email: [e-mail]]]

Not a regression, and only visible in certain unusual dragging scenarios - I think we shouldn't be tracking this for upcoming releases.

Comment 12

[Cornel Ionce [:noni]]

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:13.0) Gecko/20100101 Firefox/13.0

After further investigations I've notice that these issues started to reproduce way back to Firefox 13 (build ID: 20120601045813), when the Tile Grid was introduced on the New Page.

Comment 13

[David Hinojosa [:LMS]]

I'm working on bug 1068354
the mozregession is the same, so I will put this information here aswell.

2013-07-24 = bad
2013-07-23 = good

http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=5ceea82a79c7&tochange=a4c1961bf723

Comment 14

[Benjamin Kerensa [:bkerensa]]

No longer tracking this as it doesn't meet criteria and has shipped with little impact to our users.

Comment 16

[[:Towkir] Ahmed]

(In reply to Cornel Ionce [QA] from comment #1)
> Added the video to the URL, could not attach it due to it's size.

As the bug I said a duplicate of this one, still adding a video url here to make it clear what are these issus exactly, 

please compare 
http://screencast.com/t/NuclmbZm 
and 
https://vimeo.com/130934210

Thanks

Comment 17

[Jessilyn Davis]

Hello!
This bug has been closed due to inactivity and/or the potential for this bug to no longer be an issue with the new Discovery Stream-powered New Tab experience.
Please help us triage by reopening if this issue still persists and should be addressed.
Thanks!