Rework compression quality settings
ClosedPublic

Authored by MrPepe on Mon, Sep 30, 1:49 PM.

Details

Summary

Removed ticks from the slider.
Substituted the label with a SpinBox, also adding a percent sign.

Test Plan

Set compression quality using both the spinbox and the slider -> Quality of saved images changes accordingly

Diff Detail

Repository
R166 Spectacle
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
MrPepe created this revision.Mon, Sep 30, 1:49 PM
Restricted Application added a project: Spectacle. · View Herald TranscriptMon, Sep 30, 1:49 PM
MrPepe requested review of this revision.Mon, Sep 30, 1:49 PM
MrPepe updated this revision to Diff 67160.Wed, Oct 2, 2:45 AM
MrPepe retitled this revision from Remove ticks from compression quality slider and add percent sign to its label to Rework compression quality settings.
MrPepe edited the summary of this revision. (Show Details)
MrPepe edited the test plan for this revision. (Show Details)
MrPepe added a reviewer: ngraham.
ngraham accepted this revision.Wed, Oct 2, 4:55 PM

Thanks for the patch! This is a nice complement to the Gwenview one (D24309) to make these user interfaces more consistent.

This revision is now accepted and ready to land.Wed, Oct 2, 4:55 PM
ngraham edited the summary of this revision. (Show Details)Wed, Oct 2, 4:56 PM

BTW:

The slider knob does not scale with bigger font sizes and I couldn't find an easy solution for that. Is there one?

That would probably be a bug in the Breeze widget style.

This revision was automatically updated to reflect the committed changes.