This patch adds new keyboard shortcuts to KRuler: one can now use Alt+Arrow keys to move the mouse cursor. Combined with the existing Arrow keys shortcuts which moves the ruler it makes it easy to measure lengths precisely.
Details
Details
Diff Detail
Diff Detail
- Repository
- R375 KRuler
- Lint
Lint Skipped - Unit
Unit Tests Skipped
klineal.cpp | ||
---|---|---|
585 | The call to update was seemingly not needed before (I guess it doesn't hurt), why introduce it here? |
klineal.cpp | ||
---|---|---|
585 | It was buggy before :) Moving the ruler window with the arrow keys did not update the length indicator. update() is only needed when moving the window because the code does not call update() on move events, whereas moving the cursor triggers mouse events, which trigger update() calls. |
Comment Actions
Sure, add it. I didn't intend to block it, but the commit message didn't mention the bug fix (might be worthwile), I think this is certainly good, I don't know the code, you are much more likely to get it right Aurélien :)