[ 
https://issues.apache.org/jira/browse/SHINDIG-649?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12639174#action_12639174
 ] 

Kevin Jin commented on SHINDIG-649:
-----------------------------------

The new patch Whitespace-Test-Fixes.patch replaces fix-649-test.patch -- 
submitter only needs to apply the new one.
It'll be better to remove _T_Conditional_Number, _T_Conditional_Number, 
_T_Conditional_Number, _T_Tag_input in ost_test.html, and change from double 
quotes (") to single quotes (') for JS strings.

> OST: Fix some whitespace issues on IE, use gadgets.util.registerOnLoadHandler 
> when available
> --------------------------------------------------------------------------------------------
>
>                 Key: SHINDIG-649
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-649
>             Project: Shindig
>          Issue Type: Bug
>          Components: Features (Javascript)
>            Reporter: Lev Epshteyn
>            Priority: Minor
>         Attachments: fix-649-test.patch, OnLoad-and-Whitespace-Fixes.patch, 
> Whitespace-Test-Fixes.patch
>
>
> IE interprets whitespace literally in calls to document.createTextNode(). 
> Special handling existed for this before, and this patch fixes some issues 
> with it. In particular, trailing/leading spaces will be stripped off of 
> <os:Render/>, and whitespace between tags will now be properly detected.
> Also added support for gadgets.util.registerOnLoadHandler() when it is 
> available.
> Code review: http://codereview.appspot.com/7450

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to