Re: [PHPTAL] Proposal: Auto-fill slots with #ids

2011-08-19 Thread Kornel Lesiński

On Fri, 19 Aug 2011 11:22:54 +0100, aaatoja  wrote:


When creating typical web page we all use #ids, and something like:

layout.html>
metal:define-macro="html"
  metal:define-slot="content"

index.html>
metal:use-macro="../layout.html/html
  metal:fill-slot="content"

And most probably content will be . So maybe just to  
make our work easier PHPTAL should also fill directly element with  
defined ids?

layout.html>
metal:define-macro="html"
  some text

index.html>
metal:use-macro="../layout.html/html
  this will replace "some text" in master  
template


What do You think?


This looks like XUL overlays, which I think are a very nice idea.

However, I don't think this fits the TAL way of working: explicit  
namespaced attributes and no "magic" happening with other attributes. I'm  
also worried that addition of such feature could break existing templates.


Hopefully, this isn't too hard to implement with prefilters, so you can  
have this feature for your templates, if you wish.


--
regards, Kornel Lesiński

___
PHPTAL mailing list
PHPTAL@lists.motion-twin.com
http://lists.motion-twin.com/mailman/listinfo/phptal


Re: [PHPTAL] Proposal: Auto-fill slots with #ids

2011-08-19 Thread Per Bernhardt
I totally disagree that "we all use #ids" :)

When it comes to styling or semantic markup, ids have no advantage to 
well-choosen classes.
It is rather problematic because there are some bad things about ids:

 *   They mess up CSS specificity because there are way too strong
 *   They have to be unique which may lead to invalid html markup in the long 
run
 *   ...

So I think we shouldn't mess up the clean and concise syntax of phptal only to 
support something I wouldn't agree with at all.

My two cents worth...
Per



Am 19.08.11 12:22 schrieb "aaatoja" unter :

When creating typical web page we all use #ids, and something like:

layout.html>
metal:define-macro="html"
  metal:define-slot="content"

index.html>
metal:use-macro="../layout.html/html
  metal:fill-slot="content"

And most probably content will be . So maybe just to make our 
work easier PHPTAL should also fill directly element with defined ids?
layout.html>
metal:define-macro="html"
  some text

index.html>
metal:use-macro="../layout.html/html
  this will replace "some text" in master template

What do You think?



___
PHPTAL mailing list
PHPTAL@lists.motion-twin.com
http://lists.motion-twin.com/mailman/listinfo/phptal



--

webfactory GmbH
Per Bernhardt, Anwendungsentwickler

Lessingstraße 60
53113 Bonn
Germany

Fon +49 228 9114455
Fax +49 228 9114499
www.webfactory.de
p...@webfactory.de

Geschäftsführer: Sebastian Kugler, Matthias Pigulla
Handelsregister: HRB 8673, Amtsgericht Bonn
___
PHPTAL mailing list
PHPTAL@lists.motion-twin.com
http://lists.motion-twin.com/mailman/listinfo/phptal