[Webkit-unassigned] [Bug 105479] New: [Bugzilla] It’s confusing to show r+ and r- as review flag values on new attachment page

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 19 16:46:12 PST 2012


https://bugs.webkit.org/show_bug.cgi?id=105479

           Summary: [Bugzilla] It’s confusing to show r+ and r- as review
                    flag values on new attachment page
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebKit Website
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: rniwa at webkit.org
                CC: eric at webkit.org, wsiegrist at apple.com,
                    jparent at google.com, lforschler at apple.com


It seems misleading that we show r- and r+ as options on new attachment page.

I’ve seen multiple cases where a new contributor mistakenly set r+ or r- flags on their attachments as done in:
https://bugs.webkit.org/show_bug.cgi?id=105463

Since the person attaching a patch shouldn’t be reviewing a patch (unless he/she is attaching a patch he/she has already reviewed but doesn’t that ever happen?), we can probably just remove those flags from new attachment page.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the webkit-unassigned mailing list