Bug #10892
open
Large number of VLAN/LANs make floating rules are to read
Added by Joe Slent over 3 years ago.
Updated about 3 years ago.
Affected Version:
2.4.5-p1
Affected Architecture:
All
Description
Since the following commit for 2.4.5, the interface column was introduced for floating rules: https://redmine.pfsense.org/projects/pfsense/repository/revisions/bf83fb9ab93435e605d28b67e0352d32ce63ba2d/diff/src/usr/local/www/firewall_rules.php
However this has dramatically decreased the usability of the WebGUI when analyzing floating rules in case of many interfaces being assigned to the rules due to the way they are listed:
echo implode('<br/>', $selected_descs);
I.e. If I assign a couple of rules to 10 interfaces each, I have eaten away my whole screen estate.
This issue is to enhance that, and we could do it in a couple of ways:
1. very easily, change the implode to comma separate the list, instead of using <br>, e.g. implode(', ', $selected_descs); This is my favourite. It compresses the raws without decreasing readability and it's a very safe change to make.
2. keep the <br>, but hide if more than 1 lines. Example https://css-tricks.com/almanac/properties/l/line-clamp/ (this should retain the page text-searchable).
3. any other css/display things with some link to show/hide
Files
Note there is a typo in the title -- meant to write "hard to read".
- Status changed from New to Pull Request Review
- Target version set to 2.5.0
- Status changed from Pull Request Review to Feedback
- Assignee set to Renato Botelho
- % Done changed from 0 to 100
PR has been merged. Thanks!
Tested in
2.5.0-DEVELOPMENT (amd64)
built on Sun Sep 20 06:59:15 EDT 2020
FreeBSD 12.2-PRERELEASE
and patch in
2.4.5-RELEASE-p1 (amd64)
built on Tue Jun 02 17:51:17 EDT 2020
FreeBSD 11.3-STABLE
Interfaces now appear as a comma separated imploded list as expected. Marking the ticket resolved.
- Category changed from Web Interface to Rules / NAT
Hello all,
for me this fix doesn't really solve the problem. It sometimes saves one or two lines but most of the times it still breaks to a new line for each interface. See attached image from a live-in-production system with LAN names edited but retaining the same numbers of chars. These aren't really long names.
I think I prefer my solution of simply using implode(', ', $selected_descs); (and if you want setting a style="max-width: 400px;" on the column listing the interfaces). This is not an elegant solution but at least I'm able to better see the rules (see below, same rule as above..much clearer).
I have also increased the width of this page (something like: <style>@media (min-width:1200px){.container{width:95%}}</style>). Maybe this can also be considered to make up for the space that was lost due to the addition of this column.
Also, an idea worth considering could be to display something like "All" if all interfaces are selected. Would save a lot of screen estate.
Thank you!
- Status changed from Resolved to New
- Assignee deleted (
Renato Botelho)
Hello Dan,
I think that is a very good idea! I would advise though to make it something like "All interfaces" as opposed to "ALL" to avoid conflict with someone who might have an interface called ALL.
I would also advise to pursue this ticket (10892) though irrespective of the "ALL" enhancement given that we would still have the issue if all but one interface are selected (like maybe a dozen LANs but not the WAN..). For this scenario I still think the implode+max-width+bigger-container-width combo might be good to consider (or a better solution if it can be found).
Thanks!
- Assignee set to Jared Dillard
- Priority changed from Normal to Low
- Target version changed from 2.5.0 to CE-Next
Also available in: Atom
PDF
Updated by 
Updated by Viktor Gurov 
Updated by 
Updated by 
Updated by 
Updated by 
Updated by Scott Long 
Updated by Anonymous 
