[Vimperator] Addon search keywords

Ted Pavlic ted at tedpavlic.com
Sun Jul 13 07:43:26 PDT 2008


All of my shortcuts use GET strings, and so the POST stuff isn't 
important to me. I didn't realize FF supported creating POST shortcuts.

This patch seems to work as expected. :) (NOTE: I haven't actually 
tested it on a shortcut that only exists in del.icio.us; at the moment, 
all of my FF bookmarks match my del.icio.us bookmarks...)

--Ted

Kipling Inscore wrote:
> I never should have touched that regex. I thought I changed it 
> back...and in the attached patch I have, finally. I think that's it 
> for this section but please let me know if you find any more bugs,
> and again I cannot test with POST data because I don't even know
> where to find or how to create such a keyword or alias.
> 
> On Fri, Jul 11, 2008 at 8:17 PM, Ted Pavlic <ted at tedpavlic.com>
> wrote:
>> I was using the patch attached to the 6/25/08 message (i.e., the
>> message that I was responding to).
>> 
>> From what I can tell from the thread, it's the most recent patch.
>> 
>> --Ted
>> 
>> Kipling Inscore wrote:
>>> Is that with the most recent patch in the thread? I realize this
>>> was the case in the first patch but I didn't think that problem
>>> was still there. I'll resubmit if so. Also there are some cases I
>>> can't test yet as I don't have engines with POST data, nor have I
>>> used vimperator's search-engine-dependent suggestions (which I
>>> think is somewhat independent of this anyway).
>>> 
>>> On Fri, Jul 11, 2008 at 6:36 PM, Ted Pavlic <ted at tedpavlic.com> 
>>> wrote:
>>>> NOTE: I just applied this patch to 1.2pre, and now every time I
>>>> try to open a URL, it goes to Google UNLESS it's prefixed with 
>>>> "http://"
>>>> 
>>>> For example,
>>>> 
>>>> t www.google.com
>>>> 
>>>> brings up
>>>> 
>>>> 
>>>> 
>>>> http://www.google.com/search?q=www.google.com&ie=utf-8&oe=utf-8&aq=t&rls=org.mozilla:en-US:official&client=firefox-a
>>>> 
>>>> 
>>>> 
>>>> :( So I guess the patch isn't quite ready.
>>>> 
>>>> --Ted
>>>> 
>>>> 
>>>> Ted Pavlic wrote:
>>>>> It looks like this patch isn't applied to the tip of the
>>>>> repo.
>>>>> 
>>>>> Any plans for that?
>>>>> 
>>>>> --Ted
>>>>> 
>>>>> Kipling Inscore wrote:
>>>>>> On Wed, Jun 25, 2008 at 9:53 AM, Kipling Inscore
>>>>>> <k at bijna.net> wrote:
>>>>>>> On Wed, Jun 25, 2008 at 9:43 AM, Kipling Inscore 
>>>>>>> <k at bijna.net> wrote:
>>>>>>>> New patch below...
>>>>>>> Sorry. Lines got auto-wrapped in the previous
>>>>>>> message--patch is likely not to work. Attached now
>>>>>>> instead of inline.
>>>>>> ...and it won't work for other reasons. I hadn't thoroughly
>>>>>>  tested it...accidentally dropped "[" from the URL test
>>>>>> regexp (which I was thinking maybe should be changed to ://
>>>>>> rather than . or : or /).
>>>> -- Ted Pavlic <ted at tedpavlic.com> 
>>>> _______________________________________________ Vimperator
>>>> mailing list Vimperator at mozdev.org 
>>>> https://www.mozdev.org/mailman/listinfo/vimperator
>>>> 
>>> _______________________________________________ Vimperator
>>> mailing list Vimperator at mozdev.org 
>>> https://www.mozdev.org/mailman/listinfo/vimperator
>>> 
>> -- Ted Pavlic <ted at tedpavlic.com> 
>> _______________________________________________ Vimperator mailing
>> list Vimperator at mozdev.org 
>> https://www.mozdev.org/mailman/listinfo/vimperator
>> 
>> 
>> ------------------------------------------------------------------------
>> 
>> 
>> _______________________________________________ Vimperator mailing
>> list Vimperator at mozdev.org 
>> https://www.mozdev.org/mailman/listinfo/vimperator

-- 
Ted Pavlic <ted at tedpavlic.com>


More information about the Vimperator mailing list