Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

What would probably be easier is if we had a AHAH checkbox/textarea combo for adding/removing custom search engines.

David_Rothstein’s picture

Here's a newer version of the patch that uses jQuery to put a simple "Add more" link to show the textarea (and a "hide" link once it's revealed). It's also smart enough to show or hide the custom textarea by default depending on whether it already has entries in it.

Screenshots are attached as well.

David_Rothstein’s picture

Title: Move custom engine url textarea into a collapsed fieldset » Improve the display of the custom URL engine textarea

Status: Needs review » Needs work

The last submitted patch, xmlsitemap-809250-2.patch, failed testing.

David_Rothstein’s picture

Status: Needs work » Needs review

#2: xmlsitemap-809250-2.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, xmlsitemap-809250-2.patch, failed testing.

effulgentsia’s picture

Status: Needs work » Needs review

#2: xmlsitemap-809250-2.patch queued for re-testing.

effulgentsia’s picture

Category: task » feature

Status: Needs review » Needs work

The last submitted patch, xmlsitemap-809250-2.patch, failed testing.

psynaptic’s picture

Issue summary: View changes

Is this issue outdated/fixed or still relevant/active?

naveenvalecha’s picture

Status: Needs work » Needs review
FileSize
5.23 KB

Straight Reroll of #2 patch

gbisht’s picture

Rerolling the patch with the latest dev version.

gbisht’s picture

Spacing issue fixed in the patch added in #12

Status: Needs review » Needs work

The last submitted patch, 13: improve-custom-url-display-809250-13.patch, failed testing. View results