- User Since
- Jun 3 2015, 8:56 AM (277 w, 2 d)
May 25 2020
May 15 2020
May 14 2020
May 12 2020
Thanks for the patch! I had not noticed the problem before :)
May 11 2020
May 8 2020
May 7 2020
May 6 2020
May 5 2020
I'm starting to think that we need an option for enabling/disabling this change/feature. I would not want to have the extra space between the lines, but at the same time I can see that actually not seeing the whole character is not an acceptable situation...
May 4 2020
I would like to use "this" as parent for the QShortcuts in stead of the parent of KSaneWidget that actually might be nullptr.
May 3 2020
It seems it could be quite hard to get this to work with only the QPushButton, so your idea with a separate QShortcut is a good workaround.
This seems to be the reason...
@ltoscano Yes sure, I have not made a release in over two years :( so waiting a bit more should not be a problem ;)
May 2 2020
Apr 27 2020
That should work, but I can confirm that it does not work in libksane at the moment.
Apr 26 2020
Yes please add that to libksane :)
Sorry for the faaaar to late response...
Apr 13 2020
I would like to eventually have the "ctrl-..." along sides the auto generated shortcuts. This is basically a workaround to gett a shortcut (hopefully "Alt-s") for the scan button in stead of no shortcut at the moment.
Apr 12 2020
I'm not a big fan of the assigning of shortcuts with &, because it becomes a big hassle to keep track of what has been set where and translations makes things even more complicated...
Mar 1 2020
Both Konsole and Kate, take the default font from the global fixed with font settings. Once you change the setting in the application, it no-longer follows the global setting. I have also noticed that the default font settings seem to be a bit distro specific.
Feb 19 2020
Feb 16 2020
Feb 14 2020
Feb 2 2020
Jan 21 2020
Jan 16 2020
Jan 13 2020
Thank you both for the reviews :)
But I'm fine without the changes also :)
Jan 9 2020
- Fix inch as display unit: 1 inch = 25.4 mm (it was not cm->inch)
Everything is big in America :)
Everything is big in America ;) (Legal should be 8.5 X 14 inch)
@antonarnold I'm not exactly sure what is wrong in those ;)
Jan 8 2020
@antonarnold Good bug catch! All the scan sources I have here, have the same size so I did never reach that code path...
- Set the page size to "Custom" when the source size changes.
Jan 3 2020
Ah OK :). The spinbox+slider is a reused widget that is used for all numeric properties in libksane :)
Jan 2 2020
The sliders provide the size of the scanned area in numeric form and there is the possibility to fine-tune the scan area without having to be too steady on the hand when doing the selection. So I do see a value in having the sliders in some form...
I'm not 100% happy with having the size sliders there directly after the color options.
Jan 1 2020
Dec 27 2019
Dec 26 2019
Dec 17 2019
Dec 11 2019
Do you want to send a new version or should I Just remove the separator removal and use "... = QLocale::toFloat(tmp);" and commit in your name?
Dec 10 2019
Thanks for these fixes :)
Dec 7 2019
Yes it works :)
Dec 5 2019
I have been a bit reluctant to add the predefined page sizes, but maybe it is time to rethink it...
Dec 4 2019
I would agree that a UNIX/POSIX env var is not the most logical thing to use on Windows, but how do we ensure that we are in sync with ki18n? The usage of QLocale in ki18n looks like it is just adding extra languages to a list of languages, not actually using QLocale to determine what catalog to load. At least LANGUAGE overrides any QLocale settings. Should ki18n not be moved to using QLocale and setDefault()?
I was actually wondering why the F4 shortcut was put to Focus/Defocus Terminal and not to the "Show Terminal" action. The later action works the same as Dolphin (except that the focus to the terminal got broken at some point this fall or summer).
So the conclusion is that we are not using setDefault() yet and ki18n & gettext uses the LANGUAGE env so the logical thing, for now, is to also use LANGUAGE env here in KCoreAddons for the plugin metadata translations?
Dec 3 2019
Dec 1 2019
ki18n does not seem to use QLocale to get the language for translation, but uses LANGUAGE env. Also gettext uses the env variable.
Nov 29 2019
BTW should this also be done to to other options setting numbers?
Nov 28 2019
Nov 24 2019
Scratch that 19.12 comment, the 19.12 feature freeze was already the 14th... The bug fixes can go in as they are bug fixes ;) so no hurry.
That is absolutely fine!
Nov 17 2019
I hope I did not scare you off with my comments. These fixes are really welcome!
Oct 30 2019
Oct 29 2019
Isn't the change in setOptVals() the only change needed for fixing the scan option order problem?
Generally good changes :)
Oct 27 2019
Hmm yes, that order in which we restore the saved options could have a priority order. We could just check if the map contains scan source and execute that first. Then check for mode and resolution and then after that the rest.
Oct 25 2019
Thanks for your review request! :)
Jul 17 2019
Jul 15 2019
@ngraham Unfortunately there is no way with UI files to dynamically modify the layout depending on the size of the container, so munging the layout in C++ is the best way I know so far :)
Jul 10 2019
Sorry for the late review.
Jun 26 2019