From ddcfc546fbf89cdac562e6044558b513695b72c9 Mon Sep 17 00:00:00 2001
From: Deluan <deluan@deluan.com>
Date: Sun, 31 May 2020 13:57:17 -0400
Subject: [PATCH] Link is not on the album cover, leaving a gap between albums.

Other small improvements
---
 ui/src/album/AlbumGridView.js | 43 ++++++++++++++++-------------------
 ui/src/album/styles.js        |  2 +-
 ui/src/common/SongDatagrid.js |  8 +++----
 3 files changed, 25 insertions(+), 28 deletions(-)

diff --git a/ui/src/album/AlbumGridView.js b/ui/src/album/AlbumGridView.js
index 8f6425e06..874282293 100644
--- a/ui/src/album/AlbumGridView.js
+++ b/ui/src/album/AlbumGridView.js
@@ -71,7 +71,7 @@ const Cover = withContentRect('bounds')(
   }
 )
 
-const AlbumGridTile = ({ record }) => {
+const AlbumGridTile = ({ record, basePath }) => {
   const isDesktop = useMediaQuery((theme) => theme.breakpoints.up('md'))
   const classes = useStyles()
   const [visible, setVisible] = useState(false)
@@ -85,21 +85,23 @@ const AlbumGridTile = ({ record }) => {
         setVisible(false)
       }}
     >
-      <Cover album={record} />
-      {(!isDesktop || visible) && (
-        <GridListTileBar
-          className={classes.tileBar}
-          title={record.name}
-          subtitle={
-            <div className={classes.albumArtistName}>
-              <ArtistLinkField record={record} className={classes.artistLink}>
-                {record.albumArtist}
-              </ArtistLinkField>
-            </div>
-          }
-          actionIcon={<AlbumContextMenu record={record} color={'white'} />}
-        />
-      )}
+      <Link to={linkToRecord(basePath, record.id, 'show')}>
+        <Cover album={record} />
+        {(!isDesktop || visible) && (
+          <GridListTileBar
+            className={classes.tileBar}
+            title={record.name}
+            subtitle={
+              <div className={classes.albumArtistName}>
+                <ArtistLinkField record={record} className={classes.artistLink}>
+                  {record.albumArtist}
+                </ArtistLinkField>
+              </div>
+            }
+            actionIcon={<AlbumContextMenu record={record} color={'white'} />}
+          />
+        )}
+      </Link>
     </div>
   )
 }
@@ -116,13 +118,8 @@ const LoadedAlbumGrid = ({ ids, data, basePath, width }) => {
         spacing={20}
       >
         {ids.map((id) => (
-          <GridListTile
-            className={classes.gridListTile}
-            component={Link}
-            key={id}
-            to={linkToRecord(basePath, id, 'show')}
-          >
-            <AlbumGridTile record={data[id]} />
+          <GridListTile className={classes.gridListTile} key={id}>
+            <AlbumGridTile record={data[id]} basePath={basePath} />
           </GridListTile>
         ))}
       </GridList>
diff --git a/ui/src/album/styles.js b/ui/src/album/styles.js
index 0881fdabe..7d12f9d8c 100644
--- a/ui/src/album/styles.js
+++ b/ui/src/album/styles.js
@@ -36,7 +36,7 @@ export const useStyles = makeStyles((theme) => ({
       width: '26em',
     },
     [theme.breakpoints.up('lg')]: {
-      width: '38em',
+      width: '43em',
     },
   },
   albumTitle: {
diff --git a/ui/src/common/SongDatagrid.js b/ui/src/common/SongDatagrid.js
index 797b32503..3206f63eb 100644
--- a/ui/src/common/SongDatagrid.js
+++ b/ui/src/common/SongDatagrid.js
@@ -26,14 +26,14 @@ const useStyles = makeStyles({
 
 const DiscSubtitleRow = ({
   record,
-  onClickDiscSubtitle,
+  onClick,
   colSpan,
   contextAlwaysVisible,
 }) => {
   const classes = useStyles()
   const [visible, setVisible] = useState(false)
   const handlePlayDisc = (discNumber) => () => {
-    onClickDiscSubtitle(discNumber)
+    onClick(discNumber)
   }
   return (
     <TableRow
@@ -79,14 +79,14 @@ export const SongDatagridRow = ({
       {multiDisc && record.trackNumber === 1 && (
         <DiscSubtitleRow
           record={record}
-          onClickDiscSubtitle={onClickDiscSubtitle}
+          onClick={onClickDiscSubtitle}
           contextAlwaysVisible={contextAlwaysVisible}
           colSpan={childCount + (rest.expand ? 1 : 0)}
         />
       )}
       <DatagridRow
         record={record}
-        onMouseEnter={() => setVisible(true)}
+        onMouseMove={() => setVisible(true)}
         onMouseLeave={() => setVisible(false)}
         {...rest}
       >