Skip to content

Commit

Permalink
Merge pull request #4075 from glo71317/PWA-2615
Browse files Browse the repository at this point in the history
Venia header/layout broken in offline mode
  • Loading branch information
glo82145 authored May 15, 2023
2 parents 99925fa + 4420b4b commit 879a25c
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,6 @@ Array [
>
<NavTrigger />
</div>
<OnlineIndicator
hasBeenOffline={false}
isOnline={true}
/>
<MegaMenu />
<div
className="secondaryActions"
Expand All @@ -42,5 +38,9 @@ Array [
id="pageLoadingIndicator"
/>
</header>,
<OnlineIndicator
hasBeenOffline={false}
isOnline={true}
/>,
]
`;
9 changes: 5 additions & 4 deletions packages/venia-ui/lib/components/Header/header.js
Original file line number Diff line number Diff line change
Expand Up @@ -68,10 +68,7 @@ const Header = props => {
<div className={classes.primaryActions}>
<NavTrigger />
</div>
<OnlineIndicator
hasBeenOffline={hasBeenOffline}
isOnline={isOnline}
/>

<Link
aria-label={title}
to={resourceUrl('/')}
Expand All @@ -93,6 +90,10 @@ const Header = props => {
{searchBar}
<PageLoadingIndicator absolute />
</header>
<OnlineIndicator
hasBeenOffline={hasBeenOffline}
isOnline={isOnline}
/>
</Fragment>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,8 @@
composes: ml-18 from global;
composes: w-[3rem] from global;
grid-area: primary;

composes: sm_justify-self-start from global;
composes: sm_ml-10 from global;
composes: ml-6 from global;
}

@media (min-width: 640px) {
Expand Down

0 comments on commit 879a25c

Please sign in to comment.