Code review comment for lp:~cmulk/cardapio/duckduckgo-plugin

Revision history for this message
Thiago Teixeira (tvst) wrote :

Looks good!

Regarding your comment about the result_limit variable: that variable can't be set in the constructor because sometimes it changes after the plugin has been loaded. For instance, when doing a keyword search, result_limit changes from 4 to 20 (those are default values).

review: Approve

« Back to merge proposal