Re: [Kde-hardware-devel] Review Request 122343: Make sure that brightness up/down actions produce visible changes.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122343/#review75135 --- Ship it! Ship It! - Kai Uwe Broulik On Jan. 31, 2015, 7:04 nachm., Nikita Skovoroda wrote: --- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122343/ --- (Updated Jan. 31, 2015, 7:04 nachm.) Review request for Solid and Kai Uwe Broulik. Repository: powerdevil Description --- Atm when maxBrightness 100, brightness up/down actions could produce no visible change in some situations. For example, when maxBrightness = 1000, brightness = 496: * Brightness is shown as 50% (rounded to whole pecents). * «Brightness up» action increases the brightness by 1 to 497 and rounds it to the nearest step upwards 500, 50%. * The above logic is correct for, for example, brightness = 480 (so that it's raised to 500 = 50%, not to 530 = 53%), but in the described situation, when the brightness was manually set to 496 through a slider, the action does no visible change, which is unexpected. This patch solves this issue by comparing the visible percentage of the resulting brightness to the visible percentage of the original brightness and adding/subtracting one more step if there was no visible change. Diffs - daemon/powerdevilbrightnesslogic.h d7a02c5 daemon/powerdevilbrightnesslogic.cpp 5f61628 Diff: https://git.reviewboard.kde.org/r/122343/diff/ Testing --- Compiles. Trivial, should not break anything. Thanks, Nikita Skovoroda ___ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-hardware-devel
Re: [Kde-hardware-devel] Review Request 122343: Make sure that brightness up/down actions produce visible changes.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122343/ --- (Updated Feb. 1, 2015, 12:58 p.m.) Status -- This change has been marked as submitted. Review request for Solid and Kai Uwe Broulik. Repository: powerdevil Description --- Atm when maxBrightness 100, brightness up/down actions could produce no visible change in some situations. For example, when maxBrightness = 1000, brightness = 496: * Brightness is shown as 50% (rounded to whole pecents). * «Brightness up» action increases the brightness by 1 to 497 and rounds it to the nearest step upwards 500, 50%. * The above logic is correct for, for example, brightness = 480 (so that it's raised to 500 = 50%, not to 530 = 53%), but in the described situation, when the brightness was manually set to 496 through a slider, the action does no visible change, which is unexpected. This patch solves this issue by comparing the visible percentage of the resulting brightness to the visible percentage of the original brightness and adding/subtracting one more step if there was no visible change. Diffs - daemon/powerdevilbrightnesslogic.h d7a02c5 daemon/powerdevilbrightnesslogic.cpp 5f61628 Diff: https://git.reviewboard.kde.org/r/122343/diff/ Testing --- Compiles. Trivial, should not break anything. Thanks, Nikita Skovoroda ___ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-hardware-devel
Re: [Kde-hardware-devel] Review Request 122343: Make sure that brightness up/down actions produce visible changes.
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122343/ --- (Updated Янв. 31, 2015, 7:04 п.п.) Review request for Solid and Kai Uwe Broulik. Changes --- Minor change in the comments. Repository: powerdevil Description --- Atm when maxBrightness 100, brightness up/down actions could produce no visible change in some situations. For example, when maxBrightness = 1000, brightness = 496: * Brightness is shown as 50% (rounded to whole pecents). * «Brightness up» action increases the brightness by 1 to 497 and rounds it to the nearest step upwards 500, 50%. * The above logic is correct for, for example, brightness = 480 (so that it's raised to 500 = 50%, not to 530 = 53%), but in the described situation, when the brightness was manually set to 496 through a slider, the action does no visible change, which is unexpected. This patch solves this issue by comparing the visible percentage of the resulting brightness to the visible percentage of the original brightness and adding/subtracting one more step if there was no visible change. Diffs (updated) - daemon/powerdevilbrightnesslogic.h d7a02c5 daemon/powerdevilbrightnesslogic.cpp 5f61628 Diff: https://git.reviewboard.kde.org/r/122343/diff/ Testing --- Compiles. Trivial, should not break anything. Thanks, Nikita Skovoroda ___ Kde-hardware-devel mailing list Kde-hardware-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-hardware-devel