Closed Bug 1104803 Opened 8 years ago Closed 8 years ago

Ui Tour associated with the new searchbar not opened when searchbar is moved to overflow panel.

Categories

(Firefox :: General, defect)

34 Branch
All
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED WONTFIX
Tracking Status
firefox33 --- unaffected
firefox34 - wontfix
firefox35 --- affected
firefox36 --- affected
firefox-esr31 --- unaffected

People

(Reporter: VarCat, Unassigned)

References

Details

Environment:

Firefox tinderbox build
http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-beta-win32/latest/
Build Id:20141124160229
OS: Win 7 x64
STR:

1. Resize the window so that the searchbar is moved to the overflow panel.
2. Go to https://www-demo5.allizom.org/en-US/firefox/34.0/whatsnew/?oldversion=33.1.0

Issue:
When the searchbar is moved to the overflow panel the doorhanger associated with the tour and the search bar are not appearing.
The test was done using a tinderbox builds but is related to a feature associated with Firefox 34.
This is an edge case that we won't fix in 34. I've marked 35+ as affected.
Flags: firefox-backlog? → firefox-backlog+
(In reply to Catalin Varga [QA][:VarCat] from comment #0)
> When the searchbar is moved to the overflow panel the doorhanger associated
> with the tour and the search bar are not appearing.

I think this might have been intentional. Matt/Alex, do you guys recall details?
Flags: needinfo?(agibson)
Flags: needinfo?(MattN+bmo)
No longer blocks: fx34
(In reply to :Gavin Sharp [email: gavin@gavinsharp.com] from comment #3)
> (In reply to Catalin Varga [QA][:VarCat] from comment #0)
> > When the searchbar is moved to the overflow panel the doorhanger associated
> > with the tour and the search bar are not appearing.
> 
> I think this might have been intentional. Matt/Alex, do you guys recall
> details?

Yes, this was intentional AIUI: https://mxr.mozilla.org/mozilla-central/source/browser/modules/UITour.jsm?rev=6c84bbab2848&mark=1587,1607-1612#1584
Flags: needinfo?(MattN+bmo)
As Matt said, clearing needs info. Thanks
Flags: needinfo?(agibson)
As a side note, web pages are currently also having to hide infoPanels on resize until bug 1091785 is fixed.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.