Author: sh-sh-sh Date: Fri Oct 19 13:25:03 2007 New Revision: 41485 URL: http://svn.opensuse.org/viewcvs/yast?rev=41485&view=rev Log: Reverted bubli's last change: Use virtual function that some derived classes override, not changing the item directly without telling the derived class about it. As the comment in that code block says, it's an exceptional case and an app error anyway if this code branch ever gets executed. But when it is, let's do it correctly. Modified: branches/tmp/sh/mod-ui/core/libyui/src/YSelectionWidget.cc Modified: branches/tmp/sh/mod-ui/core/libyui/src/YSelectionWidget.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/core/libyui/src/YSelectionWidget.cc?rev=41485&r1=41484&r2=41485&view=diff ============================================================================== --- branches/tmp/sh/mod-ui/core/libyui/src/YSelectionWidget.cc (original) +++ branches/tmp/sh/mod-ui/core/libyui/src/YSelectionWidget.cc Fri Oct 19 13:25:03 2007 @@ -178,10 +178,10 @@ { //deselect old item oldSelectedItem->setSelected( false ); - //explicitely set a new item to selected state + //and inform parent widget that a new item has been selected //(i.e. override initial selection in construction like this: //`item(`id(1),"foo"), `item (`id(2), "bar, true) ) - item->setSelected( true ); + selectItem( item ); } } -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org