[ 
https://wso2.org/jira/browse/MASHUP-1084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=19223#action_19223
 ] 

Jonathan Marsh commented on MASHUP-1084:
----------------------------------------

Fixed in rev 21486.

Verified there is no place where AddCss() could be called that doesn't already 
have a correct css declaration.  Thus removed the whole function and it's usage.

> Download error in tooltip.js
> ----------------------------
>
>                 Key: MASHUP-1084
>                 URL: https://wso2.org/jira/browse/MASHUP-1084
>             Project: WSO2 Mashup Server
>          Issue Type: Bug
>            Reporter: Jonathan Marsh
>            Assignee: Jonathan Marsh
>             Fix For: 1.6
>
>
> In Firefox 3, using Firebug, there is a failing stylesheet download.  The 
> links are shown below.  The second one is apparently added dynamically by 
> tooltip.js.
> <link type="text/css" rel="stylesheet" href="css/styles.css">
> <link class="undefined" style="display: block;" type="text/css" 
> rel="stylesheet" href="styles.css" media="screen">
> Options:
> 1) Leave for now, postpone till next release - it doesn't appear to hurt 
> anything as is.
> 2) tooltip.js is in the header.jsp, which is included broadly.  Is there 
> anywhere it is used that doesn't already import styles.css?  If not, the 
> AddCss() function could simply be removed.
> 3) AddCss() could check whether there already is a stylesheet, in which case 
> it won't add one.
> 4) Fix the href to be css/styles.css.
> If the function stays, we should remove class and style attributes from it - 
> seems like they might cause validity-related problems somewhere.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
https://wso2.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

_______________________________________________
Mashup-dev mailing list
[email protected]
http://mailman.wso2.org/cgi-bin/mailman/listinfo/mashup-dev

Reply via email to