You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Werner Punz (Jira)" <de...@myfaces.apache.org> on 2023/03/27 15:17:00 UTC

[jira] [Resolved] (MYFACES-4596) faces.ts style tags must retain their original position (and script tags as well)

     [ https://issues.apache.org/jira/browse/MYFACES-4596?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Werner Punz resolved MYFACES-4596.
----------------------------------
    Fix Version/s: 4.0.1
       Resolution: Fixed

fixed also tested by the Tobag guys fixes their issue!

 

> faces.ts style tags must retain their original position (and script tags as well)
> ---------------------------------------------------------------------------------
>
>                 Key: MYFACES-4596
>                 URL: https://issues.apache.org/jira/browse/MYFACES-4596
>             Project: MyFaces Core
>          Issue Type: Bug
>    Affects Versions: 4.0.0
>            Reporter: Werner Punz
>            Assignee: Werner Punz
>            Priority: Minor
>             Fix For: 4.0.1
>
>
> This bug came from a corner case the Tobago guys had.
> The problem is that the style tags are not retained at their proper position, but are evaled like script tags via the head appendix method.
> While script tags only can be evaluated by appending them on the head for a short period of time, this is not needed for style tags.
> I have a solution for this (see [https://github.com/werpu/jsfs_js_ts/pull/35 in the comments)|https://github.com/werpu/jsfs_js_ts/pull/35]
> The Tobago guys thankfully have provided a test case. 
> Fix will be in 4.0.1 including the provided unit test!
>  
> Note: I consider this a minor, bug, because it does not prevent the style tags to be properly evaluated by the browser, just if code needs to check for embedded style tags being in their proper location, this becomes a major issue! This very likely only affects component libraries (I have yet to have had this case in a project, hence I did not have it on the radar)
> Nevertheless it will be fixed!
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)