Closed Bug 1224310 Opened 4 years ago Closed 4 years ago

Suggested tiles requires two blocks to stop showing in newtab

Categories

(Firefox :: New Tab Page, defect, P1)

defect
Points:
5

Tracking

()

VERIFIED FIXED
Firefox 45
Iteration:
45.1 - Nov 16
Tracking Status
firefox44 --- verified
firefox45 --- verified

People

(Reporter: mzhilyaev, Assigned: mzhilyaev)

Details

Attachments

(2 files)

Occasionally, a user still needs to block a suggested tile twice before it disappears from newtab page.  The expected behavior is when suggested tiles stop showing after a user blocks it ones.

This is a followup bug from Bug 1204361.

I managed to reproduce it on nightly. Suspect a caching issue.
Assigning to myself for root-cause analysis.
(In reply to maxim zhilyaev from comment #0)
> Occasionally, a user still needs to block a suggested tile twice before it disappears

Clarifying this language slightly -- we seem to be showing *two different suggested tiles* in a row when this happens. At least, that's the case in Paul's screencast http://screencast.com/t/G0mEszAxu8R -- first he's got a "Quartz" suggested tile, and then he x's that out and gets a "The Scene" suggested tile.  (Not sure if this matches maxim's experience when he reproduced this.)
Attachment #8687381 - Flags: review?(msamuel)
Comment on attachment 8687381 [details] [diff] [review]
updated suggested tiles when suggested tile is blocked

Review of attachment 8687381 [details] [diff] [review]:
-----------------------------------------------------------------

Was this line previously deleted or something? Any idea how this bug came about?
Attachment #8687381 - Flags: review?(mcaceres) → review+
Did not see "double block" behavior when fixing Bug 1204361.
Which is why this was overlooked.
https://hg.mozilla.org/mozilla-central/rev/70bb205d2c85
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 45
Flags: qe-verify+
QA Contact: cornel.ionce
Attached patch v1. aurora patchSplinter Review
Approval Request Comment
[Feature/regressing bug #]:
Bug 1224310 - Suggested tiles requires two blocks to stop showing in newtab
[User impact if declined]:
Irritating need to block suggested tile twice to make it go away
[Describe test coverage new/current, TreeHerder]:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8e4b672a0b3c
manual testing
[Risks and why]: 
One liner change - risks are minimal
[String/UUID change made/needed]:
None
Attachment #8687978 - Flags: approval-mozilla-aurora?
Comment on attachment 8687978 [details] [diff] [review]
v1. aurora patch

Seems like a good bug to fix, it is annoying if it takes more than one try to remove a suggested tile. Let's uplift to Aurora44.
Attachment #8687978 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Suggested tiles can be closed now by one click.
Verified fixed 45.0a1 (2015-11-17) Win 7.
Status: RESOLVED → VERIFIED
QA Contact: cornel.ionce → paul.silaghi
Question: no matter what I do, I can see only one suggested tile. Is it possible to have more suggested tiles displayed?
Flags: needinfo?(mzhilyaev)
Paul,

Do you want to have two different suggested tiles displayed on same newtab page simultaneously?
Or do you want to have different suggested tiles being recommended to you in the same tile?

The former, I do not think can be changed. If it's the latter, you should see a different suggested tile if you load a new page a few times, which would change you history tiles display.
Flags: needinfo?(mzhilyaev)
(In reply to maxim zhilyaev from comment #12)
> Do you want to have two different suggested tiles displayed on same newtab
> page simultaneously?
Yes
> The former, I do not think can be changed
Ok, considering the current behavior as expected. Thanks
Verified fixed FF 44.0a2 (2015-11-19) OS X 10.8.5
You need to log in before you can comment on or make changes to this bug.