Refine commit popover experience
<!--IssueSummary start--> <details> <summary> Everyone can contribute. [Help move this issue forward](https://handbook.gitlab.com/handbook/marketing/developer-relations/contributor-success/community-contributors-workflows/#contributor-links) while earning points, leveling up and collecting rewards. </summary> - [Close this issue](https://contributors.gitlab.com/manage-issue?action=close&projectId=278964&issueIid=589643) </details> <!--IssueSummary end--> # Context https://gitlab.com/gitlab-org/gitlab/-/work_items/561788 implemented a commit popover to give users at-a-glance information for commits without forcing users to leave the context of the page they're on. <details> <summary> ##### Reference: Popovers in Source Code </summary> From https://gitlab.com/gitlab-org/gitlab/-/issues/448868#note_2117391060 | Popover Element | Screenshot | |-----------------|------------| | [MR](https://gitlab.com/gitlab-com/www-gitlab-com/-/merge_requests/136006) | ![popover-mr.png](/uploads/6ded8552762f874143264f40f46cd081/popover-mr.png){width="288" height="151"} | | [Issue](https://gitlab.com/gitlab-org/gitlab/-/issues/448868) | ![popover-issue.png](/uploads/2fea0c20f09acc8584881c29777ec3d9/popover-issue.png){width="282" height="151"} | | [Epic](https://gitlab.com/groups/gitlab-org/-/epics/8075) | ![popover-epic.png](/uploads/54416fda1f0060b6f6291909dfabfe9b/popover-epic.png){width="219" height="132"} | | [Open MR popover](https://gitlab.com/gitlab-org/gitlab/-/blob/master/.gitlab/CODEOWNERS) | ![openmr.png](/uploads/604c42b5f9d1455f9ea0de826e26b287/openmr.png){width="410" height="326"} | | [Username](https://gitlab.com/mcbabin) | ![popover-user.png](/uploads/167d996921e97bb6f272fddd8440a35a/popover-user.png){width="299" height="179"} | </details> # Design proposal 1. Refine commit popover details * Increase the size of the Commit title from 12px to 14px (h5) for consistency with the [MR](https://gitlab.com/gitlab-com/www-gitlab-com/-/merge_requests/136006) / [Issue](https://gitlab.com/gitlab-org/gitlab/-/issues/448868) / [Epic](https://gitlab.com/groups/gitlab-org/-/epics/8075) popovers. * The username and commit metadata appear on different lines. Elements should be inline with each other and wrap for consistency with other popovers. <table> <tr> <th>Current</th> <th>Proposed</th> </tr> <tr> <td> ![popover.png](/uploads/e1fbff43e1ce251a9cd82cc7c8254317/popover.png){width="228" height="138"} Commit title is 12px </td> <td> ![commit popover.png](/uploads/b896205f15a148d01c77b2a85b74a0d5/commit_popover.png){width="296" height="120"} Commit title should be an h5 (14px) </td> </tr> </table> 2. Add this popover to other areas in Source Code / GitLab where the commit link appears: <table> <tr> <th>Product area</th> <th>Location</th> <th>Screen</th> </tr> <tr> <td>Source Code</td> <td> Code \> Branches \> User hovers over commit link </td> <td> ![branches.png](/uploads/01af4d180ebdc529e799e290585e2d62/branches.png){width="900" height="515"} </td> </tr> <tr> <td>Source Code</td> <td> Code \> Tags \> User hovers over commit link </td> <td> ![tags.png](/uploads/5bacc640f007e1967d4e802fe5dadf17/tags.png){width="900" height="461"} </td> </tr> <tr> <td>Code Review</td> <td> Merge request \> User links to a commit in an MR description / comment </td> <td> ![mr-comment.png](/uploads/8886160d5388ff4dcf116c1570222f8c/mr-comment.png){width=900 height=361} </td> </tr> <tr> <td>Plan</td> <td> Issues \> User links to a commit in an Issue description / comment </td> <td> ![issues.png](/uploads/2a8e7619ef2f95106a66e64b55395956/issues.png){width="652" height="107"} </td> </tr> <tr> <td>CI/CD</td> <td> Pipeline \> User hovers over a commit link * Show commit popover Pipeline \> User hovers over an MR * Show MR popover </td> <td> ![pipeline.png](/uploads/d247d9fcc9b59ade115173313a45603a/pipeline.png){width="900" height="574"} </td> </tr> <tr> <td>CI/CD</td> <td> Jobs \> User hovers over a commit link </td> <td> ![jobs.png](/uploads/11ed453b1f7617ca9587f8502370335d/jobs.png){width="755" height="600"} </td> </tr> <tr> <td>CI/CD</td> <td> Artifacts \> User hovers over a commit link </td> <td> ![artifacts.png](/uploads/d010b7ba5d599083d9738057ef28f737/artifacts.png){width="748" height="600"} </td> </tr> </table>
issue