| Edwin Kempin | 4fd84c9 | 2014-04-30 13:23:15 +0200 | [diff] [blame] | 1 | = Review UI |
| 2 | |
| 3 | Reviewing changes is an important task and the Gerrit Web UI provides |
| 4 | many functionalities to make the review process comfortable and |
| 5 | efficient. This is a guide through the review UI that explains the |
| 6 | different functions and UI elements. |
| 7 | |
| 8 | [[change-screen]] |
| 9 | == Change Screen |
| 10 | |
| 11 | The change screen shows the details of a single change and provides |
| 12 | various actions on it. |
| 13 | |
| 14 | image::images/user-review-ui-change-screen.png[width=800, link="images/user-review-ui-change-screen.png"] |
| 15 | |
| 16 | [[commit-message]] |
| 17 | === Commit Message Block |
| 18 | |
| 19 | The focus of the change screen is on the commit message since this is |
| 20 | the most important information about a change. The numeric change ID |
| 21 | and the change status are displayed right above the commit message. |
| 22 | |
| 23 | image::images/user-review-ui-change-screen-commit-message.png[width=800, link="images/user-review-ui-change-screen-commit-message.png"] |
| 24 | |
| 25 | The commit message can be edited directly in the Web UI by clicking on |
| 26 | the `Edit Message` button in the change header. This opens a drop-down |
| 27 | editor box in which the commit message can be edited. Saving |
| 28 | modifications of the commit message automatically creates a new patch |
| 29 | set for the change. The commit message may only be edited on the |
| 30 | current patch set. |
| 31 | |
| 32 | image::images/user-review-ui-change-screen-edit-commit-message.png[width=800, link="images/user-review-ui-change-screen-edit-commit-message.png"] |
| 33 | |
| 34 | The numeric change ID is a link to the change and clicking on it |
| 35 | refreshes the change screen. By copying the link location you can get |
| 36 | the permalink of the change. |
| 37 | |
| 38 | image::images/user-review-ui-change-screen-permalink.png[width=800, link="images/user-review-ui-change-screen-permalink.png"] |
| 39 | |
| 40 | The change status shows the state of the change: |
| 41 | |
| 42 | - `Needs <label>`: |
| 43 | + |
| 44 | The change is in review and an approval on the shown label is still |
| 45 | required to make the change submittable. |
| 46 | |
| 47 | - `Not <label>`: |
| 48 | + |
| 49 | The change is in review and a veto vote on the shown label is |
| 50 | preventing the submit. |
| 51 | |
| 52 | - `Not Current`: |
| 53 | + |
| 54 | The currently viewed patch set is outdated. |
| 55 | + |
| 56 | Please note that some operations, like voting, are not available on |
| 57 | outdated patch sets, but only on the current patch set. |
| 58 | |
| 59 | - `Ready to Submit`: |
| 60 | + |
| 61 | The change has all necessary approvals and may be submitted. |
| 62 | |
| 63 | - `Submitted, Merge Pending`: |
| 64 | + |
| 65 | The change was submitted and was added to the merge queue. |
| 66 | + |
| 67 | The change stays in the merge queue if it depends on a change that is |
| 68 | still in review. In this case it will get automatically merged when all |
| 69 | predecessor changes have been merged. |
| 70 | + |
| 71 | This status can also mean that the change depends on an abandoned |
| 72 | change or on an outdated patch set of another change. In this case you |
| 73 | may want to rebase the change. |
| 74 | |
| 75 | - `Merged`: |
| 76 | + |
| 77 | The change was successfully merged into the destination branch. |
| 78 | |
| 79 | - `Abandoned`: |
| 80 | + |
| 81 | The change was abandoned. |
| 82 | |
| 83 | - `Draft`: |
| 84 | + |
| 85 | The change is a draft that is only visible to the change owner, the |
| 86 | reviewers that were explicitly added to the change, and users who have |
| 87 | the link:access-control.html#category_view_drafts[View Drafts] global |
| 88 | capability assigned. |
| 89 | |
| 90 | [[commit-info]] |
| 91 | === Commit Info Block |
| 92 | |
| 93 | The commit info block shows information about the commit of the |
| 94 | currently viewed patch set. |
| 95 | |
| 96 | It displays the author and the committer as links to a list of this |
| 97 | person's changes that have the same status as the currently viewed |
| 98 | change. |
| 99 | |
| 100 | The commit ID and the link:user-changeid.html[Change-Id] are both |
| 101 | displayed with a copy-to-clipboard icon that allows the ID to be copied |
| 102 | into the clipboard. |
| 103 | |
| 104 | If a Git web browser, such as GitWeb or Gitiles, is configured, there |
| 105 | is also a link to the commit in the Git web browser. |
| 106 | |
| 107 | image::images/user-review-ui-change-screen-commit-info.png[width=800, link="images/user-review-ui-change-screen-commit-info.png"] |
| 108 | |
| Edwin Kempin | f1a84f9 | 2014-04-30 15:37:48 +0200 | [diff] [blame] | 109 | [[change-info]] |
| 110 | === Change Info Block |
| 111 | |
| 112 | The change info block contains detailed information about the change |
| 113 | and offers actions on the change. |
| 114 | |
| 115 | image::images/user-review-ui-change-screen-change-info.png[width=800, link="images/user-review-ui-change-screen-change-info.png"] |
| 116 | |
| 117 | - Change Owner: |
| 118 | + |
| 119 | The owner of the change is displayed as a link to a list of the owner's |
| 120 | changes that have the same status as the currently viewed change. |
| 121 | + |
| 122 | image::images/user-review-ui-change-screen-change-info-owner.png[width=800, link="images/user-review-ui-change-screen-change-info-owner.png"] |
| 123 | |
| 124 | - Reviewers: |
| 125 | + |
| 126 | The reviewers of the change are displayed as chip tokens. |
| 127 | + |
| 128 | For each reviewer there is a tooltip that shows on which labels the |
| 129 | reviewer is allowed to vote. |
| 130 | + |
| 131 | New reviewers can be added by clicking on the `Add...` button. Typing |
| 132 | into the pop-up text field activates auto completion of user and group |
| 133 | names. |
| 134 | + |
| 135 | Reviewers can be removed from the change by clicking on the `x` icon |
| 136 | in the reviewer's chip token. Removing a reviewer also removes the |
| 137 | current votes of the reviewer. The removal of votes is recorded as a |
| 138 | message on the change. |
| 139 | + |
| 140 | Removing reviewers is protected by permissions: |
| 141 | |
| 142 | ** Users can always remove themselves. |
| 143 | ** The change owner may remove any zero or positive score. |
| 144 | ** Users with the link:access-control.html#category_remove_reviewer[ |
| 145 | Remove Reviewer] access right, the branch owner, the project owner |
| 146 | and Gerrit administrators may remove anyone. |
| 147 | |
| 148 | + |
| 149 | image::images/user-review-ui-change-screen-change-info-reviewers.png[width=800, link="images/user-review-ui-change-screen-change-info-reviewers.png"] |
| 150 | |
| 151 | - Project / Branch / Topic: |
| 152 | + |
| 153 | The name of the project for which the change was done is displayed as a |
| 154 | link to the link:user-dashboards.html#project-default-dashboard[default |
| 155 | dashboard] of the project. If no default dashboard is defined, the link |
| 156 | opens a list of open changes on the project. |
| 157 | + |
| 158 | Clicking on the settings icon on the right side navigates to the |
| 159 | project administration screen. |
| 160 | + |
| 161 | The name of the destination branch is displayed as a link to a list |
| 162 | with all changes on this branch that have the same status as the |
| 163 | currently viewed change. |
| 164 | + |
| 165 | If a topic was assigned to the change it is displayed below the branch. |
| 166 | By clicking on the edit icon the topic can be set. This requires the |
| 167 | link:access-control.html#category_edit_topic_name[Edit Topic Name] |
| 168 | access right. To be able to set a topic on a closed change, the |
| 169 | `Edit Topic Name` must be assigned with the `force` flag. |
| 170 | + |
| 171 | image::images/user-review-ui-change-screen-change-info-project-branch-topic.png[width=800, link="images/user-review-ui-change-screen-change-info-project-branch-topic.png"] |
| 172 | |
| 173 | - Submit Strategy: |
| 174 | + |
| 175 | The link:project-setup.html#submit_type[submit strategy] that will be |
| 176 | used to submit the change. The submit strategy is only displayed for |
| 177 | open changes. |
| 178 | + |
| 179 | image::images/user-review-ui-change-screen-change-info-submit-strategy.png[width=800, link="images/user-review-ui-change-screen-change-info-submit-strategy.png"] |
| 180 | + |
| 181 | If a change cannot be merged due to path conflicts this is highlighted |
| 182 | by a bold red `Cannot Merge` label. |
| 183 | + |
| 184 | image::images/user-review-ui-change-screen-change-info-cannot-merge.png[width=800, link="images/user-review-ui-change-screen-change-info-cannot-merge.png"] |
| 185 | |
| 186 | - Time of Last Update: |
| 187 | + |
| 188 | image::images/user-review-ui-change-screen-change-info-last-update.png[width=800, link="images/user-review-ui-change-screen-change-info-last-update.png"] |
| 189 | |
| 190 | - Actions: |
| 191 | + |
| 192 | Depending on the change state and the permissions of the user, different |
| 193 | actions are available on the change: |
| 194 | |
| 195 | ** `Merge Change`: |
| 196 | + |
| 197 | Submits the change and adds it to the merge queue. If possible the |
| 198 | change is merged into the destination branch. |
| 199 | + |
| 200 | The `Merge Change` button is available if the change is submittable and |
| 201 | the link:access-control.html#category_submit[Submit] access right is |
| 202 | assigned. |
| 203 | + |
| 204 | It is also possible to submit changes that have merge conflicts. This |
| 205 | allows to do the conflict resolution for a change series in a single |
| 206 | merge commit and submit the changes in reverse order. |
| 207 | |
| 208 | ** `Abandon`: |
| 209 | + |
| 210 | Abandons the change. |
| 211 | + |
| 212 | The `Abandon` button is only available if the change is open and the |
| 213 | link:access-control.html#category_abandon[Abandon] access right is |
| 214 | assigned. |
| 215 | + |
| 216 | When a change is abandoned, a panel appears that allows one to type a |
| 217 | comment message to explain why the change is being abandoned. |
| 218 | |
| 219 | ** `Restore`: |
| 220 | + |
| 221 | Restores the change. |
| 222 | + |
| 223 | The `Restore` button is only available if the change is abandoned and |
| 224 | the link:access-control.html#category_abandon[Abandon] and the |
| 225 | link:access-control.html#category_push[Push] access right is |
| 226 | assigned. |
| 227 | + |
| 228 | When a change is restored, a panel appears that allows one to type a |
| 229 | comment message to explain why the change is being restored. |
| 230 | |
| 231 | ** `Rebase`: |
| 232 | + |
| 233 | Rebases the change. The rebase is always done with content merge |
| 234 | enabled. If the rebase is successful a new patch set with the rebased |
| 235 | commit is created. If the rebase fails, there are conflicts that have |
| 236 | to be resolved manually. |
| 237 | + |
| 238 | If the change does not depend on another open change, it is rebased |
| 239 | onto the tip of the destination branch. |
| 240 | + |
| 241 | If the change depends on another open change, it is rebased onto the |
| 242 | current patch set of that other change. |
| 243 | + |
| 244 | The `Rebase` button is only available if the change can be rebased and |
| 245 | the link:access-control.html#category_rebase[Rebase] access right is |
| 246 | assigned. Rebasing merge commits is not supported. |
| 247 | |
| 248 | ** `Cherry-Pick`: |
| 249 | + |
| 250 | Allows to cherry-pick the change to another branch. The destination |
| 251 | branch can be selected from a dialog. Cherry-picking a change creates a |
| 252 | new open change on the selected destination branch. |
| 253 | + |
| 254 | It is also possible to cherry-pick a change to the same branch. This is |
| 255 | effectively the same as rebasing it to the current tip of the |
| 256 | destination branch. This can be used to remove dependencies on other |
| 257 | open changes. |
| 258 | + |
| 259 | Users can only cherry-pick changes to branches for which they are |
| 260 | allowed to upload changes for review. |
| 261 | |
| 262 | ** `Publish`: |
| 263 | + |
| 264 | Publishes the currently viewed draft patch set. If this is the first |
| 265 | patch set of a change that is published, the change will be published |
| 266 | as well. |
| 267 | + |
| 268 | The `Publish` button is only available if a draft patch set is viewed |
| 269 | and the user is the change owner or has the |
| 270 | link:access-control.html#category_publish_drafts[Publish Drafts] access |
| 271 | right assigned. |
| 272 | |
| 273 | ** `Delete Change` / `Delete Revision`: |
| 274 | + |
| 275 | Deletes the draft change / the currently viewed draft patch set. |
| 276 | + |
| 277 | The `Delete Change` / `Delete Revision` buttons are only available if a |
| 278 | draft patch set is viewed and the user is the change owner or has the |
| 279 | link:access-control.html#category_delete_drafts[Delete Drafts] access |
| 280 | right assigned. |
| 281 | |
| 282 | ** Further actions may be available if plugins are installed. |
| 283 | |
| 284 | + |
| 285 | image::images/user-review-ui-change-screen-change-info-actions.png[width=800, link="images/user-review-ui-change-screen-change-info-actions.png"] |
| 286 | |
| 287 | - Labels & Votes: |
| 288 | + |
| 289 | Approving votes are colored green; veto votes are colored red. |
| 290 | + |
| 291 | image::images/user-review-ui-change-screen-change-info-labels.png[width=800, link="images/user-review-ui-change-screen-change-info-labels.png"] |
| 292 | |
| Edwin Kempin | ab777f1 | 2014-05-02 11:47:57 +0200 | [diff] [blame^] | 293 | [[files]] |
| 294 | === File List |
| 295 | |
| 296 | The file list shows the files that are modified in the currently viewed |
| 297 | patch set. |
| 298 | |
| 299 | image::images/user-review-ui-change-screen-file-list.png[width=800, link="images/user-review-ui-change-screen-file-list.png"] |
| 300 | |
| 301 | The checkboxes in front of the file names allow files to be marked as reviewed. |
| 302 | |
| 303 | image::images/user-review-ui-change-screen-file-list-mark-as-reviewed.png[width=800, link="images/user-review-ui-change-screen-file-list-mark-as-reviewed.png"] |
| 304 | |
| 305 | The type of a file modification is indicated by the character in front |
| 306 | of the file name: |
| 307 | |
| 308 | - 'no character' (Modified): |
| 309 | + |
| 310 | The file existed before this change and is modified. |
| 311 | |
| 312 | - `A` (Added): |
| 313 | + |
| 314 | The file is newly added. |
| 315 | |
| 316 | - `D` (Deleted): |
| 317 | + |
| 318 | The file is deleted. |
| 319 | |
| 320 | - `R` (Renamed): |
| 321 | + |
| 322 | The file is renamed. |
| 323 | |
| 324 | - `C` (Copied): |
| 325 | + |
| 326 | The file is new and is copied from an existing file. |
| 327 | |
| 328 | image::images/user-review-ui-change-screen-file-list-modification-type.png[width=800, link="images/user-review-ui-change-screen-file-list-modification-type.png"] |
| 329 | |
| 330 | If a file is renamed or copied, the name of the original file is |
| 331 | displayed in gray below the file name. |
| 332 | |
| 333 | image::images/user-review-ui-change-screen-file-list-rename.png[width=800, link="images/user-review-ui-change-screen-file-list-rename.png"] |
| 334 | |
| 335 | Repeating path segments are grayed out. |
| 336 | |
| 337 | image::images/user-review-ui-change-screen-file-list-repeating-paths.png[width=800, link="images/user-review-ui-change-screen-file-list-repeating-paths.png"] |
| 338 | |
| 339 | Inline comments on a file are shown in the `Comments` column. |
| 340 | |
| 341 | Draft comments, i.e. comments that have been written by the current |
| 342 | user but not yet published, are highlighted in red. |
| 343 | |
| 344 | New comments from other users, that were published after the current |
| 345 | user last reviewed this change, are highlighted in bold. |
| 346 | |
| 347 | image::images/user-review-ui-change-screen-file-list-comments.png[width=800, link="images/user-review-ui-change-screen-file-list-comments.png"] |
| 348 | |
| 349 | The size of the modifications in the files can be seen in the `Size` |
| 350 | column. The footer row shows the total size of the change. |
| 351 | |
| 352 | For files, the `Size` column shows the sum of inserted and deleted |
| 353 | lines as one number. For the total size, inserted and deleted lines are |
| 354 | shown separately. In addition, the number of insertions and deletions |
| 355 | is shown as a bar. The size of the bar indicates the amount of changed |
| 356 | lines, and its coloring in green and red shows the proportion of |
| 357 | insertions to deletions. |
| 358 | |
| 359 | The size information is useful to easily spot the files that contain |
| 360 | the most modifications; these files are likely to be the most relevant |
| 361 | files for this change. The total change size gives an estimate of how |
| 362 | long a review of this change may take. |
| 363 | |
| 364 | image::images/user-review-ui-change-screen-file-list-size.png[width=800, link="images/user-review-ui-change-screen-file-list-size.png"] |
| 365 | |
| 366 | In the header of the file list, the `Diff Against` selection can be |
| 367 | changed. This selection allows one to choose if the currently viewed |
| 368 | patch set should be compared against its base or against another patch |
| 369 | set of this change. The file list is updated accordingly. |
| 370 | |
| 371 | The file list header also provides an `Open All` button that opens the |
| 372 | diff views for all files in the file list. |
| 373 | |
| 374 | image::images/user-review-ui-change-screen-file-list-header.png[width=800, link="images/user-review-ui-change-screen-file-list-header.png"] |
| 375 | |
| Edwin Kempin | ae57e18 | 2014-04-30 13:38:55 +0200 | [diff] [blame] | 376 | [[old-change-screen]] |
| 377 | === Old Change Screen |
| 378 | |
| 379 | In addition to the normal change screen, this Gerrit version still |
| 380 | includes the old change screen that was used in earlier Gerrit |
| 381 | versions. Users that want to continue using the old change screen can |
| 382 | configure it in their preferences under |
| 383 | `Settings` > `Preferences` > `Change View`: |
| 384 | |
| 385 | image::images/user-review-ui-change-view-preference.png[width=800, link="images/user-review-ui-change-view-preference.png"] |
| 386 | |
| 387 | [WARNING] |
| 388 | The old change screen will be removed in a later version of Gerrit. |
| 389 | |
| Edwin Kempin | 4fd84c9 | 2014-04-30 13:23:15 +0200 | [diff] [blame] | 390 | GERRIT |
| 391 | ------ |
| 392 | Part of link:index.html[Gerrit Code Review] |
| 393 | |
| 394 | SEARCHBOX |
| 395 | --------- |