zoneminder patch to options screen

[bovine]

Here's a patch I made that improves a couple of minor usability concerns in the options screens of the default skin (remove the spaces):

http :// gist . github . com / 2271967

display configuration help in a tooltip or alert box rather than a new window. It seems rather wasteful to create yet another browser window just for a very small amount of text, especially since it's easy enough for that window to lose focus and lost behind so many other windows already opened. So now you can hover over the "?" to see the tip, or if you click on it the text will open in a modal browser alert box.

make configuration radio buttons easier to click by using labels. This lets you click on the words associated with the radio button (like "hashed", "plain", or "none"), and can also be considered an accessibility/screen-reader improvement.

[nightcrawler]

Hey thats nice,

can you tell me how I install this web-skins-classic-views-options.php.DIFF file?
Just rename&replace the old file to .php (remove DIFF) isn't right or does it?

[bovine]

Hey thats nice,

can you tell me how I install this web-skins-classic-views-options.php.DIFF file?
Just rename&replace the old file to .php (remove DIFF) isn't right or does it?

It is just a standard "diff" file, so you need to use the "patch" utility to apply it. For example, save the web-skins-classic-views-options.php.diff file to disk and run "patch -p1 < web-skins-classic-views-options.php.diff" from the /usr/share/zoneminder/www/ directory or wherever your zoneminder web files were installed.

Or since it's a relatively short patch, you can look at the lines being changed (red lines are the old lines, green lines are new ones) and manually edit the options.php file with your favorite editor.

[nightcrawler]

tnx bovine,

the patch works great (had to mod the patch a bit because the /web/ directory isn't the directory for my HTML files but that's no problem).
Nice pop-up boxes when I press the ? mark.