Skip to content

Commit 91878ed

Browse files
authored
Remove unintended drop shadow in light mode (#1391)
Summary: There was an elevation shadow from Material that was no longer needed. Without change... ![image](https://github.com/pixie-io/pixie/assets/314133/ca89b65f-b09f-4a29-a944-ae34d609edd2) With change... <img width="717" alt="image" src="https://github.com/pixie-io/pixie/assets/314133/198c7c3e-16f3-44bf-8fd2-4f064621aecb"> Type of change: /kind bugfix Test Plan: Open live view in light mode. No more weird boxy shadow on the breadcrumbs. --------- Signed-off-by: Nick Lanam <nlanam@pixielabs.ai>
1 parent e7b2b48 commit 91878ed

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/ui/src/components/breadcrumbs/__snapshots__/breadcrumbs-test.tsx.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
exports[`<Breadcrumbs/> renders correctly 1`] = `
44
<div>
55
<div
6-
class="MuiPaper-root MuiPaper-elevation MuiPaper-rounded MuiPaper-elevation1 Breadcrumbs-triggerWrapper-1 css-zsf01h-MuiPaper-root"
6+
class="MuiPaper-root MuiPaper-elevation MuiPaper-rounded MuiPaper-elevation0 Breadcrumbs-triggerWrapper-1 css-mvwo72-MuiPaper-root"
77
>
88
<div
99
class="Breadcrumbs-breadcrumbs-2"

src/ui/src/components/breadcrumbs/breadcrumbs.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -496,7 +496,7 @@ export const Breadcrumbs: React.FC<BreadcrumbsProps> = React.memo(({ breadcrumbs
496496
AutocompleteContext,
497497
);
498498
return (
499-
<Paper className={classes.triggerWrapper}>
499+
<Paper className={classes.triggerWrapper} elevation={0}>
500500
<div className={classes.breadcrumbs}>
501501
{breadcrumbs.map((breadcrumb, i) => (
502502
// Key is needed to prevent a console error when a key is missing in a list element.

0 commit comments

Comments
 (0)