fix sort icon logic so that the correct icon shows asc or desc