Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.We should probably display the description for each poll.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | advpoll_descriptions_0.patch | 1.3 KB | anders.fajerson |
| advpoll_descriptions.patch | 4.67 KB | ChrisKennedy |
Comments
Comment #1
anders.fajerson commentedYes we should :) Just looked at the patch briefly and here arae some hings that came up:
* We should probably use the same class name all over, "description" is probably enough, haven't done any research though to see if a class name for the body (which this is) is uses anywhere else in Drupal.
* The input format doesn't seem to be used (I have no linebreaks and no formatting for other than HTML)
* For block and on frontpage we should probably only display the teaser.
Comment #2
anders.fajerson commentedHow about this?
Note that the Ajax-messages will come after the description, don't know if that's a bad thing, anyhow it's outside the scope of this patch. I also get a br for empty descriptions when using filtered HTML, pretty sure it's related to http://drupal.org/node/100679. I don't wrap the description with any divs, can't see a need for it yet, and you could always override the theme function.
Comment #3
ChrisKennedy commentedCommitted - http://drupal.org/cvs?commit=48227
I realized afterward that this causes the title to come after the description in blocks. I'm not sure if this can be solved by using #weight, modifying the theme function, or something else.
Comment #4
anders.fajerson commentedWe need to look over the title (label) anyhow, it's not displayed correct in ranking/binary - list/block/individual.
Comment #5
(not verified) commented