docs: don't link to database engine pages (apache#11242)

* docs: don't link to database engine pages * making the linter happy * updating database installation link Co-authored-by: Evan Rusackas <evan@preset.io>
betodealmeida · Oct 13, 2020 · 8676c3e · 8676c3e
1 parent 31cc415
commit 8676c3e
Show file tree

Hide file tree

Showing 5 changed files with 32 additions and 39 deletions.
diff --git a/docs/src/images/databases/apache-kylin.png b/docs/src/images/databases/apache-kylin.png
diff --git a/docs/src/images/databases/clickhouse.png b/docs/src/images/databases/clickhouse.png
diff --git a/docs/src/images/databases/firebird.png b/docs/src/images/databases/firebird.png
diff --git a/docs/src/images/databases/oraclelogo.png b/docs/src/images/databases/oraclelogo.png
diff --git a/docs/src/pages/index.tsx b/docs/src/pages/index.tsx
@@ -190,8 +190,8 @@ const integrationSection = css`
     flex-wrap: wrap;
     justify-content: space-around;
     margin-bottom: 50px;
-    a {
-      margin: 15px;
+    li {
+      padding: 15px;
     }
   }
 `;
@@ -275,6 +275,7 @@ const Feature = ({ icon, title, descr }: featureProps) => (
     </div>
   </li>
 );
+
 const Theme = () => {
   const config = useConfig();
   const slider = useRef(null);
@@ -445,44 +446,36 @@ const Theme = () => {
                 </div>
               </Carousel>
             </div>
-          </div>
-          <div css={integrationSection}>
-            <h2 css={secondaryHeading}>Supported Databases</h2>
+            <div css={integrationSection}>
+              <h2 css={secondaryHeading}>Supported Databases</h2>
 
-            <ul className="database-list">
-              {Databases.map(
-                ({
-                  title, href, imgName: imageName, width, height,
-                }) => (
-                  <a
-                    href={href}
-                    target="_blank"
-                    key={imageName}
-                    rel="noreferrer nofollow"
-                  >
-                    <DbImage
-                      {...{
-                        imageName,
-                        width,
-                        height,
-                        alt: title,
-                      }}
-                    />
-                  </a>
-                ),
-              )}
-            </ul>
-            <span className="databaseSub">
-              {' '}
-              ... and any other SQLAlchemy
-              {' '}
-              <a href="https://superset.incubator.apache.org/installation.html#database-dependencies">
-                {' '}
-                compatible databases
-                {' '}
-              </a>
-              {' '}
-            </span>
+              <ul className="database-list">
+                {Databases.map(
+                  ({
+                    title, imgName: imageName, width, height,
+                  }) => (
+                    <li>
+                      <DbImage
+                        {...{
+                          imageName,
+                          width,
+                          height,
+                          alt: title,
+                        }}
+                      />
+                    </li>
+                  ),
+                )}
+              </ul>
+              <span className="databaseSub">
+                ... and any other
+                <a href="https://superset.apache.org/docs/databases/installing-database-drivers">
+                  {' '}
+                  compatible databases
+                  {' '}
+                </a>
+              </span>
+            </div>
           </div>
         </div>
       </Layout>