CVS edit link for duckzland

I have created a module that retrieves node using ajax and display the content in a popup utilizing simplemodal jquery plugin
CommentFileSizeAuthor
#1 drupie_ajax_node_simplemodal.zip47.45 KBduckzland
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

duckzland’s picture

Status: Needs work » Postponed (maintainer needs more info)
Issue tags: -Module review
FileSize
47.45 KB

The module will provide an easy way to display a node in a popup using jQuery simplemodal plugin and ajax to retrieve the node body content only without the rest of the page (such as header, sidebar, footer etc).

When I created the module, the main purpose is to let user display contact us form and terms of service notification via a popup as simple as possible.

To utilize the module we need to call

theme('simplemodal', link-text-to-be-displayed, target-nid);

in our theme or modules. then the theme function will build the link using the link-text-to-be-displayed as the link text and pointing it to grab the content of the target-nid

AFAIK similar module to this is the popup modules.

apaderno’s picture

Status: Postponed (maintainer needs more info) » Needs work
Issue tags: +Module review

Hello, and thanks for applying for a CVS account. I am adding the review tags, and some volunteers will review the code, pointing out what it needs to be changed.

As per requirements, the motivation message should be expanded to contain more features of the proposed project. For themes, it should include also a screenshot of the theme, and (when possible) a link to a working demo site using the proposed theme; for modules, it should include also a comparison with the existing solutions.

duckzland’s picture

Status: Postponed (maintainer needs more info) » Needs work
Issue tags: +Module review

Could anyone point me for what needed to be fixed in this module?

apaderno’s picture

Status: Needs work » Closed (won't fix)
apaderno’s picture

Component: Miscellaneous » new project application
Issue summary: View changes
Status: Closed (won't fix) » Closed (duplicate)
Related issues: +#684330: duckzland [duckzland]