Improvement to table pages
-
would it be ok to create an "icon-embed-btn' class in pfSense.css, to get rid of the inline style ?
-
Looking good.
If I open the info text (click the "i" icon) the text is displayed - good.
Then there is an "x" in the top right corner of the info text box. If I click that the info text box closes - seems reasonable :)
But then clicking the "i" icon again does not open the info text. I guess the info text box is "gone for good". (until a page refresh)
-
Unfortunante 'X' has been suppressed in jQuery
-
All of the files I identified have been updated and pushed. (I got carried away).
These files look like they still need some fonts-awesome love and need it more than most :) :
system_camanager.php
system_certmanager.phpOr perhaps I missed a PR?
Centralizing the embedded icon style sounds like a very good idea. icon-embed-button would be good, or perhaps just icon-button.
It should be possible to use sed (or similar) to replace all occurrences of '" style="margin-top: 10px;"' with ' icon-embed-button"' I would think.
-
Vpn menu fa convcert = https://github.com/pfsense/pfsense/pull/2017
-
Thank you :)
-
It should be possible to use sed (or similar) to replace all occurrences of '" style="margin-top: 10px;"' with ' icon-embed-button"' I would think.
https://github.com/pfsense/pfsense/pull/2027
sed= easy / git=hard :p
-
Yes!
-
If you see a good idea, copy it ;)
Inspired by Heper's changes above I have done something similar for the confirm on delete pop-ups.
Now all instances of "fa-trash" will automatically get a "Are you sure you wish to . . " confirmation dialog added, so all of the "onclick="return confirm("Do you . . " stuff has been removed from the HTML.
There are one or two places where this behavior is not desired (e.g.: vpn_ipsec.php). Adding the "no-confirm" class to the icon will suppress the pop-up.
I also moved the styling used in
<nav class="action-buttons">to the css file.
Thanks again for everyone's help.</nav>
-
Improvement to infoblock mechanism
The infoblock mechanism now works slightly differently. Here is an example:
Notice the 'false' at the end of the print_info_box? That is a new argument that suppresses the 'X' close button that is normally present in a text box.
is exactly the same, except that the information block is initially open. It is now possible to have more than one infoblock (or infoblock_open) on a page. To do that, the classes _must begin with_ infoblock or infoblock_open but must also have a unique suffix. For example: