• Resolved pha3z

    (@pha3z)


    I didn’t know if you have a place specifically for Suggested Feature posts, so I’m posting here.

    I mentioned a few days ago that you probably ought to change the default wrap from a SPAN to a DIV. But now I’ve got a suggestion that has greater importance.

    It appears that the output for the title doesn’t get wrapped in paragraph tags. This creates a problem. It makes it pretty much impossible to select the text block itself to do anything particularly fancy.

    For instance. I’ve got a gradient background on the SPAN container (which I’ve changed to a DIV). But I want to set a custom background image left-aligned and use it for a little drop-down icon. I want to set left padding on the paragraph so that multiple lines of text will not go underneath the icon. The icon will “hang” in the margin like an outside bullet point.

    Doing bullet-style icons as background images like this is extremely popular and effective technique. Its honestly the best way I’ve seen anyone discuss. There are hackier ways to get it done, but they’re just that — hackish and problematic.

    Would you be able to change collapseomatic to wrap title text in P tags? I’d suggest adding an option to enable or disable it and make ENABLED the default. We will always have more success if we default to best practices.

    http://wordpress.org/extend/plugins/jquery-collapse-o-matic/

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author Baden

    (@baden03)

    Couple of things.

    1. Have you read the documentation, especially the part about the startwrap/endwrap attributes?
    2. If you want to enable wrapping each of your titles in P tags, the pro version of the plugin allows the setting of default startwrap and endwrap tags for every expand element

    As for defaulting to best practices: As mentioned before the next release of the plugin will use DIV rather than SPAN tags to wrap the title text. However, we will not be default wrapping the title text in P tags, as we like to keep things as streamlined as possible. Default for the trigger text will be <div ...>title text</div>. If you want <div ...><p>title text</p></div> you can either use the start-/endwrap attributes described in the documentation, or upgrade to the pro version and set it as default for your install.

    But thanks for the feedback!

    Thread Starter pha3z

    (@pha3z)

    Aha!

    I failed at RTFM. Sorry!

    Good to know about that feature in Pro though. I will look at that, too!

Viewing 2 replies - 1 through 2 (of 2 total)

The topic ‘Suggested Feature’ is closed to new replies.