Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=784359
--- Comment #13 from Volker Fröhlich volker27@gmx.at 2012-02-23 18:06:50 EST --- Please use the version macro in Source0. The group should rather be Applications/System, I guess.
http://fedoraproject.org/wiki/RPMGroups
Following this discussion, http://lists.fedoraproject.org/pipermail/devel/2012-January/161102.html, hicolor-icon-theme should be Required.
One last thing I stumbled upon, is fedora-review complaining about the desktop file having a different name than the package. I haven't found that in the guidelines explicitly.
As you mentioned, the missing manpage is not a blocker.