Honor configuration for Web Shortcuts

Review Request #108902 - Created Feb. 11, 2013 and submitted

Information
Dimitrios Christidis
rekonq
master
309470
Reviewers
rekonq
christidis
The issue is partly described in bug #309470. Rekonq does not honor two configuration options for Web Shortcuts: "EnableWebShortcuts" and "UsePreferredWebShortcutsOnly".

The first part of the patch attempts to address this issue. I am not particularly proud of the current solution; please advise me on proper variable naming and overall code structure. Furthermore, I am not aware of the correct way to compare two KService::Ptr objects. I ended up comparing the filename of the desktop entry, but is there a better way? Is it even correct? Lastly, should SearchEngine::favorites() be modified as well? If not, preferred engine will still appear in the suggestions, but won't be usable.

The second part of the patch is about the change of "Search Engines" to "Web Shortcuts". Should the conditional compilation directives be removed or is it too early?


This review will be split into two commits.
+ With EnableWebShortcuts=false:
Typing a web shortcut followed by the delimiter does not suggest any engine. Pressing Enter uses the default engine.

+ With EnableWebShortcuts=true and UsePreferredWebShortcutsOnly=true:
Engines that are not preferred are not used.

+ With EnableWebShortcuts=true and UsePreferredWebShortcutsOnly=false:
Works as before.
Andrea Diamantini
Andrea Diamantini
Dimitrios Christidis
Commit Hook
Dimitrios Christidis
Review request changed

Status: Closed (submitted)

Loading...