Hi,

I've added some code to give the administrator the possibility of choosing on which node types to render the links; the patch (from "bzr diff") is attached.

There is at least one thing I know that should be added/refactored (e.g. the way to get the module options) but, first, I need to know if this is a feature you would like to add to the module, so I can work more on it.

jedihe

CommentFileSizeAuthor
#5 fasttoggle-645152-5.patch6.79 KBtraviscarden
#4 fasttoggle-645152-4.patch6.75 KBtraviscarden
#2 fasttoggle-645152-2.patch5.81 KBtraviscarden
#2 fasttoggle-645152-2.gif13.38 KBtraviscarden
checks.patch2.57 KBjedihe

Comments

jedihe’s picture

jedihe commented
Status: Active » Needs review

It makes more sense for a patch to be marked as "needs review", I think.

jedihe

traviscarden’s picture

traviscarden commented
Title: Settings for choosing on which node types will the links be rendered (patch included) » Add ability to enable/disable post settings and individual node links per content type
Version: 6.x-1.3 » 7.x-1.x-dev
StatusFileSize
new fasttoggle-645152-2.gif13.38 KB
new fasttoggle-645152-2.patch5.81 KB

Here's a patch that lets you set which node options are available on a per content type basis as well as add or remove links from the node links individually on a per content type basis. It moves the settings to the content type config screen as depicted below. As a consequence of the design change, stored settings are moved into separate variables around content types, which makes it possible to separate them into individual Features modules.

fasttoggle-645152-2.gif

traviscarden’s picture

traviscarden commented

Marked #1407222: Make Fast Toggle node links optional as a duplicate of this issue (and, by extension, #1201386: Option to hide links on pages and #1264254: Include option to hide toggle link from node.).

traviscarden’s picture

traviscarden commented
StatusFileSize
new fasttoggle-645152-4.patch6.75 KB

Here's an updated version of the patch that includes a little JavaScript UI enhancement that disables "Add to node links" options that aren't enabled "Togglable options". Of course the backend wouldn't add links to non-togglable options anyway, but it's a confusing user experience to be able to select them for inclusion as though it would add them.

traviscarden’s picture

traviscarden commented
StatusFileSize
new fasttoggle-645152-5.patch6.79 KB

Oops. I found a small issue (the form was presenting the "Comment settings" option even if comment module was disabled). Here's an updated patch.

nigelcunningham’s picture

nigelcunningham commented
Status: Needs review » Closed (fixed)

Patch applied (with small adjustments for previously applied patches). Thanks!

  • Commit 9efadf2 on master, 7.x-1.x by NigelCunningham: 
    Add content type support (#645152)

Apply the patch from:

http://drupal...