Bug 19838
| Summary: | Top menu layout broken at www.ati.su | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Alexey Proskuryakov <ap> |
| Component: | Evangelism | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | webkit |
| Priority: | P2 | ||
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
| URL: | http://www.ati.su/trace/ | ||
Alexey Proskuryakov
Steps to reproduce: open <http://www.ati.su/trace/>. At the top, there is supposed to be a conventional menu. In Safari, each item of the menu is on a separate line, taking 100% width.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Robert Blaut
It's an evangelism issue. Menu works fine when Safari uses Firefox User Agent. http://www.ati.su/ScriptResource.axd?d=OPZPYRY9FNX5cM9gnPsEvN9mVsee7Kvw_yq6Rb6CBff95ZSDtYstA09I_T-XkEkfIIA3jyEszbxLGKfpv72KuAQDbieVhNI30&t=633457542181373055 is the script containing UA sniffing code:
if(navigator.userAgent.indexOf(" MSIE ") >- 1) {
Sys.Browser.agent = Sys.Browser.InternetExplorer;
Sys.Browser.version = parseFloat(navigator.userAgent.match(/MSIE (\d+\.\d+)/)[1]);
Sys.Browser.hasDebuggerStatement = true}
else if(navigator.userAgent.indexOf(" Firefox/") >- 1) {
Sys.Browser.agent = Sys.Browser.Firefox;
Sys.Browser.version = parseFloat(navigator.userAgent.match(/Firefox\/(\d+\.\d+)/)[1]);
Sys.Browser.name = "Firefox";
Sys.Browser.hasDebuggerStatement = true}
else if(navigator.userAgent.indexOf(" Safari/") >- 1) {
Sys.Browser.agent = Sys.Browser.Safari;
Sys.Browser.version = parseFloat(navigator.userAgent.match(/Safari\/(\d+\.\d+)/)[1]);
Sys.Browser.name = "Safari"}
else if(navigator.userAgent.indexOf("Opera/") >- 1)Sys.Browser.agent = Sys.Browser.Opera;
Robert Blaut
I've checked the reported site in the latest WebKit 4.0.4 (6531.21.10, r50918) and found no problem now.
The site was changed.
I close the bug as resolved fixed an evangelism issue.