Mailinglist Archive: opensuse-bugs (19817 mails)

[bugzilla_noreply@xxxxxxxxxx]

https://bugzilla.novell.com/show_bug.cgi?id=390471

User sanfordarmstrong@xxxxxxxxx added comment
https://bugzilla.novell.com/show_bug.cgi?id=390471#c9





--- Comment #9 from Sanford Armstrong <sanfordarmstrong@xxxxxxxxx>  2008-05-18 
17:04:36 MST ---
Created an attachment (id=216267)
 --> (https://bugzilla.novell.com/attachment.cgi?id=216267)
Patch to add label to Install button and conditionally enable Apply button

This fixes the two big issues I have related to this bug:
 1) When yast starts the Apply button is disabled...at that point its
sensitivity is determined based on whether or not the Changes box is shown.
 2) The Install button is labeled.  I labeled it with "Install" to match the
tooltip, but perhaps "Mark for Install" or "Add" may be better?

The hacky part of this patch is that it gives access to the YGtkWizard to the
PackageSelector, so that the PackageSelector can control the enabled state of
the Apply button.  I could have done this with signals instead but I wanted to
get a working patch in.

This is my first time looking at yast source code, so I apologize in advance
for obvious blunders.


-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.