Converting link property to button.

mlibrary · Aug 18, 2023 · 1e51a8a · 1e51a8a
1 parent fc0f4d1
commit 1e51a8a
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 13 deletions.
diff --git a/src/modules/core/components/SearchHeader/index.js b/src/modules/core/components/SearchHeader/index.js
@@ -24,16 +24,9 @@ function SearchHeader (props) {
             marginRight: '1rem',
             marginBottom: '0.5rem'
           },
-          'a, button.link': {
-            background: 'none',
-            border: '0',
-            color: 'white',
-            fontFamily: 'inherit',
-            margin: '0',
-            padding: '0',
+          a: {
             textDecoration: 'none',
             '&:hover': {
-              cursor: 'pointer',
               textDecoration: 'underline'
             }
           },
@@ -48,7 +41,7 @@ function SearchHeader (props) {
         }}
       >
         <a href='https://account.lib.umich.edu/'>Account</a>
-        <Authentication logout={props.isAuthenticated} link />
+        <Authentication logout={props.isAuthenticated} />
       </nav>
     </m-website-header>
   );

diff --git a/src/modules/getthis/components/GetThisOptionList/index.js b/src/modules/getthis/components/GetThisOptionList/index.js
@@ -61,7 +61,7 @@ function GetThisOptions (props) {
   if (status === 'Not logged in') {
     return (
       <Section>
-        <Authentication>
+        <Authentication button>
           <span className='strong'>Log in</span> to view request options
         </Authentication>
       </Section>

diff --git a/src/modules/profile/components/Authentication/index.js b/src/modules/profile/components/Authentication/index.js
@@ -6,7 +6,7 @@ export default function Authentication (props) {
   return (
     <a
       href={props.logout ? config.logoutUrl : config.loginUrl}
-      className={props.link ? 'link' : 'button'}
+      className={props.button && 'button'}
     >
       {props.children || 'Log ' + (props.logout ? 'out' : 'in')}
     </a>
@@ -15,7 +15,7 @@ export default function Authentication (props) {
 
 Authentication.propTypes = {
   logout: PropTypes.bool,
-  link: PropTypes.bool,
+  button: PropTypes.bool,
   children: PropTypes.oneOfType([
     PropTypes.arrayOf(PropTypes.node),
     PropTypes.node

diff --git a/src/modules/profile/components/AuthenticationRequired/index.js b/src/modules/profile/components/AuthenticationRequired/index.js
@@ -13,7 +13,7 @@ function AuthenticationRequired (props) {
   }
 
   return (
-    <Authentication>
+    <Authentication button>
       <span className='strong'>Log in</span> to continue
     </Authentication>
   );