+ renderWithLocalizationProvider(
+
Split layout content
);
- // Split layout does not render the card wrapper
- expect(container.querySelector('.card')).not.toBeInTheDocument();
+ expect(
+ screen.getByRole('img', {
+ name: MOCK_CMS_INFO_SPLIT_LAYOUT_BG.shared.backgrounds
+ ?.splitLayoutAltText as string,
+ })
+ ).toBeInTheDocument();
+
screen.getByText('Split layout content');
});
@@ -420,14 +426,19 @@ describe('', () => {
isLoading: false,
});
- const { container } = renderWithLocalizationProvider(
+ renderWithLocalizationProvider(
Default layout content
);
- // Default layout renders with card wrapper
- expect(container.querySelector('.card')).toBeInTheDocument();
+ expect(
+ screen.queryByRole('img', {
+ name: MOCK_CMS_INFO_SPLIT_LAYOUT_BG.shared.backgrounds
+ ?.splitLayoutAltText as string,
+ })
+ ).not.toBeInTheDocument();
+
screen.getByText('Default layout content');
// CMS info is still applied even when split layout is disabled
expect(screen.getByAltText('CMS Custom Logo')).toBeInTheDocument();
diff --git a/packages/fxa-settings/src/components/AppLayout/index.tsx b/packages/fxa-settings/src/components/AppLayout/index.tsx
index e75d576cd9c..953d0b4472d 100644
--- a/packages/fxa-settings/src/components/AppLayout/index.tsx
+++ b/packages/fxa-settings/src/components/AppLayout/index.tsx
@@ -12,7 +12,6 @@ import { RelierCmsInfo } from '../../models/integrations';
import { LocaleToggle } from '../LocaleToggle';
import { useConfig } from '../../models/hooks';
import { CardLoadingSpinner } from '../CardLoadingSpinner';
-import LoadingSpinner from 'fxa-react/components/LoadingSpinner';
import { useDynamicLocalization } from '../../contexts/DynamicLocalizationContext';
type AppLayoutProps = {
@@ -193,13 +192,13 @@ export const AppLayout = ({
/>
-
- {loading ? (
-
- ) : (
- children
- )}
-
+ {loading ? (
+
+ ) : (
+
+ )}
{showLocaleToggle && (