Here's my first template for a module. Single theme function for actions_display.

I worked off Merlinofchaos' example so I hope it's consistent.

CommentFileSizeAuthor
#2 tplify_actions_3.patch3.35 KBdvessel
#1 tplify_actions_2.patch3.35 KBdvessel
tplify_actions_1.patch3.27 KBdvessel

Comments

dvessel’s picture

dvessel commented
StatusFileSize
new tplify_actions_2.patch3.35 KB

Cleaned up a bit. odd/even classes wasn't surrounded by quotes. Changed template name to use dashes instead of underscores. Added tableheader.js and made code a bit more consistent.

dvessel’s picture

dvessel commented
StatusFileSize
new tplify_actions_3.patch3.35 KB

minor doxygen change.

moshe weitzman’s picture

moshe weitzman commented

actions list only appears in admin. probably better off focusing on end user theme functions. this is fin, just low value.

dvessel’s picture

dvessel commented

Yeah, I know. Was starting alphabetically which wasn't so smart but I'm working on more important ones now.

rstamm’s picture

rstamm commented
Status: Needs review » Needs work

Quick review:

CVS Id tag broken and missing blank line after cvs id tag.

dvessel’s picture

dvessel commented
Status: Needs work » Closed (won't fix)

Yeah, I don't think this'll get in. Not much value in theming admin pages. Was more for practice I guess. :)