Created attachment 809171 [details] ui mockup show reminder of pending review, feedback, and needinfo requests in the header. to avoid performing extra queries on each page load, we'll have to add three columns to the profiles table, and update them when flags are set/updated/cleared.
Created attachment 819551 [details] [diff] [review] 920026_1.patch
Attachment #819551 - Flags: review?(dkl)
Comment on attachment 819551 [details] [diff] [review] 920026_1.patch Review of attachment 819551 [details] [diff] [review]: ----------------------------------------------------------------- Very nice as always. Works well in testing and code looks sane except for minor cosmetics. r=dkl ::: extensions/Review/lib/Util.pm @@ +25,5 @@ > + COUNT(*) as count > + FROM flags > + INNER JOIN profiles ON profiles.userid = flags.requestee_id > + INNER JOIN flagtypes ON flagtypes.id = flags.type_id > + WHERE status = '?' nit: s/flags/flags.status/ ::: extensions/Review/template/en/default/hook/global/header-message.html.tmpl @@ +12,5 @@ > + || user.needinfo_request_count > +%] > + > +<span id="badge" title="Flags requested of you: > + [%- " review(" _ user.review_request_count _ ")" IF user.review_request_count -%] nit: space between review and (. Same with the others. ::: extensions/Review/web/styles/badge.css @@ +10,5 @@ > + font-size: small; > + font-weight: bold; > + padding: 0px 5px; > + border-radius: 10px; > + margin: 0 2px; Nit: Increase to maybe 5px so it looks better for non-Mozilla skins. Otherwise it is almost touching the title for me. YMMV.
Attachment #819551 - Flags: review?(dkl) → review+
thanks dkl :) Committing to: bzr+ssh://email@example.com/bmo/4.2/ modified Bugzilla/Flag.pm modified extensions/Review/Extension.pm added extensions/Review/bin added extensions/Review/bin/review_requests_rebuild.pl added extensions/Review/lib/Util.pm added extensions/Review/template/en/default/hook/global/header-message.html.tmpl modified extensions/Review/template/en/default/hook/global/header-start.html.tmpl added extensions/Review/template/en/default/hook/global/user-error-auth_failure_object.html.tmpl added extensions/Review/template/en/default/pages/review_requests_rebuild.html.tmpl added extensions/Review/web/styles/badge.css modified template/en/default/global/header.html.tmpl Committed revision 9115.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.