Re: [PATCH] zoneconf: reduce the width of inputs for vlanid

[Alexander Koch <ipfire@starkstromkonsument.de>]

[-- Attachment #1: Type: text/plain, Size: 2017 bytes --]

Hi,

the Problem occurs if you use four zones and the "ipfire-legacy"-Theme. 
But the "ipfire"-Theme also renders the vlanid-input pretty wide and 
wraps the line inside the cell. There actually is no need for this, as 
an VLAN-ID has a maximum length of four digits.

Some Screenshots: https://starkstromlahn.spdns.org/s/qb4iFZmxCQqxb7f

Regards, Alex


-------- Original Message --------
From: Michael Tremer [mailto:michael.tremer(a)ipfire.org]
Sent: Monday, 2 September 2019, 13:32 CEST
To: Alex Koch <ipfire(a)starkstromkonsument.de>
Cc: development(a)lists.ipfire.org
Subject: [PATCH] zoneconf: reduce the width of inputs for vlanid

Hi,

Just for my own illustration, could you send a screenshot?

Best,
-Michael

On 1 Sep 2019, at 23:47, Alex Koch <ipfire(a)starkstromkonsument.de> wrote:

The inputs for the vlanids are overlapping the borders of their cells 
(using a recent Firefox on Linux Mint, Android or Windows 7). This patch 
fixes this by limiting the width to a fixed value.

Signed-off-by: Alex Koch <ipfire(a)starkstromkonsument.de>
---
html/cgi-bin/zoneconf.cgi | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/html/cgi-bin/zoneconf.cgi b/html/cgi-bin/zoneconf.cgi
index 40239c14e..6b8642818 100644
--- a/html/cgi-bin/zoneconf.cgi
+++ b/html/cgi-bin/zoneconf.cgi
@@ -93,6 +93,10 @@ my $css = <<END
		text-align: center;
	}

+	input.vlanid {
+		width: 4em;
+	}
+
	#submit-container {
		width: 100%;
		padding-top: 20px;
@@ -432,7 +436,7 @@ foreach (@nics) {
					<option value="NATIVE" 
$access_selected{"NATIVE"}>$Lang::tr{"zoneconf access native"}</option>
					<option value="VLAN" $access_selected{"VLAN"} 
$vlan_disabled>$Lang::tr{"zoneconf access vlan"}</option>
				</select>
-				<input type="number" id="TAG $uc $mac" name="TAG $uc $mac" min="1" 
max="4095" value="$zone_vlan_id" $field_disabled>
+				<input type="number" class="vlanid" id="TAG $uc $mac" name="TAG $uc 
$mac" min="1" max="4095" value="$zone_vlan_id" $field_disabled>
			</td>
END
;
-- 
2.17.1