Bug 571919

Summary: CancelOKButtonBox() does not use `ButtonBox yet
Product: [openSUSE] openSUSE 11.2 Reporter: Keith Briscoe <bbregister>
Component: YaST2Assignee: J. Daniel Schmidt <jdsn>
Status: RESOLVED DUPLICATE QA Contact: Jiri Srain <jsrain>
Severity: Enhancement    
Priority: P4 - Low CC: jdsn, jsrain
Version: Final   
Target Milestone: ---   
Hardware: x86-64   
OS: openSUSE 11.2   
Whiteboard:
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---

Description Keith Briscoe 2010-01-19 16:42:34 UTC
User-Agent:       Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 6.1; WOW64; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; .NET CLR 1.1.4322; MS-RTC LM 8; MS-RTC EA 2)

Specifically, this applet uses Cancel/OK, and therefore looks out of place on a KDE desktop.  There may also be other places where this applet uses the wrong button order, and it may be worth reviewing every dialog for both button order and placement.

Please refer to the wiki about how to handle Gnome/KDE button order differences using the ButtonBox widget:
http://en.opensuse.org/YaST/Development/Misc/Button_Order

Reproducible: Always

Steps to Reproduce:
1.
2.
3.
Comment 1 J. Daniel Schmidt 2010-01-20 16:19:12 UTC
This is not a specific bug for the Online Update Configuration module, it is a general bug. This module uses the function Wizard::OpenOKDialog which in turn opens a GenericDialog( CancelOKButtonBox() ) and the function CancelOKButtonBox() in Wizard.ycp does not use the ButtonBox yet. If it would use it the buttons were automatically aligned correctly according to the desktop environment.

I'm lowering this bug to enhancement an keep it  -  or are you maintaining that code now, Jiri?
Comment 2 J. Daniel Schmidt 2010-01-20 16:21:12 UTC
Changing summary from:
YaST2 Software/Online Update Configuration uses Gnome button order in KDE
to:
CancelOKButtonBox() does not use `ButtonBox yet
Comment 3 J. Daniel Schmidt 2011-04-04 14:06:55 UTC
Closing as duplicate of a bug that tracks the switch to a consistent use of the ButtonBox in all YaST modules.

*** This bug has been marked as a duplicate of bug 684863 ***