Allow to disable blinking cursor completely
ClosedPublic

Authored by kfunk on May 9 2017, 10:23 PM.

Diff Detail

Repository
R135 Integration for Qt applications in Plasma
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kfunk created this revision.May 9 2017, 10:23 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 9 2017, 10:23 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
kfunk added a comment.May 9 2017, 10:24 PM

This is so trivial, yet I saw the demand for being able to disable the blinking cursor several times...

Am I missing something or why was the minimum 200?

kossebau added a subscriber: kossebau.EditedMay 9 2017, 10:26 PM
In D5791#108454, @kfunk wrote:

This is so trivial, yet I saw the demand for being able to disable the blinking cursor several times...

Am I missing something or why was the minimum 200?

Probably if blinking that was the most sane value.

What about doing instead an additional check for 0?

kfunk updated this revision to Diff 14355.May 9 2017, 10:37 PM

Allow 0, but otherwise keep the lower bound of 200

kfunk added a comment.Jun 5 2017, 12:08 PM

@Plasma devs: Comments?

There are some users out there who'd like to see this feature.

broulik accepted this revision.Jun 5 2017, 12:10 PM
This revision is now accepted and ready to land.Jun 5 2017, 12:10 PM
This revision was automatically updated to reflect the committed changes.