For the project we are working on, we don't need or require the autocomplete functionality. But, as written, there is no way to switch it off.

This patch is a proposed solution that adds a configuration option that enables the autocomplete functionality.

CommentFileSizeAuthor
#4 gss-ability-to-disable-autocomplete-2293169-4.patch1.87 KBvalentin schmid
#2 gss-ability-to-disable-autocomplete-2293169-2.patch1.54 KBvalentin schmid
#1 2293169_gss_autocomplete_option.patch1.62 KByang_yi_cn
gss_fix_2.patch1.72 KBJoseph_Fowler

Comments

yang_yi_cn’s picture

yang_yi_cn commented
Version: 7.x-1.10 » 7.x-1.11
Status: Active » Needs review
StatusFileSize
new 2293169_gss_autocomplete_option.patch1.62 KB

yeah, we have the same need as well (to disable autocomplete) as we want to avoid using jQuery UI, which is quite heavy. But having the auto-complete feature enabled by default will force the loading of jQuery UI, ui.widget, ui.autocomplete etc.

The original patch doesn't work for 7.x-1.11 so I re-rolled it.

valentin schmid’s picture

valentin schmid commented
StatusFileSize
new gss-ability-to-disable-autocomplete-2293169-2.patch1.54 KB

Re-rolled the patch for 7.x-1.x-dev.

ram4nd’s picture

ram4nd commented
Version: 7.x-1.11 » 7.x-1.x-dev
Status: Needs review » Needs work

Variable has to be in uninstall hook as well, reroll it against dev.

valentin schmid’s picture

valentin schmid commented
StatusFileSize
new gss-ability-to-disable-autocomplete-2293169-4.patch1.87 KB

Re-rolled patch again.
Added variable gss_autocomplete to the uninstall hook.

valentin schmid’s picture

valentin schmid commented
Status: Needs work » Needs review
ram4nd’s picture

ram4nd commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.