Bug in netmask dropdown for FW rules

Started by verfluchten, October 29, 2024, 07:21:41 PM

Previous topic - Next topic
I typed in the IP address, tabbed into the netmask dropdown and entered 11. It shows 127.

It generally switches automatically to IPv4 or IPv6 netmasks/prefix lengths depending on the content of the address field. Did you perchance by accident type a colon in your address?
Deciso DEC750
People who think they know everything are a great annoyance to those of us who do. (Isaac Asimov)

No colons.
I paste the net from ipinfo.com: aa.bb.cc.dd/ee, backspace the last 3 characters, and tab into the netmask dropdown. It should be able to figure out that it is IPv4. So, it is bugged.

Then please create an issue on github.
Deciso DEC750
People who think they know everything are a great annoyance to those of us who do. (Isaac Asimov)

This is opnsense forum, I've already reported it here, and I am done.

This is the OPNsense community forum. Users interacting with users. This is neither an official support platform nor a bug tracker. If you want to notify "the OPNsense project" of a bug, the place is here:

https://github.com/opnsense/core/issues
Deciso DEC750
People who think they know everything are a great annoyance to those of us who do. (Isaac Asimov)

The netmask dropdown in the firewall rule screen is INCREDIBLY bugged and HORRIBLY user un-friendly.
So far the most problems are with low netmasks in the ballpark of 12. It just does not want to enter them and results in 11 or 13, i.e. 1 up or 1 down.
And to add insult to injury, your custom control does not position the list on the selected item, when it is open.
Folks, stop your sophisticated programming and just give us a plain-Jane dropdown, 32-item tall. Whatever you developed for this control is not working. Did you test at all?

Which browser are you using?

FWIW, I'm not experiencing any of the issues you've reported lately (at least the ones I've seen).
Specifically here, once you tab out of address field, the suffix range is 1-32 unless there's a colon in the previous field (in which case the range is 1-128). The suffix itself doesn't seem to be adjusted unless you switch between IPv4 (32) and IPv6 (64).
If you open the suffix dropdown with a mouse, the selection is whatever value was there.
If you open it with the up or down arrow, the selection is +1 or -1 whatever value was there. Not unreasonable.

In any case, at least in the rules page (where I've used these controls the most), I have no complaints.

Plus, repeating myself - issue on github including screen shots and steps to reproduce, please.

Anything else is just "man yelling at cloud".
Deciso DEC750
People who think they know everything are a great annoyance to those of us who do. (Isaac Asimov)

Quote from: EricPerl on November 02, 2024, 08:51:32 PM
Which browser are you using?
That should not matter. Coding appliance UI for a specific browser would have been EXTREMELY lame.