anthonyfieroni added inline comments.

INLINE COMMENTS

> KoKineticScroller.cpp:28
> +{
> +    KConfigGroup config = KSharedConfig::openConfig()->group("");
> +    int sensitivity = config.readEntry("KineticScrollingSensitivity", 75);

Let's have some name, KoKineticScroller or something. I also notice that you 
never save option, are they user configurable? It's used always default ones.

> KoKineticScroller.cpp:92
> +{
> +    KConfigGroup config = KSharedConfig::openConfig()->group("");
> +    int gesture = config.readEntry("KineticScrollingGesture", 0);

Same name here.

REPOSITORY
  R8 Calligra

REVISION DETAIL
  https://phabricator.kde.org/D16721

To: ognarb, #krita, #calligra:_3.0
Cc: anthonyfieroni, Calligra-Devel-list, dcaliste, cochise, vandenoever

Reply via email to