Add #{get 'moreStyles' /} and #{get 'moreScripts' /} in the default main.html

Bug #485907 reported by Guillaume Bort
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
play framework
Status tracked in 1.0
1.0
Fix Released
Wishlist
Nicolas Leroux
1.1
Fix Committed
Undecided
Unassigned

Bug Description

Because it is useful as starting point.

Changed in play:
status: New → Confirmed
Revision history for this message
Nicolas Leroux (nicolas-lunatech) wrote :

I think we need to extend the set tag because it is impossible to do:

#{set moreStyles:'<link rel="stylesheet" type="text/css" media="screen" href="@{'/public/stylesheets/xxx.css'}" />'}

or

#{set moreStyles}
<link rel="stylesheet" type="text/css" media="screen" href="@{'/public/stylesheets/xxx.css'}" />
#{/set}

at the moment. However,

#{set moreStyles:'<link rel="stylesheet" type="text/css" media="screen" href="/public/stylesheets/xxx.css" />'/}

does work but it is not really elegant, especially when you have multiple css/scripts.

The best will of course to support 'tiles', but this is another issue.

Revision history for this message
Guillaume Bort (guillaume-bort) wrote : Re: [Bug 485907] Re: Add #{get 'moreStyles' /} and #{get 'moreScripts' /} in the default main.html

#{set 'moreStyles'}
<link rel="stylesheet" type="text/css" media="screen"
href="@{'/public/stylesheets/xxx.css'}" />
#{/set}

Works.

On Sun, Nov 22, 2009 at 1:33 PM, Nicolas <email address hidden> wrote:
> I think we need to extend the set tag because it is impossible to do:
>
> #{set moreStyles:'<link rel="stylesheet" type="text/css" media="screen" href="@{'/public/stylesheets/xxx.css'}" />'}
>
> or
>
> #{set moreStyles}
> <link rel="stylesheet" type="text/css" media="screen" href="@{'/public/stylesheets/xxx.css'}" />
> #{/set}
>
> at the moment. However,
>
> #{set moreStyles:'<link rel="stylesheet" type="text/css" media="screen"
> href="/public/stylesheets/xxx.css" />'/}
>
> does work but it is not really elegant, especially when you have
> multiple css/scripts.
>
> The best will of course to support 'tiles', but this is another issue.
>
> --
> Add #{get 'moreStyles' /} and #{get 'moreScripts' /} in the default main.html
> https://bugs.launchpad.net/bugs/485907
> You received this bug notification because you are a member of play
> framework developers, which is subscribed to play framework.
>

Revision history for this message
Nicolas Leroux (nicolas-lunatech) wrote :

I see I made a typo. Thanks.

Changed in play:
assignee: nobody → Nicolas (nicolas-lunatech)
Changed in play:
status: Confirmed → Fix Committed
Changed in play:
importance: Undecided → Wishlist
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.