chore: minor changes

AKharytonchyk · Mar 28, 2024 · 2dcaf85 · 2dcaf85
1 parent 7fed759
commit 2dcaf85
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 8 deletions.
diff --git a/README.md b/README.md
@@ -23,6 +23,10 @@ The GitHub PR Dashboard is a comprehensive tool designed to streamline the monit
 
 ![Settings](docs/images/settings.jpeg)
 
+### Failed Checks
+
+![Failed Checks](docs/images/failed-checks.jpeg)
+
 ## Getting Started
 
 Follow these instructions to get your GitHub PR Dashboard up and running on your local machine for development and testing purposes.

diff --git a/docs/images/dashboard.jpeg b/docs/images/dashboard.jpeg
diff --git a/docs/images/failed-checks.jpeg b/docs/images/failed-checks.jpeg
diff --git a/src/components/PullRequestCard.tsx b/src/components/PullRequestCard.tsx
@@ -58,7 +58,7 @@ const PullRequestCard: React.FC<PullRequestCardProps> = ({ pr }) => {
           sx={{ marginRight: "auto" }}
         />
         {pr.locked && <Lock /> }
-        {pr.draft && <DesignServices color="secondary"/>}
+        {pr.draft && <Tooltip title="Draft PR"><DesignServices color="secondary"/></Tooltip>}
         {pr.labels.map((label) => (<Chip key={label.id} label={label.name} size="small" color={getLabelColor(label.name)}/> ))}
         <Chip
           label={pr.state.toUpperCase()}

diff --git a/src/components/PullRequestChecks.tsx b/src/components/PullRequestChecks.tsx
@@ -1,7 +1,7 @@
 import React from "react";
 import { ConfigContext } from "../App";
 import { CheckRun } from "../models/CheckRun";
-import { Box, Dialog, Link, Typography } from "@mui/material";
+import { Box, Dialog, Link, Tooltip, Typography } from "@mui/material";
 import { CheckCircle, Error, ErrorOutline } from "@mui/icons-material";
 
 export type PullRequestChecksProps = {
@@ -44,9 +44,9 @@ export const PullRequestChecks: React.FC<PullRequestChecksProps> = ({
       >
         Checks:{" "}
         {allChecksPassed ? (
-          <CheckCircle color="success" sx={{display: "flex", alignItems: "center"}} />
+          <Tooltip title="All checks passed"><CheckCircle color="success" sx={{display: "flex", alignItems: "center"}} /></Tooltip>
         ) : (
-          <Error color="error" />
+          <Tooltip title="Something went wrong. Click for details"><Error color="error" /></Tooltip>
         )}
       </Typography>
       <Dialog open={open} onClose={() => setOpen(false)} sx={{ padding: "2em" }}>

diff --git a/src/components/PullRequestsApprovals.tsx b/src/components/PullRequestsApprovals.tsx
@@ -1,7 +1,7 @@
 import React from "react";
 import { ConfigContext } from "../App";
 import { Approvals } from "../models/Approvals";
-import { Avatar, Badge, Box } from '@mui/material';
+import { Avatar, Badge, Box, Tooltip } from '@mui/material';
 
 export type PullRequestsApprovalsProps = {
   owner: string;
@@ -45,9 +45,11 @@ export const PullRequestsApprovals: React.FC<PullRequestsApprovalsProps> = ({
   const allApprovals = React.useMemo(() => approvals?.filter((approval) => approval.state !== "DISMISSED"), [approvals]);
 
   const approvalAvatars = React.useMemo(() => allApprovals?.map((approval) => (
-    <Badge key={approval.user.login} {...getBadgeProps(approval.state)} sx={{ height: "1em", display: "flex", alignItems: "center" }}>
-      <Avatar key={approval.user.login} alt={approval.user.login} src={approval.user.avatar_url} sx={{height: "1.5em", width: "1.5em"}} />
-    </Badge>
+    <Tooltip key={approval.user.login} title={approval.state}>
+      <Badge {...getBadgeProps(approval.state)} sx={{ height: "1em", display: "flex", alignItems: "center" }}>
+        <Avatar alt={approval.user.login} src={approval.user.avatar_url} sx={{height: "1.5em", width: "1.5em"}} />
+      </Badge>
+    </Tooltip>
     )), [allApprovals]);
 
   return <>