Comment 1 Felipe Heidrich 2011-03-21 13:42:32 EDT

> ScrollBar.setVisible() doesn't work for Tree and Table 

I believe that is not supposed to work anyway.

Comment 2 Lakshmi P Shanmugam 2011-03-31 07:06:13 EDT

(In reply to comment #1)
> > ScrollBar.setVisible() doesn't work for Tree and Table 
> 
> I believe that is not supposed to work anyway.

Hi Felipe,
But, Table/Tree.getVerticalBar()/getHorizontalBar().setVisible() works on all the platforms except Cocoa. Why is not supposed to work?

Comment 3 Felipe Heidrich 2011-03-31 10:49:47 EDT

(In reply to comment #2)
> Hi Felipe,
> But, Table/Tree.getVerticalBar()/getHorizontalBar().setVisible() works on all
> the platforms except Cocoa. Why is not supposed to work?

Because the scrollbar belongs to the control (table and tree in this case), it is the control's job to configure the scrollbar (max, min, selection, thumb, visibility).
The only way the user should be able to change something in the scrollbar is by using API available in the control.

For example, if the user wants the control to scroll verticall it has to use setTopIndex(), calling getVerticalBar().setSelection() is invalid.

Likewise, if the user does not want the scrollbars to show it has to create the control with SWT.NO_SCROLL.

Comment 4 Felipe Heidrich 2011-03-31 10:59:30 EDT

see Bug 341495

the problem is, if you say  Table/Tree.getVerticalBar()/getHorizontalBar().setVisible()  is supported. Then people will expect all API in Scollbar to be supported also.

Our position is that get* method can be used in a scrollbar for tree/table, but set* can not.

closing as wont fix.

Comment 5 Felipe Heidrich 2011-03-31 11:01:34 EDT

> But, Table/Tree.getVerticalBar()/getHorizontalBar().setVisible() works on all
> the platforms except Cocoa. Why is not supposed to work?

btw, I don't think this is true, if you hide the scrollbar using setVisible(false) it will hide but if you resize the control (or add/remove item) the control will eventually show the scrollbar again.