'Thumbnail removed' message overlaps search bar from newtab

VERIFIED FIXED in Firefox 42

Status

()

Firefox
New Tab Page
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: bogdan_maris, Assigned: maxim zhilyaev)

Tracking

(Blocks: 2 bugs, {regression})

unspecified
Firefox 43
regression
Points:
1
Dependency tree / graph
Bug Flags:
qe-verify +

Firefox Tracking Flags

(firefox42 verified, firefox43 verified)

Details

Attachments

(5 attachments)

(Reporter)

Description

3 years ago
Created attachment 8648735 [details]
Screenshot showing the issue

Affected platforms: 
- Windows 7 64-bit
- Mac OS X 10.10.4
- Ubuntu 14.04 32-bit

Affected builds:
- latest Nightly 43.0a1
- latest Aurora 42.0a2

STR:
1. Open Firefox and navigate to about:newtab
2. Reduce the browsers width
2. Delete one thumbnail

Expected results: No visual issues can be seen.

Actual results: The undo menu overlaps Search bar from new tab

Comment 1

3 years ago
Probably made worse from bug 1180387 with it pushing the search bar upwards.
Blocks: 1180387
Keywords: regression

Updated

3 years ago
Blocks: 1195113
Iteration: --- → 42.3 - Aug 10
Points: --- → 1
(Assignee)

Comment 2

3 years ago
(In reply to Ed Lee :Mardak from comment #1)
> Probably made worse from bug 1180387 with it pushing the search bar upwards.

1180387 change was never uplifted to Aurora 42.0a2.
I verified that latest Aurora 42.0a2 build has the problem, and it does not have 1180387 changes.
Since 1180387 is not the root cause, 1180387 should depend on this regression. 
Hence, removing the dependency.
No longer blocks: 1180387
(Assignee)

Comment 3

3 years ago
(In reply to maxim zhilyaev from comment #2)

> Since 1180387 is not the root cause, 1180387 should depend on this

This should read - "1180387 should NOT depend on this regression"

Comment 4

3 years ago
It was made worse by bug 1180387 because before that change, there usually was empty space for the undo message.
Blocks: 1180387
(Assignee)

Comment 5

3 years ago
Created attachment 8650128 [details]
Aurora 42.0a2 showing same problem wihtout 1180387 changes

Comment 6

3 years ago
When I said "it was made worse," I mean this bug is more likely to be seen by more users. Yes, even without the patch, it's possible to trigger the bug if the user's window is sized a certain way, but with the patch, the bug happens for a lot more window sizes.
(Assignee)

Comment 7

3 years ago
Created attachment 8650202 [details] [diff] [review]
1195321.patch.v1

Increasing top margin of the search-container to avoid overlaps with undo message.
Attachment #8650202 - Flags: review?(edilee)
(Assignee)

Comment 8

3 years ago
Created attachment 8650205 [details]
Undo message and search container with uncreased margin-top

Requesting info from aaron if this would be an acceptable fix.
Flags: needinfo?(athornburgh)

Comment 9

3 years ago
Comment on attachment 8650202 [details] [diff] [review]
1195321.patch.v1

Approach is fine. Wait for dcrobot's spec on how far the search bar should be from the undo container as well as the customize/gear button.
Attachment #8650202 - Flags: review?(edilee) → review+

Comment 10

3 years ago
would it be possible to show this beneath the actual tile deleted, instead of in a fixed position upper/left?
Flags: needinfo?(athornburgh)
(Assignee)

Comment 11

3 years ago
(In reply to Aaron from comment #10)
> would it be possible to show this beneath the actual tile deleted, instead
> of in a fixed position upper/left?

It could be, but I would prefer to file that as a separate bug. I need to uplift all changes related to 1180387 (this bug included) to aurora/beta, preferably this Friday. I would  much prefer if we do a simple fix now.

Comment 12

3 years ago
Max, I'm good with your solution. Don't worry about my comment above for now.

Thanks.

Updated

3 years ago
Assignee: nobody → mzhilyaev
https://hg.mozilla.org/mozilla-central/rev/e7355d3ce8cc
https://hg.mozilla.org/mozilla-central/rev/2ad682b0da99
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox43: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 43
Confirming the fix on latest Nightly, build ID: 20150825030212.
Tested on Windows 7 64-bit, Windows 10 64-bit, Mac OS X 10.9.5 and Ubuntu 12.04 32-bit.

Filled bug 1198754, the cog wheel overlaps the thumbnail removal message when the window is resized.
Status: RESOLVED → VERIFIED
status-firefox43: fixed → verified
Flags: qe-verify+
QA Contact: cornel.ionce
(Assignee)

Comment 17

3 years ago
Created attachment 8654659 [details] [diff] [review]
V1. 1195321 aurora patch

Approval Request Comment
[Feature/regressing bug #]:
A regression of 1180387
[User impact if declined]:
Significant - UX looks ugly
[Describe test coverage new/current, TreeHerder]:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ab58a4b34f15
[Risks and why]: 
Low risk - the change is one-liner for css margin setting

[String/UUID change made/needed]: NONW
Attachment #8654659 - Flags: approval-mozilla-aurora?
(Assignee)

Updated

3 years ago
Attachment #8654659 - Attachment description: 1195321.aurora.v1 → V1. 1195321 aurora patch
status-firefox42: --- → affected
Comment on attachment 8654659 [details] [diff] [review]
V1. 1195321 aurora patch

Bad UX, taking it.
Attachment #8654659 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
The uplift of bug 1194895 actually fixed those failures before this backout landed. Re-landed.
https://hg.mozilla.org/releases/mozilla-aurora/rev/214492ce4267
status-firefox42: affected → fixed

Updated

3 years ago
Blocks: 1198204

Updated

3 years ago
Blocks: 1205313

Updated

3 years ago
No longer blocks: 1205313
Verified fixed on Windows 7 64-bit, Windows 10 32-bit, Mac OS X 10.9.5 and Ubuntu 12.04 32-bit using Firefox 42 beta 2, build ID: 20150928102225.
status-firefox42: fixed → verified
You need to log in before you can comment on or make changes to this bug.