Standardise css list padding

On Status->Services, the Services Status widget when a service status is down, the red "x" button does not line up with the green "running" button in the column.
This is because listbg has:
padding-right: 16px;
But the normal listr has:
padding-right: 6px;
It seems really odd that listbg as 16 pixels right padding. I expected it to have all the same padding as listr, just with a different color.
listn and listbggrey have the same "feature" of extra padding-right.
listn does not seem to be used anywhere anyway - might as well "fix" it in case something uses it some time.
listbggrey seems to only be used by pkg_mgr_installed.php in the case when an installed package version is in advance of the online version - quite an unusual thing! Changing padding-right from 16px back to 6px is fine for that.
Then the "feature" is lso in listhdr and listhdrr - lots of examples like firewall_aliases.php have the column headings as listhdr or listhdrr, so in theory if the column is not so wide already, it will put 16px on the right after the column heading text. But the actual rows of data underneath use listr and listbg, which only specify 6px right padding. In the end it usually does not make any difference - there is space already and the browser lines it all up anyway. But it seems to me an odd difference in the css.
listtopic also has this 16px right padding thing - but it seems to be used just for text display that is not in columns, so I did not mess with that.
Changing this css fixes the lineup of the red "x" in the services status column and does not seem to break anything else. In theory it could make other things better also.
Someone who knows how the css was intended to be used could comment on this! If I have got the right idea, and these "16px" things were just a mistake from the past, then it should be checked in the css for all themes and corrected appropriately... 
I am happy to do that, once someone confirms it is the thing to do.
Typo CIDR in UPnP GUI page

As reported by johnpoz https://forum.pfsense.org/index.php?topic=84964.0
Merge pull request #1369 from phil-davis/patch-4
Merge pull request #1368 from phil-davis/patch-3
Standardise css list padding

for _corporate
Standardise css list padding

for code-red theme
Standardise css list padding

for metallic theme
Standardise css list padding

for nervecenter theme
Standardise css list padding

for pfsense-dropdown theme
Standardise css list padding

for pfsense theme
Standardise css list padding

for pfsense_ng_fs theme
Standardise css list padding

for the_wall theme
Provide success return indication from console_configure_dhcpd

Recent commit https://github.com/pfsense/pfsense/commit/9ea554ee5cb25ea3bf5bb6bf7997c6c7379ce349 added testing of the return status of console_configure_dhcpd() - this let a user effectively abort from doing anything if they have answered "y" to prompt_for_enable_dhcp_server() and are being asked for the start and end of the range, and then decide they do not want to proceed.
However, even when they gave good answers, status 0 was being returned. This prevented changes ever being implemented.
Redmine: https://redmine.pfsense.org/issues/4080
The fix is to return 1 at the routine end, when all is good and the code should proceed.
Merge pull request #1370 from phil-davis/patch-5
Merge pull request #1371 from phil-davis/patch-6
Merge remote-tracking branch 'upstream/master'