[webmin-devel] [RFC] cosmetic change for edit_hdparm.cgi

Jean-Christophe Dubois <[email protected]>
Newsgroups gmane.comp.web.webmin.devel
Message-ID <[email protected]>
Hi,

When you are on the edit_hdparm form, there is no mention of the disk you are 
dealing with. This is not so bad when you have only one disk and most of the 
time you know for which disk you are editing the parametersµ.

However if you have numerous disks, it doesn't hurt to remind the user which 
disk it is playing with.

This is what this patch is doing.

JC

--- ./fdisk/edit_hdparm.cgi.org	2007-10-11 22:04:39.000000000 +0200
+++ ./fdisk/edit_hdparm.cgi	2007-10-12 18:32:46.000000000 +0200
@@ -33,7 +33,7 @@
 print(
 "<form action=apply_hdparm.cgi><table border cols=1 width=\"100%\"><input 
type=hidden name=drive value=", $d -> { 'device' }, ">",
     "<tr ", $tb, ">",
-    	"<td><b>", $text{ 'hdparm_label' }, "</b></td>",
+    	"<td><b>", $d -> { 'desc' }, " (", $d->{'device'},  "): ", 
$text{ 'hdparm_label' }, "</b></td>",
     "</tr><tr ", $cb, "><td>",
 	"<table width=\"100%\">",
 	    "<tr>",

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
-
Forwarded by the Webmin development list at [email protected]
To remove yourself from this list, go to
http://lists.sourceforge.net/lists/listinfo/webadmin-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.