No scrollbar in structures and isodoses tabs

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Open a file with a lot of structures

What is the expected output? What do you see instead?
It's impossible to see all the structures.

What version of the product are you using? On what operating system?
Dicompyler 0.4a1 under Fedora (linux).

Please provide any additional information below.
Attached a patch using wx.ScrolledWindow

Original issue reported on code.google.com by gllm.ber...@gmail.com on 28 Mar 2011 at 4:14

Attachments:

• patch.diff

[GoogleCodeExporter]

Thank you for the patch. I was unable to get this to work previously because I 
was trying to use a wx.ScrolledPanel. It seems that it works with a 
wx.ScrolledWindow though!

However, there is only one issue that I am seeing: a horizontal scrollbar shows 
up when the list of structures is less than the window height. Interestingly 
enough, this does not occur on the isodose list.

I will try to see if I can fix that and will then commit the patch.

Original comment by bastula on 28 Mar 2011 at 6:29

• Changed state: Accepted

Attachments:

• horizontal_scrollbar.png
• no_horizontal_scrollbar.png

[GoogleCodeExporter]

The previous patch works only with the structure tab.

Here is a new patch that should works for any ColorCheckListBox.

I don't have any issue with the horizontal scrollbar under linux.

Original comment by gllm.ber...@gmail.com on 29 Mar 2011 at 2:15

Attachments:

• patch.diff
• dicompyler-structures.png
• dicompyler-isodoses.png

[GoogleCodeExporter]

This issue was closed by revision ea22a0c2fadd.

Original comment by bastula on 29 Mar 2011 at 4:08

• Changed state: Fixed

[GoogleCodeExporter]

The patch seems to work fine. I intermittently get issues where the scrollbar 
has a wider range than the text, but I think that is a wxWidgets issue. It 
sometimes happens with random patients and sometimes doesn't. Not a big deal, 
but definitely your patch is infinitely more useful than not having a scrollbar 
at all.

Thanks again.

Original comment by bastula on 29 Mar 2011 at 4:15