[drupal-devel] [feature] make profiles more themable: do not pass along huge chunks of HTML

[Dublin Drupaller]

Issue status update for 
http://drupal.org/node/31123
Post a follow up: 
http://drupal.org/project/comments/add/31123

 Project:      Drupal
 Version:      cvs
 Component:    user system
 Category:     feature requests
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  Bèr Kessels
 Updated by:   Dublin Drupaller
 Status:       patch (code needs review)

I agree with the objective of making profiles more themable and in an
easier way. I am working with kiev1.org on a patch for flexinode that
unleashes themeability control, which is not reliant on any theme
engine..the work in progress patch is called FlexiMAX and a this
screenshot illustrates the simplicity of how it works [1] in a
flexinode context.


Without digressing from your patch, I was thinking about applying the
flexiMAX idea to the profile.module as well. i.e. a flexiMAX type
layout field is displayed with Profile_fields populated and converted
into simple to use tags to insert into a bespoke layout.


I like the simplicity of control and it's non-dependence on the
phptemplate.engine and tpl.php overrides..putting the power back into
the .CSS file and with non-php programmers or designers without much
php savvy. The phptemplate engine is superb. It's just a struggle
sometimes to delve into relatively heavy coding for a simple design
tweak.


Just my 2eurocents.


Dub


[1] http://drupal.org/node/30376\" title=\"click for the ongoing
discussion about the fleximax.module patch




Dublin Drupaller



Previous comments:
------------------------------------------------------------------------

Tue, 13 Sep 2005 18:35:50 +0000 : Bèr Kessels

Attachment: http://drupal.org/files/issues/user_profile_themable_mutliarray.patch (1.46 KB)

I want more power of the theming the user profiles. The recent patch for
profiles imporves ti a little it, but leaves one mayor issue open;
All fields in a "category' are passed along as one huge chunk. That is
a pity, IMO, for you cannot really theme it anymore, exept re-use tha
chunk of HTML. Or, if you really wish, you could pregreplace teh HTML. 


This minor patch leaves the default result exactly as it is, yet it
allows themes to do some advanced stuff, without having to do database
queries in the theme.


An example is http://newsphoto.nl/user/2 where I curerntly need about
40 lines of code and two database calls, while with this simple patch,
it will be theme function of only five lines!
And then it allows me to use a .tpl.php file to use reach field as a
variable.