diff --git a/web/src/ui/components/AnchorLink/AnchorLink.js b/web/src/ui/components/AnchorLink/AnchorLink.js index 4f3e22ae..a2b504b9 100644 --- a/web/src/ui/components/AnchorLink/AnchorLink.js +++ b/web/src/ui/components/AnchorLink/AnchorLink.js @@ -16,7 +16,7 @@ const AnchorLink = ({ 'badge-secondary', styles.badge, ) - const iconClasses = classNames(styles['copy-link-icon'], { [styles.block]: isBlock }) + const iconClasses = classNames(styles.copyLinkIcon, isBlock ? styles.block : styles.inline) const location = useLocation() return ( diff --git a/web/src/ui/components/AnchorLink/AnchorLink.module.scss b/web/src/ui/components/AnchorLink/AnchorLink.module.scss index 79a48218..4ab71ee4 100644 --- a/web/src/ui/components/AnchorLink/AnchorLink.module.scss +++ b/web/src/ui/components/AnchorLink/AnchorLink.module.scss @@ -4,17 +4,24 @@ position: absolute; } -.copy-link-icon { +.copyLinkIcon.inline { + cursor: pointer; + display: flex; + svg { + height: $font-size-sm; + } + @include xs { + display: none; + } +} + +.copyLinkIcon.block { display: none; @include xs { + display: flex; cursor: pointer; - display: block; - height: 1rem; - &.block { - display: flex; - width: 100%; - justify-content: center; - margin-top: 0.5rem; - } + width: 100%; + justify-content: center; + margin-top: 0.5rem; } } diff --git a/web/src/ui/components/Build/ArtifactRow.js b/web/src/ui/components/Build/ArtifactRow.js index 24100311..35cb2258 100644 --- a/web/src/ui/components/Build/ArtifactRow.js +++ b/web/src/ui/components/Build/ArtifactRow.js @@ -36,6 +36,12 @@ const QrCode = ({ artifactPlistSignedUrl, closeAction }) => ( ) +export const SharableArtifactLink = ({ buildId, artifactKind, isBlock }) => ( + + + +) + const ArtifactDownloadButton = ({ artifactPlistSignedUrl = '', artifactDlArtifactSignedUrl = '', @@ -138,14 +144,15 @@ const ArtifactDriver = ({ artifactDriver, theme }) => artifactDriver && ( ) -const SharableArtifactLink = ({ artifactId, implemented = false }) => implemented && ( - - - -) +// const SharableArtifactLink = ({ artifactId, implemented = false }) => implemented && ( +// +// +// +// ) const ArtifactRow = ({ artifact, + buildId, buildMergeUpdatedAt, buildStartedAt, buildFinishedAt, @@ -155,7 +162,6 @@ const ArtifactRow = ({ const { theme } = useContext(ThemeContext) const [showingQrModal, toggleShowQrModal] = useState(false) const { - id: artifactId = '', plist_signed_url: artifactPlistSignedUrl = '', dl_artifact_signed_url: artifactDlArtifactSignedUrl = '', kind: artifactKind = '', @@ -177,13 +183,14 @@ const ArtifactRow = ({ style={{ color: theme.text.sectionText, ...containerBorderBottomStyle }} >
- +
+
diff --git a/web/src/ui/components/Build/BuildAndMrContainer.js b/web/src/ui/components/Build/BuildAndMrContainer.js index b04d7ba1..b50c4804 100644 --- a/web/src/ui/components/Build/BuildAndMrContainer.js +++ b/web/src/ui/components/Build/BuildAndMrContainer.js @@ -110,6 +110,7 @@ const BuildAndMrContainer = ({ && buildHasArtifacts.map((artifact, i) => (