Regression: Top Sites area on New Tab Page is no longer collapsable
Categories
(Firefox :: New Tab Page, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox76 | --- | unaffected |
firefox77 | --- | unaffected |
firefox78 | + | verified |
firefox79 | --- | verified |
People
(Reporter: mehmet.sahin, Assigned: bugzilla)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
jcristau
:
approval-mozilla-beta+
|
Details |
Nightly 78.0a1 (2020-05-29) (64-Bit)
macOS 10.15.5
STR:
1.) Enable Top Sites on your NTP
2.) Open a NTP
3.) Click on the Top Sites icon / Triangle above the Top Sites
Expected: Top Sites area should collapse.
Actual: Top Sites area does not collapse.
This is a recent regression.
Comment 1•4 years ago
|
||
Looks like the section id changed for top sites feed but not for the content?
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 2•4 years ago
|
||
[Tracking Requested - why for this release]:
Easy-to-fix but annoying regression resulting from a 77 patch: bug 1634279.
Assignee | ||
Comment 3•4 years ago
|
||
[Tracking Requested - why for this release]:
Sorry, I meant 78 :)
Assignee | ||
Comment 4•4 years ago
|
||
Updated•4 years ago
|
Just a question: I haven't mentioned that in my bugreport, but when you click on the triangle to try to collapse the Top Sites sections, the collapse does not work AND the tiles are no longer clickable. Will the "not clickable" issue also be fixed with your patch?
Assignee | ||
Comment 7•4 years ago
|
||
Yes, my patch should fix that issue.
(In reply to Harry Twyford [:harry] from comment #7)
Yes, my patch should fix that issue.
Thanks for your feedback :)
Comment 9•4 years ago
|
||
bugherder |
Updated•4 years ago
|
Comment 10•4 years ago
|
||
The patch landed in nightly and beta is affected.
:harry, is this bug important enough to require an uplift?
If not please set status_beta
to wontfix
.
For more information, please visit auto_nag documentation.
Assignee | ||
Comment 11•4 years ago
|
||
Comment on attachment 9153141 [details]
Bug 1642082 - Restore the ability to collapse the NTP Top Sites section. r?thecount!
Beta/Release Uplift Approval Request
- User impact if declined: Regression where the Top Sites section on the NTP can't be collapsed.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Very small (2-line) change that's baked on Nightly for a few days and that only affects the front-end. If something goes wrong with this patch, it would be easy to undo.
- String changes made/needed:
Comment 12•4 years ago
|
||
Comment on attachment 9153141 [details]
Bug 1642082 - Restore the ability to collapse the NTP Top Sites section. r?thecount!
regression fix for new tab, approved for 78.0b3
Comment 13•4 years ago
|
||
bugherder uplift |
Comment 14•4 years ago
|
||
I have verified the following, using Firefox Beta 78.0b3 (Build ID: 20200604213430), and Firefox Nightly 79.0a1 (Build ID: 20200605043926) on Windows 10 x64, macOS 10.15, and Ubuntu Linux 18.04:
- The "Top Sites" section is collapsed when the title is clicked.
- The "Top Sites" section is expanded when the title is clicked again.
- The "Top Sites" section is collapsed when the "Collapse Section" option is selected from the "Section Context Menu" (...).
- The "Top Sites" section is expanded when the "Expand Section" option is selected from the "Section Context Menu" (...).
- The "Top Sites" tiles can be clicked after the "Top Sites" section is expanded.
Description
•