[PATCH] Support for bookmarklets

[Aurélien Gâteau]

Matt Williams wrote:
> On Thursday 29 January 2009 22:25:54 Aurélien Gâteau wrote:
>> (I sent this message to kfm-devel@ but did not receive a firm answer, so
>> I am reposting here)
>>
>> Hello,
>>
>> Attached is a patch which adds support for bookmarklets to Konqueror.
>>
>> In case you don't know about them, bookmarklets are one-line javascript
>> bookmarks which can be triggered by the user to do nifty things with the
>> current web page, for example enforcing a sane combination of background
>> and foreground colors. One of the most common bookmarklets is the one
>> provided by Del.icio.us to bookmark pages to their service. You can find
>> it here:
>>
>> http://delicious.com/help/bookmarklets
>>
>> The patch is quite simple, but requires linking libkdeinit_konqueror
>> with khtml.
>>
>> What do you think about it? Is it ok to commit?
> 
> Is this not what Tools -> Minitools does?

Yes, it is, but I find Minitools to be a pain to use compared to simply 
clicking a link in the bookmark bar. Additionally, when a 
not-so-experienced user browses to a page which suggest to just "drag 
the link to the bookmark bar" (cf Delicious website) it does not work 
with Konqueror.
I don't think we can expect websites to publish Konqueror-specific 
instructions, so it's better to adapt IMO.

Aurélien