Bug 432446

Summary:	Yast-gtk local security module "overview" is not readable and well organized
Product:	[openSUSE] openSUSE 11.1	Reporter:	Alberto Passalacqua <alberto.passalacqua>
Component:	YaST2	Assignee:	Jiří Suchomel <jsuchome>
Status:	RESOLVED FIXED	QA Contact:	Jiri Srain <jsrain>
Severity:	Minor
Priority:	P5 - None	CC:	dmacvicar, forgotten_h13THG8RK1, jsuchome, mmeeks
Version:	Beta 2
Target Milestone:	---
Hardware:	Other
OS:	Other
Whiteboard:
Found By:	---	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---
Attachments:	Yast local security overview screen Qt screenshot

Description Alberto Passalacqua 2008-10-06 03:48:48 UTC

In Yast-gtk local security module, the "Security overview" is not readable because the columns are not properly aligned with the corresponding title (See screenshot).

Regards,
Alberto

Comment 1 Alberto Passalacqua 2008-10-06 03:49:16 UTC

Created attachment 243539 [details]
Yast local security overview screen

Comment 2 Duncan Mac-Vicar 2008-10-06 09:04:22 UTC

May be a html problem, or the gtk widget is not that capable?

Comment 3 Forgotten User h13THG8RK1 2008-10-06 14:32:10 UTC

Duncan, no, yast2-gtk doesn't support tables, and there is no GTK HTML widget. There are tons of half-backed ones that require a lot of dependencies. Are either slow (mozilla) or unstable (like Novell's libgtkhtml for which anything not used for Evolution seems to crash the damn thing). And the APIs looks like mazes and none seems well maintained. Since we haven't found an instance where tables were needed in yast, we came up with our own html widget. But gtkhtml2 can be easily enabled at compilation, and we can easily plug into our html wrapper more widgets, so if you guys want, we can get support for tables.

It also looks like the security tool has no condition path for which to generate nice html for the ncurses UI:
 if (display_info["RichTextSupportsTable"]:true) {} else {}

Either way, assign this to the security tool guy, and I'll work in the GTK UI end if needed.

Comment 4 Jiří Suchomel 2008-10-06 14:37:10 UTC

Created attachment 243662 [details]
Qt screenshot

Ricardo, I don't know (yet) where's the problem, but in Qt this dialog looks correct...

Comment 5 Forgotten User h13THG8RK1 2008-10-06 14:56:14 UTC

Jiri, yast2-gtk doesn't support HTML tables. So the all thing looks broken.

We can't add table support for our custom HTML renderer. It would mean not only re-writing it, but writing a lot of code that we currently re-use from the text widget we base it from.

As stated, the GTK HTML rendering landscape isn't a very pretty sight. I'm sure in the near future Gnome will ship with a nice WebKit-based HTML renderer, so if possible, I would prefer to get by, and you guys would have to generate some nice HTML by testing "RichTextSupportsTable". (Don't you guys want to support ncurses anyway?)

Comment 6 Forgotten User h13THG8RK1 2008-10-07 18:45:23 UTC

Michael, it seems Gnome is now shipping with WebKit. I added the hooks and we can enable the thing by uncommenting the respective code at configure.in.in. However, I got a linkage error in a Beta2 system (something like "-liciui not found"). Want to test there?

Comment 7 Jiří Suchomel 2008-10-08 06:26:18 UTC

fixed in yast2-security-2.17.5 (GTK uses Table widget, just like ncurses)