Bug 19838 - Top menu layout broken at www.ati.su
Summary: Top menu layout broken at www.ati.su
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Evangelism (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: http://www.ati.su/trace/
Keywords:
Depends on:
Blocks:
 
Reported: 2008-06-30 23:59 PDT by Alexey Proskuryakov
Modified: 2009-11-15 01:21 PST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Proskuryakov 2008-06-30 23:59:14 PDT 
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.
Comment 1 Robert Blaut 2008-07-01 00:48:36 PDT 
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&amp;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;
Comment 2 Robert Blaut 2009-11-15 01:21:15 PST 
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.