Fixes #7860.

@dopplershift I haven't written an image comparison test before, but I guess it would mean:

• Produce a simple plot example with check/radio buttons
• Generate the plot PNGs locally and commit them
• Add the image comparison test somewhere (say, test_widgets.py)

Does that sound about right?

That's it exactly.

Thanks for the input, added an image comparison test.

Since these are a) new tests and b) only broken on the new style, you will need to add style='default' to the image_comparison decorator or else you won't really be testing that the new style isn't broken.

Also, please amend the last commit so that we don't have an extra copy of the test images.

@QuLogic Ah, gotcha. Didn't realize the image_comparison decorator defaults to style='classic' - might be why it's failing.

btw, is there a way to display both sets of buttons (Radio and Check) and relocate them so they don't overlap?

Current coverage is 62.16% (diff: 100%)

Merging #7861 into master will increase coverage by 0.04%

@@             master      #7861   diff @@
==========================================
  Files           174        174          
  Lines         56072      56072          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          34831      34859    +28   
+ Misses        21241      21213    -28   
  Partials          0          0          

Powered by Codecov. Last update 7c16415...dedcac3

There is some general test flakiness (connectivity?) but it looks unrelated to this PR.

I have restarted the relevant tests but yes they look like network issues. Failing to install nose from github and failing some url based tests

btw, is there a way to display both sets of buttons (Radio and Check) and relocate them so they don't overlap?

Do you mean on the same figure? Use different values for rax; the widgets are placed within that Axes.

Thanks - actually can you hold off on merge for one sec? Seeing if I can combine the tests into 1 test instead of 2.

OK, I think this is good to merge now. Thanks!

Thanks @naoyak!

Backported to v2.0.x as 8beb59a

The backport of this seems to have broken CI somehow.

get_status was added to CheckButton in #6018 which is only in 2.1. If it's required for this patch, it might have to be reverted on v2.0.x.

@QuLogic the failing test is also from #6018. I'm not sure why only the test made it into v2.0.x.

Oh yes, you're right. Looks like it's just a merge error on backport then. Should be a bit easier to fix then.