Bug 300920

Summary:	minor issues in yast2-squid module
Product:	[openSUSE] openSUSE 10.3	Reporter:	Jiří Suchomel <jsuchome>
Component:	YaST2	Assignee:	Daniel Fiser <dfiser>
Status:	RESOLVED FIXED	QA Contact:	Jiri Srain <jsrain>
Severity:	Minor
Priority:	P5 - None
Version:	Beta 1
Target Milestone:	---
Hardware:	Other
OS:	Other
Whiteboard:
Found By:	---	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---
Attachments:	screenshot - alignment changed

Description Jiří Suchomel 2007-08-16 08:38:04 UTC

1. When I go back from Cache Setting 2, I get the message:

Unable to set correct permissions to directory /var/cache/squid.


Well, the chown operation failed because I'm running the module as a root, but is it necessary to do the action at this point, instead of during final write?


2. Squid - Cache Setting 2 dialog would look better if the widgets would be aligned somehow.

3. If you there is a checkbox under the textentry, align it to the left side (it is somewhere, but not on all places :-).

4. Tables should react at double-click - typically do the same action as Edit button (add `opt (`notify) to the `Table definition and check for table id as a result from UserInput)

5. "Squid - Logging + Timeouts Setting" IMHO the dialog label should not include the + sign . Try to center the contents, so the layout looks better

Comment 1 Daniel Fiser 2007-08-17 12:14:33 UTC

Created attachment 158196 [details]
screenshot - alignment changed

Comment 2 Daniel Fiser 2007-08-17 12:15:12 UTC

Ad 1. Yes, you are right. Operation is moved to final write.
Ad 2. Done - look at attached image. Do you think it's better?
Ad 3. Done.
Ad 4. Done.
Ad 5. Title is renamed but I don't think that centering the contents looks better.

Comment 3 Jiří Suchomel 2007-08-17 12:24:14 UTC

Thanks!

>  look at attached image. Do you think it's better?

I think it is, however I hope it could be even better :-)

What about moving all entries from Advanced Settings directly under Cache Directory and create a frame around it?

Comment 4 Daniel Fiser 2007-08-28 10:42:44 UTC

solved in yast2-squid-2.15.1

Comment 5 Jiří Suchomel 2007-08-28 11:00:11 UTC

Thanks!