ext Cédric Bellegarde wrote:
>> here a small patch to add an option to disable autoscolling.

+    _canEnableAutoScroll = !ReKonfig::disableAutoScroll() && 
!result.isContentEditable()  && result.linkUrl().isEmpty();

There are two spaces before the second &&. Not your fault obviously but 
the patch is an opportunity to fix it :)


+       <widget class="QCheckBox" name="kcfg_disableAutoScroll">
+        <property name="text">
+         <string>Disable auto scrolling</string>
+        </property>
+       </widget>

I would add a tooltip on the widget. If I have no idea what auto 
scrolling is, I would look for a tooltip to explain it.


cheers,
Benjamin
_______________________________________________
rekonq mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/rekonq

Reply via email to