blob: d2988439f46ff4df0638ece308075b365eec0b20 [file] [log] [blame] [view]
---
title: "Gerrit Project News #7: Gerrit Frontend Fixit Week 2020"
tags: news
keywords: news
permalink: 2020-02-18-gerrit-news-fe-fixit-week-2020.html
summary: "Gerrit project news from Frontend Fixit Week."
hide_sidebar: true
hide_navtoggle: true
toc: true
---
## Fixit Week Summary
In the first week of February 2020 the frontend team had a "Fixit Week".
We focused on fixing all the little things that pile up and otherwise don't get attention.
We were able to fix around 36 issues. Big thanks to all contributers who joined us.
## Fixit Highlights
### Hovercard for blame annotation
[Change](https://gerrit-review.googlesource.com/c/gerrit/+/253121)
introduces a new hovercard with commit message when hovering over the blame annotation.
![Blame Hover Card](/images/fixit-week-2020-blame-hovercard.png)
### Improving layout of label scores
[Change](https://gerrit-review.googlesource.com/c/gerrit/+/253165)
improves layout to the table so the buttons are not so far away from the label name.
Also add a background-color on hover to highlight which label you are about to vote on.
![Label Scores](/images/fixit-week-2020-label-scores.png)
### Warn user in submit dialog about unresolved comments
[Change](https://gerrit-review.googlesource.com/c/gerrit/+/253130)
introduces a new warning when submitting a change with unresolved comments.
It doesn't block submit, just gives the last notification about unresolved comments.
![Warn about unresolved comments](/images/fixit-week-2020-unresolved.png)
### Honor date format from preferences when displaying dates
[Change](https://gerrit-review.googlesource.com/c/gerrit/+/253116)
fixes some places where we didn't honor the date format from the user's preferences.
For example if user sets date format `3.Jun` instead of default `Jun 3` dashboard
will be changed as in the screenshot below.
![Honor date format](/images/fixit-week-2020-dates.png)
## All Fixit Improvements
* [Process links if leading whitespace are missed](https://gerrit-review.googlesource.com/c/gerrit/+/253169)
* [Show files with comments in diff-view file list](https://gerrit-review.googlesource.com/c/gerrit/+/253221)
* [Improve error dialog for 404 errors](https://gerrit-review.googlesource.com/c/gerrit/+/253225)
* [Add patchset navigator for findings tab](https://gerrit-review.googlesource.com/c/gerrit/+/252762)
* [Change button in gr-reply-dialog: Rename Save to Send](https://gerrit-review.googlesource.com/c/gerrit/+/253220)
* [Small redesign of diff expansion row](https://gerrit-review.googlesource.com/c/gerrit/+/252614)
* [Replace circled i from Unicode with proper icon](https://gerrit-review.googlesource.com/c/gerrit/+/252944)
* [Add title and shortcuts for some links and buttons](https://gerrit-review.googlesource.com/c/gerrit/+/253544)
* [Issue linkification should not work across two line breaks](https://issues.gerritcodereview.com/issues/40011890)
* [Add tracking (metrics) on keyboard shortcut usage](https://gerrit-review.googlesource.com/c/gerrit/+/253653)
* [Show X-Gerrit-Trace in error dialog if exists](https://gerrit-review.googlesource.com/c/gerrit/+/253541)
* [Link in Blame annotation to commit should go directly to commit](https://gerrit-review.googlesource.com/c/gerrit/+/253127)
* [Allow delete change message from the UI](https://gerrit-review.googlesource.com/c/gerrit/+/253412)
* [Add a proper message for no threads in findings tab](https://gerrit-review.googlesource.com/c/gerrit/+/253224)
* [Support ctrl+enter for move change dialog](https://gerrit-review.googlesource.com/c/gerrit/+/253414)