On /Admin/ManageUsers.aspx it would be nice to exclude the roles user is already in from the dropdown list on the bottom. If the user is already in all the roles, just hide the dropdown and the button and show some label like "The user is already in all the roles for this site".