Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

firewall: put category filter selection on top of the list/screen #2182

Closed
jodelkoenig opened this issue Feb 8, 2018 · 13 comments
Closed

firewall: put category filter selection on top of the list/screen #2182

jodelkoenig opened this issue Feb 8, 2018 · 13 comments
Assignees
Labels
cleanup Low impact changes

Comments

@jodelkoenig
Copy link

jodelkoenig commented Feb 8, 2018

Hi,

would it be possible to move the field to set the filter for rule maintenance on top of the list/screen rather than at the bottom? It'll be great to just select the filter and maintain filter specific rules right away. Rather than scrolling all the way down, set filter, and scrolling all the way up again after a filter has been applied.

Thanx

@jodelkoenig jodelkoenig changed the title Feature Request: Firewall Rule - Maintenance Page - Put Filter Selection on Top of the List/Screen (Feature Request) Firewall Rule - Maintenance Page - Put Filter Selection on Top of the List/Screen Feb 8, 2018
@AdSchellevis
Copy link
Member

@jodelkoenig you mean the category filter?

@jodelkoenig
Copy link
Author

jodelkoenig commented Feb 8, 2018

Indeed. The category filter :)

While I was thinking about it. It might make sense to put the whole section on top of the list. Meaning the line with the category filter as well as the buttons on the right (Move selected rules to end / Delete selected rules / Add new rule).

The legend should remain at the bottom.

@jodelkoenig
Copy link
Author

One more thing, related to category filer:

It'll be great if a selection would survive the switch between interfaces. With 18.1 this is no longer the case. With the latest 17.x the selection was still valid, even if I changed the - via tab - from LAN to OpenVPN or so.

Thanks :)

@AdSchellevis AdSchellevis self-assigned this Feb 13, 2018
@AdSchellevis AdSchellevis added the bug Production bug label Feb 13, 2018
@AdSchellevis
Copy link
Member

ok, this is work in progress. ac4f5a1 fixes the "sticky" categories like it was, but creates issues in the menu system (matching).

To be continued...

@AdSchellevis
Copy link
Member

Functional it would indeed be better to move the filter to the top of the page, optical it probably looks a bit better on the bottom.

I'm not sure this is the best version, but can you try the commits in this issue?

opnsense-patch ac4f5a1 5bc9fdf 0752629

@fichtner
Copy link
Member

What if you put the drop down where the "Add xxx" buttons would be?

@fichtner
Copy link
Member

screen shot 2018-02-16 at 7 10 43 am

@jodelkoenig
Copy link
Author

I like your idea, fichtner. This looks much cleaner.

Thanks AdSchellevis. For both items. Category Filter at the top of the page and persistency of filter cross interface pages.

@fichtner fichtner added the cleanup Low impact changes label Feb 16, 2018
@fichtner fichtner added this to the 18.7 milestone Feb 16, 2018
@fichtner fichtner removed the bug Production bug label Feb 16, 2018
@fichtner fichtner changed the title (Feature Request) Firewall Rule - Maintenance Page - Put Filter Selection on Top of the List/Screen firewall: put category filter selection on top of the list/screen Feb 16, 2018
@AdSchellevis
Copy link
Member

@fichtner better indeed, shall I change it accordingly

@fichtner
Copy link
Member

yes please, I only fiddled with the browser's DOM

@AdSchellevis
Copy link
Member

it was a bit of fiddling, but this should be it
image

@fichtner
Copy link
Member

Very nice, thanks!

fichtner pushed a commit that referenced this issue Feb 16, 2018
(cherry picked from commit ac4f5a1)
(cherry picked from commit 5bc9fdf)
(cherry picked from commit 0752629)
(cherry picked from commit 97cdecb)
@jodelkoenig
Copy link
Author

Indeed. Thanks a lot! Cool :)

@fichtner fichtner removed this from the 18.7 milestone Feb 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Low impact changes
Development

No branches or pull requests

3 participants