This may be solved by webchick's theme code -- http://drupal.org/node/74025 -- but by default, having an anchor that links directly to signup will be useful for nodes with large content, and it also means you can teleport directly from a teaser view to the bottom of the page.

For extra bonus points, make an AJAX function so that in the teaser, you can display an "I'm going" / "Sign up" now to toggle :P

P.S. Should "Theme" be one of the default components? UI is not really what this is...

CommentFileSizeAuthor
#3 78707_signup_anchor.3.patch1.16 KBdww

Comments

dww’s picture

i thought the cardinal rule for module authors (me) was "thou shalt not put html in your output, that's what the theme layer is for". i am not a themer. ;) now that i committed #74025, are you happy enough? can't you put your own anchor where you want it?

if you think signup.module should be generating these anchors (and i agree, that could be potentially handy), what's the right way to do it so that i don't end up pissing off some themer somewhere? ;) please attach a patch, and i'll happily review/commit it. or, at least, tell me how i'm supposed to add anchors in the right place without breaking the cardinal rule referenced above...

thanks,
-derek

dww’s picture

Version: 4.7.x-1.x-dev » 5.x-2.x-dev
Component: User interface » Themeability

This seems like a nice touch in the default theme functions we should be working on at http://drupal.org/node/47913

dww’s picture

Assigned: Unassigned » dww
Status: Active » Needs review
StatusFileSize
new1.16 KB

How's this?

dww’s picture

Status: Needs review » Fixed

Committed to HEAD.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.