I don't think it is correct to BR hicolor-icon-theme. It is correct to require it. A longer discussion on the topic is here: https://pagure.io/packaging-committee/issue/632