Skip to content

Commit

Permalink
fix edit button position
Browse files Browse the repository at this point in the history
  • Loading branch information
alexandraFlavia9 committed Nov 18, 2024
1 parent 9c3ff41 commit 3eca866
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
1 change: 1 addition & 0 deletions src/MainContent.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -331,6 +331,7 @@ function MainContent({ projectConfig, authToken, updateToken: setAuthToken }: Ma
{isMobileSize && (
<MobileVariablesTray
variables={variables}
isTimelineDisplayed={layoutIntent === LayoutIntent.digitalAnimated}
isDocumentLoaded={isDocumentLoaded}
/>
)}
Expand Down
5 changes: 3 additions & 2 deletions src/components/variables/MobileVariablesTray.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ import { getDataTestIdForSUI } from '../../utils/dataIds';

interface VariablesPanelProps {
variables: Variable[];
isTimelineDisplayed?: boolean;
isDocumentLoaded: boolean;
}

Expand All @@ -38,7 +39,7 @@ const imagePanelHeight = `
)`;

function MobileVariablesPanel(props: VariablesPanelProps) {
const { variables, isDocumentLoaded } = props;
const { variables, isDocumentLoaded, isTimelineDisplayed } = props;
const { panel } = useTheme();

const { contentType, showVariablesPanel, showDataSourcePanel } = useVariablePanelContext();
Expand Down Expand Up @@ -93,7 +94,7 @@ function MobileVariablesPanel(props: VariablesPanelProps) {

return (
<>
<EditButtonWrapper>
<EditButtonWrapper isTimelineDisplayed={isTimelineDisplayed}>
<Button
dataTestId={getDataTestIdForSUI('mobile-variables')}
variant={ButtonVariant.primary}
Expand Down
4 changes: 2 additions & 2 deletions src/components/variables/VariablesPanel.styles.ts
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
import { Colors, FontSizes, ITheme } from '@chili-publish/grafx-shared-components';
import styled from 'styled-components';

export const EditButtonWrapper = styled.div`
export const EditButtonWrapper = styled.div<{ isTimelineDisplayed?: boolean }>`
position: fixed;
left: 2rem;
bottom: 6.5rem;
bottom: ${({ isTimelineDisplayed }) => (isTimelineDisplayed ? '6.5rem' : '2.5rem')};
`;

export const VariablesContainer = styled.div<{ height?: string }>`
Expand Down

0 comments on commit 3eca866

Please sign in to comment.