Closed (won't fix)
Project:
Webform
Version:
6.x-3.4
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
6 Oct 2010 at 11:38 UTC
Updated:
23 Aug 2014 at 13:01 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
quicksketchI agree, these should run through theme functions. Patches appreciated. :-)
Comment #2
carinadigital commentedI've added a simple patch to change textfield and textarea components to run through a common theme function that can be overriden.
I've struggled to work out a generic theme implementation for component tables. Maybe the _webform_table hook itself should be a theme function with a preprocess as it's only displaying the same data but in a different view?
Comment #3
quicksketchI think the most logical solution would be a separate theme function for every component, similar to how we have separate theme functions for displaying in e-mail.
Comment #4
carinadigital commentedI think you're right. It's the webform component developers that should create appropriate default theme functions.
I'll try to submit a patch for the core components.
Comment #5
carinadigital commentedI've added a patch against webform-6.x-3.4 to add table theme override functions for textfield and textarea components. They're limited to 20 chars by default, and then theme overridable in the normal way.
I'm not sure the other default components need theme functions as their values won't be that long.
Comment #6
danchadwick commentedThere won't be further feature development in the X.x-3.x branches.