| Index: trunk/extensions/CodeReview/backend/CodeRevision.php |
| — | — | @@ -236,7 +236,7 @@ |
| 237 | 237 | * @return Array |
| 238 | 238 | */ |
| 239 | 239 | public static function getPossibleStates() { |
| 240 | | - return array( 'new', 'fixme', 'improveme', 'reverted', 'resolved', 'ok', 'deferred', 'old' ); |
| | 240 | + return array( 'new', 'fixme', 'reverted', 'resolved', 'ok', 'deferred', 'old' ); |
| 241 | 241 | } |
| 242 | 242 | |
| 243 | 243 | /** |
| Index: trunk/extensions/CodeReview/codereview.sql |
| — | — | @@ -48,7 +48,6 @@ |
| 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 |
| 53 | 52 | -- 'reverted': Was completely reverted by a later revision |
| 54 | 53 | -- 'resolved': Issues with this rev have been since resolved |
| 55 | 54 | -- 'ok': Reviewed, no issues spotted |
| Index: trunk/extensions/CodeReview/CodeReview.i18n.php |
| — | — | @@ -96,8 +96,6 @@ |
| 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' => 'improveme', |
| 101 | | - 'code-status-desc-improveme' => 'A reviewer marked this revision having some minor issues. It should be corrected.', |
| 102 | 100 | 'code-status-reverted' => 'reverted', |
| 103 | 101 | 'code-status-desc-reverted' => 'Revision was undone by a later revision.', |
| 104 | 102 | 'code-status-resolved' => 'resolved', |
| Index: trunk/extensions/CodeReview/modules/ext.codereview.css |
| — | — | @@ -69,21 +69,6 @@ |
| 70 | 70 | color: #ff0 !important; |
| 71 | 71 | } |
| 72 | 72 | |
| 73 | | -.mw-codereview-status-improveme, |
| 74 | | -.mw-codereview-status-improveme td { |
| 75 | | - background: #fc0 !important; |
| 76 | | - color: white; |
| 77 | | -} |
| 78 | | -.mw-codereview-status-improveme:hover, |
| 79 | | -.mw-codereview-status-improveme:hover td { |
| 80 | | - background: #ff3300 !important; |
| 81 | | - color: white; |
| 82 | | -} |
| 83 | | -.mw-codereview-status-improveme:hover a, |
| 84 | | -.mw-codereview-status-improveme:hover td a { |
| 85 | | - color: #ff0 !important; |
| 86 | | -} |
| 87 | | - |
| 88 | 73 | .mw-codereview-status-resolved, |
| 89 | 74 | .mw-codereview-status-resolved td { |
| 90 | 75 | background: #c0ffc0 !important; |