Skip to content
Snippets Groups Projects
Unverified Commit 42a3e787 authored by Aleksandr Lesnenko's avatar Aleksandr Lesnenko Committed by GitHub
Browse files

Fix dashboard bugs (#23812)

* fix missing enter full-screen button

* fix dashboard background trimmed in the dark mode

* update specs
parent 6507582e
No related branches found
No related tags found
No related merge requests found
......@@ -23,6 +23,7 @@
/* Fullscreen mode */
.Dashboard.Dashboard--fullscreen {
min-height: 100vh;
height: auto;
}
.Dashboard.Dashboard--fullscreen .Header-button {
......
......@@ -9,7 +9,7 @@ import MetabaseSettings from "metabase/lib/settings";
import NightModeIcon from "metabase/components/icons/NightModeIcon";
import RefreshWidget from "metabase/dashboard/components/RefreshWidget";
import Tooltip from "metabase/components/Tooltip";
import Button from "metabase/core/components/Button";
import FullscreenIcon from "metabase/components/icons/FullscreenIcon";
import { DashboardHeaderButton } from "metabase/dashboard/containers/DashboardHeader.styled";
......@@ -137,20 +137,24 @@ export const getDashboardActions = (
);
}
if (!isEditing && !isEmpty && isFullscreen) {
if (!isEditing && !isEmpty && (isPublic || isFullscreen)) {
// option click to enter fullscreen without making the browser go fullscreen
buttons.push(
<Tooltip key="fullscreen" tooltip={t`Exit fullscreen`}>
<Tooltip
key="fullscreen"
tooltip={isFullscreen ? t`Exit fullscreen` : t`Enter fullscreen`}
>
<span
data-metabase-event={"Dashboard;Fullscreen Mode;" + !isFullscreen}
>
<Button
<DashboardHeaderButton
onClick={e => onFullscreenChange(!isFullscreen, !e.altKey)}
borderless
icon="contract"
iconSize={16}
onlyIcon
/>
>
<FullscreenIcon
className="text-brand-hover"
isFullscreen={isFullscreen}
/>
</DashboardHeaderButton>
</span>
</Tooltip>,
);
......
......@@ -194,6 +194,9 @@ describe("scenarios > public", () => {
cy.contains("Add filter").click();
cy.contains(COUNT_DOOHICKEY);
// Enter full-screen button
cy.icon("expand");
});
}),
);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment