@PiBa:
I'm not taking off the apply button. I'm making that actually do something (write new config and restart service).
As for the save button on the main page (besides the 'add' button), there is nothing there that really needs saving. Like changing the order of the rows after dragging them to a different position. (Which isn't implemented there.) And it doesn't seem to do anything anyhow.?
Yeh, that's the correct thing to do. With the current code, the "Save" button actually does an "Apply" action, and the "Apply" button does nothing. After fixing that (which PR https://github.com/pfsense/pfsense/pull/3606 does), then the "Save" button has no other use, so deleting it is good.
It works in the same workflow as Firewall Aliases, Firewall Rules etc - changes are saved in the config when pressing "Save" in the edit screen. The user is then shown the "summary" screen listing all the entries and presses "Apply" to implement the changes on the running system.