Fixed: Links and already added icons overflowing on add artist/album search results

Co-Authored-By: Mark McDowall <markus101@users.noreply.github.com>
Signed-off-by: Robin Dadswell <robin@dadswell.email>

frontend/src/Search/Author/AddNewAuthorSearchResult.css

@@ -34,10 +34,20 @@
 
 .content {
   flex: 0 1 100%;
+  overflow: hidden;
 }
 
-.name {
+.nameRow {
+  display: flex;
+}
+
+.nameContainer {
   display: flex;
+  align-items: flex-end;
+  flex: 0 1 auto;
+}
+
+.name {
   font-weight: 300;
   font-size: 36px;
 }
@@ -47,6 +57,14 @@
   color: $disabledColor;
 }
 
+.icons {
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  flex: 1 0 auto;
+  height: 55px;
+}
+
 .mbLink {
   composes: link from '~Components/Link/Link.css';
 
@@ -69,3 +87,10 @@
   margin-top: 20px;
   text-align: justify;
 }
+
+@media only screen and (max-width: $breakpointMedium) {
+  .titleRow {
+    justify-content: space-between;
+    overflow: hidden;
+  }
+}

frontend/src/Search/Author/AddNewAuthorSearchResult.js

@@ -113,22 +113,26 @@ class AddNewAuthorSearchResult extends Component {
           }
 
           <div className={styles.content}>
+            <div className={styles.nameRow}>
+              <div className={styles.nameContainer}>
                 <div className={styles.name}>
                   {authorName}
 
                   {
-                !name.contains(year) && year ?
+                    !authorName.contains(year) && year ?
                       <span className={styles.year}>
                         ({year})
                       </span> :
                       null
                   }
-
                   {
                     !!disambiguation &&
                       <span className={styles.year}>({disambiguation})</span>
                   }
+                </div>
+              </div>
 
+              <div className={styles.icons}>
                 {
                   isExistingAuthor ?
                     <Icon
@@ -152,6 +156,7 @@ class AddNewAuthorSearchResult extends Component {
                   />
                 </Link>
               </div>
+            </div>
 
             <div>
               {

frontend/src/Search/Book/AddNewBookSearchResult.css

@@ -34,17 +34,22 @@
 
 .content {
   flex: 0 1 100%;
+  overflow: hidden;
 }
 
-.name {
+.titleRow {
   display: flex;
-  font-weight: 300;
-  font-size: 36px;
 }
 
-.authorName {
+.titleContainer {
+  display: flex;
+  align-items: flex-end;
+  flex: 0 1 auto;
+}
+
+.title {
   font-weight: 300;
-  font-size: 20px;
+  font-size: 36px;
 }
 
 .year {
@@ -52,6 +57,14 @@
   color: $disabledColor;
 }
 
+.icons {
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  flex: 1 0 auto;
+  height: 55px;
+}
+
 .mbLink {
   composes: link from '~Components/Link/Link.css';
 
@@ -74,3 +87,10 @@
   margin-top: 20px;
   text-align: justify;
 }
+
+@media only screen and (max-width: $breakpointMedium) {
+  .titleRow {
+    justify-content: space-between;
+    overflow: hidden;
+  }
+}

frontend/src/Search/Book/AddNewBookSearchResult.js

@@ -112,21 +112,26 @@ class AddNewBookSearchResult extends Component {
           }
 
           <div className={styles.content}>
-            <div className={styles.name}>
+            <div className={styles.titleRow}>
+              <div className={styles.titleContainer}>
+                <div className={styles.title}>
                   {title}
 
                   {
                     !!disambiguation &&
                       <span className={styles.year}>({disambiguation})</span>
                   }
+                </div>
+              </div>
 
+              <div className={styles.icons}>
                 {
                   isExistingBook ?
                     <Icon
                       className={styles.alreadyExistsIcon}
                       name={icons.CHECK_CIRCLE}
-                    size={20}
+                      size={36}
-                    title="Book already in your library"
+                      title="Already in your library"
                     /> :
                     null
                 }
@@ -134,7 +139,7 @@ class AddNewBookSearchResult extends Component {
                 <Link
                   className={styles.mbLink}
                   to={`https://goodreads.com/book/show/${editions[0].foreignEditionId}`}
-                onPress={this.onMBLinkPress}
+                  onPress={this.onTVDBLinkPress}
                 >
                   <Icon
                     className={styles.mbLinkIcon}
@@ -142,22 +147,7 @@ class AddNewBookSearchResult extends Component {
                     size={28}
                   />
                 </Link>
-
               </div>
-
-            <div>
-              <span className={styles.authorName}> By: {author.authorName}</span>
-
-              {
-                isExistingAuthor ?
-                  <Icon
-                    className={styles.alreadyExistsIcon}
-                    name={icons.CHECK_CIRCLE}
-                    size={15}
-                    title="Author already in your library"
-                  /> :
-                  null
-              }
             </div>
 
             <div>