[Webkit-unassigned] [Bug 53705] Viewport parsing no longer accepts "1.0; " value as valid.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=53705


Joseph Pecoraro <joepeck at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |klobag at gmail.com




--- Comment #6 from Joseph Pecoraro <joepeck at webkit.org>  2011-02-03 15:39:54 PST ---
(In reply to comment #5)
> I think that we might need to handle ; separately as many pages are using that
> as separators, and Android has a patch making it work as a separator. Even
> facebook uses this.

I think you are referring to:
<http://webkit.org/b/47607> meta tag parser needs to support ; as separator due to Android having made that popular

In that bug, the verdict was that Android had removed the semicolon as a separator,
despite sites like Facebook using it:
https://bugs.webkit.org/show_bug.cgi?id=47607#c6

A google search does show that Fennec made the following claim:
https://developer.mozilla.org/en/Mobile/Viewport_meta_tag#Background

>>    Mobile Safari introduced the "viewport meta tag" to let web developers control
>>    the viewport's size and scale. Many other mobile browsers now support this tag,
>>    although it is not part of any web standard. Apple's documentation does a good
>>    job explaining how web developers can use this tag, but we had to do some
>>    detective work to figure out exactly how to implement it in Fennec. For example,
>>    Safari's documentation says the content is a "comma-delimited list," but existing
>>    browsers and web pages use any mix of commas, semicolons, and spaces as
>>    separators.

If Fennec does support semicolons, then maybe others should follow suit.

All the other descriptions I've found specify comma-separated:
http://docs.blackberry.com/en/developers/deliverables/6176/HTML_ref_meta_564143_11.jsp
http://developer.apple.com/library/safari/#documentation/AppleApplications/Reference/SafariHTMLRef/Articles/MetaTags.html



> With your change, how will the following be handled?
> 
> "width=320;height=480" 
> 
> I guess it would ignore the height.

Correct.


> Apart from my above comments, I agree with your changes.

Excellent! I'm including Grace to see if she can verify that Android did in fact
remove the semicolon as a separator.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.