RFC: Remove custom RubberBand painter
AbandonedPublic

Authored by broulik on Feb 27 2016, 12:12 PM.

Details

Reviewers
hein
Group Reviewers
Plasma
Summary

This removes the custom RubberBand that is painted using QStyle and instead uses a traditional Rectangle.

We don't use "Desktop Widget Style" in Plasma anyway. The rubber band now follows the theme's highlight color and has a nice fade animation when it disappears.

Test Plan

Rubber band selection works fine.
Repeated quick moving and selecting doesn't make it go nuts, even when the fade out animation is still running.
Auto scroll in both popup and as containment is erratic; not sure if this is caused by this patch, however.

Using Tilain the rubber band is now white, for example, using Oxygen it's a fainter blue.

I did not yet remove the RubberBand cpp stuff as I wanted to get some feedback first before messing up my local repository :)

Diff Detail

Repository
R119 Plasma Desktop
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik updated this revision to Diff 2478.Feb 27 2016, 12:12 PM
broulik retitled this revision from to RFC: Remove custom RubberBand painter.
broulik updated this object.
broulik edited the test plan for this revision. (Show Details)
broulik added reviewers: Plasma, hein.
broulik set the repository for this revision to R119 Plasma Desktop.
broulik added a project: Plasma.
Restricted Application added a subscriber: plasma-devel. ยท View Herald TranscriptFeb 27 2016, 12:12 PM
hein edited edge metadata.Feb 28 2016, 5:54 AM

I'm not sure I'm on board with this - in absence of a proper Plasma Rubberband "standard", going with the QStyle one seemed (and seems) like a sane default for me. Some users also want rubberbands with just outlines, marching ants because of vision problems, whatever and this removes any sort of theming control from the rubberband. I think if we want to get rid of the CPP rubberband we need to add a rubberband element to the Plasma themes.

hein accepted this revision.Feb 28 2016, 5:56 AM
hein edited edge metadata.
This revision is now accepted and ready to land.Feb 28 2016, 5:56 AM
hein requested changes to this revision.Feb 28 2016, 5:59 AM
hein edited edge metadata.

Oops - accepted accidentally, meant the other one ...

This revision now requires changes to proceed.Feb 28 2016, 5:59 AM
broulik abandoned this revision.Apr 6 2016, 7:27 AM