| Index: trunk/extensions/CodeReview/backend/CodeRevision.php |
| — | — | @@ -230,7 +230,7 @@ |
| 231 | 231 | * @return Array |
| 232 | 232 | */ |
| 233 | 233 | public static function getPossibleStates() { |
| 234 | | - return array( 'new', 'fixme', 'reverted', 'resolved', 'ok', 'deferred', 'old' ); |
| | 234 | + return array( 'new', 'fixme', 'improveme', 'reverted', 'resolved', 'ok', 'deferred', 'old' ); |
| 235 | 235 | } |
| 236 | 236 | |
| 237 | 237 | /** |
| Index: trunk/extensions/CodeReview/codereview.sql |
| — | — | @@ -48,6 +48,7 @@ |
| 49 | 49 | -- Status key for how this thang is... |
| 50 | 50 | -- 'new': Hasn't yet been reviewed |
| 51 | 51 | -- 'fixme': This revision has some problem which needs to be resolved |
| | 52 | + -- 'improveme': This revision has no major errors, but needs further work |
| 52 | 53 | -- 'reverted': Was completely reverted by a later revision |
| 53 | 54 | -- 'resolved': Issues with this rev have been since resolved |
| 54 | 55 | -- 'ok': Reviewed, no issues spotted |
| Index: trunk/extensions/CodeReview/CodeReview.i18n.php |
| — | — | @@ -96,6 +96,8 @@ |
| 97 | 97 | 'code-status-desc-new' => 'Revision is pending an action (default status).', |
| 98 | 98 | 'code-status-fixme' => 'fixme', |
| 99 | 99 | 'code-status-desc-fixme' => 'A reviewer marked this revision introduce a bug or is broken. It should be corrected.', |
| | 100 | + 'code-status-improveme' => 'fixme', |
| | 101 | + 'code-status-desc-improveme' => 'A reviewer marked this revision having some minor issues. It should be corrected.', |
| 100 | 102 | 'code-status-reverted' => 'reverted', |
| 101 | 103 | 'code-status-desc-reverted' => 'Revision was undone by a later revision.', |
| 102 | 104 | 'code-status-resolved' => 'resolved', |
| Index: trunk/extensions/CodeReview/modules/ext.codereview.css |
| — | — | @@ -69,6 +69,11 @@ |
| 70 | 70 | color: #ff0 !important; |
| 71 | 71 | } |
| 72 | 72 | |
| | 73 | +.mw-codereview-status-improveme:hover a, |
| | 74 | +.mw-codereview-status-improveme:hover td a { |
| | 75 | + background: #ff9900 !important; |
| | 76 | +} |
| | 77 | + |
| 73 | 78 | .mw-codereview-status-resolved, |
| 74 | 79 | .mw-codereview-status-resolved td { |
| 75 | 80 | background: #c0ffc0 !important; |