Theme Tukan - edited version

[chemlud]

See first attachment, a short list of rules.

Now I disabeled the second (!) rule, and before pressing "Apply" it looks (icon color) as if it is already disabled (attachment 2). With a short list like this it might look like a minor problem, with longer lists and if you want to turn off 2-3 rules it really becomes a problem, especially as after changing the FIRSt rule the "You will have to press APPLY" message changes the screen layout (scrolling down the rules).

[opnsenseuser]

I think this is a Rebellion-theme Problem. I forwarded this to Martin.

[opnsenseuser]

I forwarded this to Martin, but his Mailbox is Full. :-(

[chemlud]

I forwarded this to Martin, but his Mailbox is Full. :-(

:-D

Just test it with your themes, I found it difficult with different themes ;-)

Isn't there a way to kind of "grey-out" rules that are REALLY inactive?

[opnsenseuser]

I forwarded this to Martin, but his Mailbox is Full. :-(

:-D

Just test it with your themes, I found it difficult with different themes ;-)

Isn't there a way to kind of "grey-out" rules that are REALLY inactive?

can you draw your idea in the picture for me?
The rules which are disabled are in cicada and vicuna yellow. Do you think grey is bettet for a dark theme?

I like to be convinced. every solution that is better is taken.

[Fright]

sorry, if I understood @chemlud correctly, he wants the rows with the rules disabled and written to the config to be somehow different from the rows with the rules that were just disabled but not written to the config yet.
may be its possible via:
tr.rule.text-muted {
    background-color: darkgray;
}
or some
but it seems to me this will only add unnecessary variegation to the table
Tukan is the most eye-friendly theme imho
that's why I asked not to make the left panel so contrast 

[chemlud]

@opnsenseuser: the "(disabled)" is an option.

Would love to see the option @Fright proposed as a draft... ;-)

[opnsenseuser]

@opnsenseuser: the "(disabled)" is an option.

Would love to see the option @Fright proposed as a draft... ;-)

Thats a Good idea. But there has to be a Note at the bottom like the rule Icon Description therefore, so everybody knows what that means. And this description has to be done in the Core source.

[opnsenseuser]

my idea would not have been to mark the entire line but only the words except the first column and the last column because of the icons.
thus an additional note about what the gray line means would also be omitted.

first example is red and second is black (you can make the colors even better visible by choosing another color or a brighter one )

[chemlud]

 strike font in red definitely an option. This is the appearance AFTER disaableing, but BEFORE pressing "Apply", correct? :-)

[opnsenseuser]

strike font in red definitely an option. This is the appearance AFTER disaableing, but BEFORE pressing "Apply", correct? :-)

No, it has to be applied first to see this in the rule section.

Disable the nat rules shows the line before applying. so it is a Core sourcecode code thing

[opnsenseuser]

strike font in red definitely an option. This is the appearance AFTER disaableing, but BEFORE pressing "Apply", correct? :-)

i did a pull request and uploaded a lot of screenshots. i think it looks nice!
https://github.com/opnsense/plugins/pull/2249

[Fright]

another wild idea for toggled-not-applied rules: what if we just start to rotate toggle icon? (of course it will work only until you refresh page by applying changes or manually)
no themes updates needed

[opnsenseuser]

another wild idea for toggled-not-applied rules: what if we just start to rotate toggle icon? (of course it will work only until you refresh page by applying changes or manually)
no themes updates needed

I think your idea is also very good. I've now solved it with crossed out. Let's see how it works when you work with it.

[Fright]

something like
https://github.com/kulikov-a/rules/issues/1