Looks good to me. Padraig
On 01/07/09 13:54, Michal Pryc wrote: > Hello, > The webrev: http://cr.opensolaris.org/~migi/3978_v1/ > > Bug: Selecting package rows should check the package row as well > http://defect.opensolaris.org/bz/show_bug.cgi?id=3978 > > From the bug description: > > 1) Enable Install/Update when a row is selected. Installing multiple > packages will still require using the checkbox. My Yahoo mail reader > works that way and I didn't even realize it for a while. > > How this works: > If the user checks something then the install/update and remove buttons > works the way it were working. If the user doesn't select any checkbox > in the package row, then the install/update button becomes enabled when > the selected package may be updated/installed and it's the same with the > remove button. > > Switching categories/sub-categories, search, status combobox or > repositories needs call to enable disable button functions since the > selection most likly is being changed, but it's only in the case > checkboxes are not checked (self.selected == 0), that is why we need the > lines: > 686-688, 711-713, 746-748, 756-758 > > _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
