* [PATCH] zoneconf: reduce the width of inputs for vlanid @ 2019-09-01 22:47 Alex Koch 2019-09-02 11:32 ` Michael Tremer 0 siblings, 1 reply; 4+ messages in thread From: Alex Koch @ 2019-09-01 22:47 UTC (permalink / raw) To: development [-- Attachment #1: Type: text/plain, Size: 1259 bytes --] 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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] zoneconf: reduce the width of inputs for vlanid 2019-09-01 22:47 [PATCH] zoneconf: reduce the width of inputs for vlanid Alex Koch @ 2019-09-02 11:32 ` Michael Tremer 2019-09-03 18:50 ` Alexander Koch 0 siblings, 1 reply; 4+ messages in thread From: Michael Tremer @ 2019-09-02 11:32 UTC (permalink / raw) To: development [-- Attachment #1: Type: text/plain, Size: 1486 bytes --] 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 > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] zoneconf: reduce the width of inputs for vlanid 2019-09-02 11:32 ` Michael Tremer @ 2019-09-03 18:50 ` Alexander Koch 2019-09-05 9:33 ` Michael Tremer 0 siblings, 1 reply; 4+ messages in thread From: Alexander Koch @ 2019-09-03 18:50 UTC (permalink / raw) To: development [-- 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 ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] zoneconf: reduce the width of inputs for vlanid 2019-09-03 18:50 ` Alexander Koch @ 2019-09-05 9:33 ` Michael Tremer 0 siblings, 0 replies; 4+ messages in thread From: Michael Tremer @ 2019-09-05 9:33 UTC (permalink / raw) To: development [-- Attachment #1: Type: text/plain, Size: 2460 bytes --] Hi, > On 3 Sep 2019, at 19:50, Alexander Koch <ipfire(a)starkstromkonsument.de> wrote: > > 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 The old theme is practically unmaintainable, but good to take a little bit of care here as well. Acked-by: Michael Tremer <michael.tremer(a)ipfire.org> -Michael > > 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 > > > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-09-05 9:33 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-09-01 22:47 [PATCH] zoneconf: reduce the width of inputs for vlanid Alex Koch 2019-09-02 11:32 ` Michael Tremer 2019-09-03 18:50 ` Alexander Koch 2019-09-05 9:33 ` Michael Tremer
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox