Skip to content

Commit 4f87431

Browse files
committed
Tweak action button
1 parent e2f1d1e commit 4f87431

File tree

2 files changed

+42
-18
lines changed

2 files changed

+42
-18
lines changed

website/src/components/client/ObjectView.tsx

Lines changed: 40 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -171,23 +171,49 @@ function ObjectListComponent({
171171
<TableCell sx={{ px: 2, py: 1 }}>{obj.getlastmodified}</TableCell>
172172
<TableCell sx={{ px: 2, py: 1 }}>
173173
{obj.iscollection ? (
174-
<Button
175-
variant="text"
176-
color="inherit"
177-
endIcon={<MenuOpen />}
178-
onClick={() => onExplore(obj.href)}
174+
<button
175+
onClick={(e) => {
176+
e.stopPropagation();
177+
onExplore(obj.href);
178+
}}
179+
aria-label={`Explore ${obj.href}`}
180+
style={{
181+
background: "transparent",
182+
border: "none",
183+
color: "var(--mui-palette-text-primary, #000)",
184+
padding: 0,
185+
cursor: "pointer",
186+
display: "inline-flex",
187+
alignItems: "center",
188+
gap: 6,
189+
fontSize: "0.95rem",
190+
}}
179191
>
180-
Explore
181-
</Button>
192+
<MenuOpen fontSize="small" />
193+
<span style={{ opacity: 0.85 }}>Explore</span>
194+
</button>
182195
) : (
183-
<Button
184-
variant="text"
185-
color="inherit"
186-
endIcon={<Download />}
187-
onClick={() => onDownload(obj.href)}
196+
<button
197+
onClick={(e) => {
198+
e.stopPropagation();
199+
onDownload(obj.href);
200+
}}
201+
aria-label={`Download ${obj.href}`}
202+
style={{
203+
background: "transparent",
204+
border: "none",
205+
color: "var(--mui-palette-text-primary, #000)",
206+
padding: 0,
207+
cursor: "pointer",
208+
display: "inline-flex",
209+
alignItems: "center",
210+
gap: 6,
211+
fontSize: "0.95rem",
212+
}}
188213
>
189-
Download
190-
</Button>
214+
<Download fontSize="small" />
215+
<span style={{ opacity: 0.85 }}>Download</span>
216+
</button>
191217
)}
192218
</TableCell>
193219
</TableRow>

website/src/components/client/PelicanWebClient.tsx

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -235,11 +235,9 @@ async function updateObjectUrlState(
235235
try {
236236
let objects: ObjectList[] = [];
237237

238-
// On every refetch, reset login state.
239-
// Otherwise, if there was an error within the URL, loginRequired would persist incorrectly.
240-
setLoginRequired(false);
241-
242238
// 1. Find normal objects
239+
// Also set loginRequired to false on every refetch, since list could fail due to 404
240+
setLoginRequired(false);
243241
try {
244242
objects = await list(`pelican://${objectPrefix}`, federation, namespace);
245243
} catch (e) {

0 commit comments

Comments
 (0)