Просмотр исходного кода

Tweak breadcrump styling (#3030)

- It's now slightly more obvious the breadcrumps are links
- The active folder looks more active and can't be clicked again
- Fix eslint warnings
Merlijn Vos 3 лет назад
Родитель
Сommit
277b989316

+ 12 - 8
packages/@uppy/provider-views/src/Breadcrumbs.js

@@ -1,31 +1,35 @@
 const { h, Fragment } = require('preact')
 
 const Breadcrumb = (props) => {
+  const { getFolder, title, isLast } = props
+
   return (
     <Fragment>
       <button
         type="button"
         className="uppy-u-reset"
-        onClick={props.getFolder}
+        onClick={getFolder}
       >
-        {props.title}
+        {title}
       </button>
-      {!props.isLast ? ' / ' : ''}
+      {!isLast ? ' / ' : ''}
     </Fragment>
   )
 }
 
 module.exports = (props) => {
+  const { getFolder, title, breadcrumbsIcon, directories } = props
+
   return (
     <div className="uppy-Provider-breadcrumbs">
-      <div className="uppy-Provider-breadcrumbsIcon">{props.breadcrumbsIcon}</div>
+      <div className="uppy-Provider-breadcrumbsIcon">{breadcrumbsIcon}</div>
       {
-        props.directories.map((directory, i) => (
+        directories.map((directory, i) => (
           <Breadcrumb
             key={directory.id}
-            getFolder={() => props.getFolder(directory.id)}
-            title={i === 0 ? props.title : directory.title}
-            isLast={i + 1 === props.directories.length}
+            getFolder={() => getFolder(directory.id)}
+            title={i === 0 ? title : directory.title}
+            isLast={i + 1 === directories.length}
           />
         ))
       }

+ 11 - 1
packages/@uppy/provider-views/src/style.scss

@@ -94,8 +94,18 @@
     padding: 4px;
     border-radius: 3px;
 
-    &:hover {
+    &:not(:last-of-type) {
       text-decoration: underline;
+    }
+
+    &:last-of-type {
+      color: $gray-800;
+      font-weight: 500;
+      pointer-events: none;
+      cursor: normal;
+    }
+
+    &:hover {
       cursor: pointer;
     }