Adding 'components' templates #116

Closed
wants to merge 11 commits into
from

Conversation

Projects
None yet
4 participants
Contributor

mpccolorado commented Jun 5, 2013

Added new attribute: 'component' (in 'field' or 'display' taglibs).
Now it is possible to specify which component is going to be used (it has to be placed inside _fields/_components/componentName/_field.gsp or _fields/_components/componentName/_display.gsp)

e.g.: (Is going to use _fields/_components/time/_field.gsp)

This is usefull to render same template multiple times across different properties.

martin added some commits Jun 5, 2013

martin New attribute added: 'component'.
Now it is posible to specify which component is going to be used (it has to be placed inside _fields/_components/componentName/_field.gsp or _fields/_components/componentName/_display.gsp).
e.g.: <field bean="someBean" property="someProperty" component="time"/> (Is going to use _fields/_components/time/_field.gsp)
420acf3
martin Merge branch 'addComponentAttribute' 6f2a07d

Rob Fletcher » grails-fields #39 FAILURE
Looks like there's a problem with this pull request
(what's this?)

Rob Fletcher » grails-fields #40 FAILURE
Looks like there's a problem with this pull request
(what's this?)

Contributor

mpccolorado commented Jun 10, 2013

I run the tests over and over, sometimes it throws errors and sometimes it doesn't. =/

Rob Fletcher » grails-fields #51 FAILURE
Looks like there's a problem with this pull request
(what's this?)

Rob Fletcher » grails-fields #57 FAILURE
Looks like there's a problem with this pull request
(what's this?)

Owner

sbglasius commented Nov 17, 2014

I would like to see this feature in the plugin, but you need to spend the time to make the pull-request build against the latest master. Thank you.

Contributor

mpccolorado commented Nov 25, 2014

I'll look into it this week!

mpccolorado closed this Jan 2, 2015

@mpccolorado did this feature got merged?

Contributor

mpccolorado commented Mar 9, 2015

@eugene-kamenev Please, see this: #180 to discuss all the next changes...

Theoretically all these changes should be trivial, but I have to invest some time to fix the tests... Sorry for the delay, I will try to merge all the changes for revision this week...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment