First, name of feature is officially "Exception Site List", initial gui is still using old name "Site Exception List"
then Jeff looked at initial implementation and had the following comments:
Security Panel:
1. Name change - "Exception Site List", as you know this caught me by surprise... Also, there is no colon necessary.
2. Could the text here be a bit darker? Also, the items in the list should not be clickable.
3. There should be more space between the "Exception Site List" title and the paragraph below it. At least 4 more pixels. There should be less space above the "Exception Site List" title.
4. There should be more space between the "Manage Exceptions" button and the other buttons below it.
ExceptionListDialog:
1. TItle should be "Exception Site List"
2. Not sure why the Add button is not enabled.
3. Noticed that you added "FILE" in here, so I will add that to the UX spec.
4. There should be more space (at least 4 pixels) below the button and above this message text.
5. When the list is empty, there should be a line of text here that says " Click Add to add an item to this list."