[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