Bug 392807

Summary: Software Repository view area is too small
Product: [openSUSE] openSUSE 11.0 Reporter: Brian Merrell <bgmerrell>
Component: YaST2Assignee: Ladislav Slezák <lslezak>
Status: RESOLVED FIXED QA Contact: Jiri Srain <jsrain>
Severity: Minor    
Priority: P5 - None Keywords: Usability
Version: Beta 3   
Target Milestone: ---   
Hardware: x86   
OS: openSUSE 11.0   
Whiteboard:
Found By: Integration Test Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: Configured Software Repository window in GNOME

Description Brian Merrell 2008-05-20 19:06:27 UTC
Created attachment 217009 [details]
Configured Software Repository window in GNOME

DETAILS

The "Configured Software Repositories" window that opens when opening "Software Repositories" from yast2 has a TreeView that lists the configured repositories, but the TreeView is too small.

REPRO

1.  run yast2
2.  open "Software Repositories"

RESULTS

The TreeView showing the list of repositories is far too small.  The user is only able to see 1 1/2 rows.  See attached screenshot.

EXPECTED

I would think the user should be able to see at least a few rows comfortably.  

COMMENTS

The screenshot is taken in GNOME.  When I tested a minimal X installation (fvwm), this was not an issue.  The window opened quite large in fact.
Comment 1 Brian Merrell 2008-05-21 17:03:07 UTC
Just as a note, this was tested on UWXGA (1920x1200) resolution, so there is plenty of desktop space.
Comment 2 Michael Meeks 2008-05-22 13:57:41 UTC
Well - I suspect this is some underlying problem in the ycp - that we simply don't see in the Qt frontend because it sets the window size to be a fixed, larger proportion of the screen. yast2-gtk by contrast, tries to do a size calculation based on the underlying containers / weights etc. in the ycp.

So - I'd push this back to the yast2 maintainers & ask for some VWeights in repositories.ycp around line 590.
Comment 3 Ladislav Slezák 2008-05-23 13:52:23 UTC
The fix was really easy:

Index: src/clients/repositories.ycp
===================================================================
--- src/clients/repositories.ycp	(revision 47834)
+++ src/clients/repositories.ycp	(working copy)
@@ -366,6 +366,7 @@
     term contents =
         `VBox(
 	      `HBox(
+		    `VSpacing(9),
 		    `Table(`id(`table),`opt(`notify, `immediate),
 			    `header(
 				// table header - priority of the repository - keep the translation as short as possible!


Tested in Gtk, Qt and ncurses UI, works as expected. At least 6 repositories are displayed in the Gtk UI now.

Fixed in yast2-packager-2.16.52.