Dont't show favicons.

Fixes #92.

Author: RubenVerborgh

popup-app/components/IdpSelect.css

@@ -6,18 +6,17 @@
 
 .idp {
   flex: 1 100%;
+
   margin: .5em;
+  padding: .75em .5em;
 
   border: none;
-  border-radius: 2px;
-  padding: .5em;
+  border-radius: 10px;
   background-color: rgb(61, 109, 211);
   color: #fff;
+  transition: background-color 0.4s;
 
-  display: flex;
-  justify-content: space-between;
-  align-items: center;
-  transition: background-color 0.4s, box-shadow 0.4s;
+  cursor: pointer;
 }
 
 .idp:hover,
@@ -26,18 +25,6 @@
   box-shadow: 1px 1px rgb(46, 80, 151);
 }
 
-.idp:active {
-  cursor: pointer;
-  box-shadow: inset 1px 1px rgb(46, 80, 152);
-}
-
-.idp .icon {
-  display: flex;
-  height: 42px;
-  border-radius: 2px;
-  margin-left: .5em;
-}
-
 .custom-idp {
   display: flex;
   flex-wrap: nowrap;

popup-app/components/IdpSelect.js

@@ -121,14 +121,7 @@ class IdpSelect extends React.Component {
 
 const Idp = ({ idp, handleSelectIdp }) => (
   <button className="idp" onClick={handleSelectIdp}>
-    <span class="label">Log in with {idp.displayName}</span>
-    {idp.iconUrl ? (
-      <img className="icon" src={idp.iconUrl} alt="" />
-    ) : (
-      <svg className="icon" width="32" viewBox="0 0 100 20" alt="">
-        <path d="M41.2,50c0-4.9,4-8.8,8.8-8.8s8.8,4,8.8,8.8c0,4.9-4,8.8-8.8,8.8S41.2,54.9,41.2,50z M80.3,41.2c-4.9,0-8.8,4-8.8,8.8 c0,4.9,4,8.8,8.8,8.8s8.8-4,8.8-8.8C89.2,45.1,85.2,41.2,80.3,41.2z M19.7,41.2c-4.9,0-8.8,4-8.8,8.8c0,4.9,4,8.8,8.8,8.8 s8.8-4,8.8-8.8C28.5,45.1,24.5,41.2,19.7,41.2z" />
-      </svg>
-    )}
+    {idp.displayName}
   </button>
 )
 

popup-app/index.js

@@ -14,13 +14,11 @@ import './index.css'
 const defaultIdps = [
   {
     displayName: 'Solid Community',
-    url: 'https://solid.community/',
-    iconUrl: 'https://solidtest.space/favicon.ico'
+    url: 'https://solid.community/'
   },
   {
     displayName: 'Solid Test Space',
-    url: 'https://solidtest.space/',
-    iconUrl: 'https://solidtest.space/favicon.ico'
+    url: 'https://solidtest.space/'
   }
 ]
 
@@ -43,11 +41,7 @@ findAppOrigin()
     } else {
       const idps = [...defaultIdps]
       if (!idps.some(idp => idp.url === baseUrl)) {
-        idps.unshift({
-          displayName: host,
-          url: baseUrl,
-          iconUrl: baseUrl + 'favicon.ico'
-        })
+        idps.unshift({ displayName: host, url: baseUrl })
       }
       element = (
         <IdpSelect idps={idps} appOrigin={appOrigin} appName={appName} />