Can't set as Home Page the Other Bookmarks folder
Categories
(Firefox :: Settings UI, defect, P5)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox65 | --- | wontfix |
firefox66 | --- | wontfix |
firefox67 | --- | verified |
People
(Reporter: cfogel, Assigned: standard8)
References
Details
(Keywords: regression)
Attachments
(2 files)
Affected versions
- 67.0a1(Build ID_20190310214003), 66.0b15, 65.0.2;
Affected platforms
- Windows 10, macOS 10.12.6, Ubuntu 16.04
Steps to reproduce
- Launch Firefox and access about:preferences#home
- Click on the dropdown for Homepage and new windows;
- Choose the Custom Urls... option;
- Click on the Use Bookmark... button;
- Click on the Other Bookmarks option;
- Click on the Ok button.
Expected result
- the bookmarks + folder links are added in the Custom URLs input field
Actual result
- onClick effect is triggered for the button, but nothing happens;
Regression range
- aTrif helped with getting the regression range:
- Last good build_date: 2018-05-25 11:35:52.672000 changeset: 8d9da33b8e4ffc559774c7c9c144cba819d6b886
- First bad build_date: 2018-05-25 11:49:21.138000 changeset: 8a4d896b435b50718c66a691c9fdec6bf7bfba2d
- Pushlog URL: link
- issue not reproduce on 60.0b3;
Additional notes
- attached recording with the issue;
- as per bug 1451756 there are some inconsistencies in this area;
- for this issue; it's that while the OK does nothing for the Other bookmarks folder, for the Bookmarks Toolbar and the Bookmarks Menu options it does "something".
Reporter | ||
Updated•5 years ago
|
Comment 1•5 years ago
|
||
Mark, looks like your patches caused a regression here. Can you investigate?
Updated•5 years ago
|
Comment 2•5 years ago
|
||
(Maybe this is intentional?)
Assignee | ||
Comment 3•5 years ago
|
||
Something that I spotted is that the virtual guids used for Other/Mobile virtual queries are invalid - I filed bug 1534322 for that. It does aid to breaking this, but not completely. For example, selecting either one of Other or Mobile those throws an error at the moment, but the toolbar & menu ones don't.
The actual issue here is that we're not getting the concrete Guid for the node, so we're using the virtual query guid to try and build what's in the folder. That's simple to fix.
I'm not quite sure why people would want to select a whole folder, but stopping that would probably currently be more complicated than fixing this.
Assignee | ||
Comment 4•5 years ago
|
||
Depends on D23004
Comment 5•5 years ago
|
||
bugherder |
Comment 6•5 years ago
|
||
Comment 7•5 years ago
|
||
I have managed to reproduce this issue using Firefox 67.0a1 (20190311215435) on Windows 10 x64.
This issue is verified fixed using Firefox 67.0a1 (20190315093917) on Windows 10 x64, Ubuntu 18.04 x64, macOS 10.14.3.
Updated•5 years ago
|
Description
•