Skip to content

Commit ad168f2

Browse files
sandroscostaigobranco
authored andcommitted
fix: wide header on mobile
Add a new flexbox rule to guarantee correct separation of components. Create a new container for the right elements.
1 parent 69b9d8d commit ad168f2

File tree

1 file changed

+28
-17
lines changed

1 file changed

+28
-17
lines changed

src/learning-header/LearningHeader.jsx

Lines changed: 28 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ const LearningHeader = ({
6464
return (
6565
<header className="learning-header">
6666
<a className="sr-only sr-only-focusable" href="#main-content">{intl.formatMessage(messages.skipNavLink)}</a>
67-
<div className="container-xl py-2 d-flex align-items-center">
67+
<div className="container-xl py-2 d-flex align-items-center justify-content-between">
6868
{headerLogo}
6969
<div className="d-none d-md-block flex-grow-1 course-title-lockup">
7070
<div className={`d-md-flex ${enableOrgLogo && 'align-items-center justify-content-center'} w-100`}>
@@ -79,23 +79,34 @@ const LearningHeader = ({
7979
</span>
8080
</div>
8181
</div>
82-
{getConfig().ENABLE_HEADER_LANG_SELECTOR && (
83-
<div className="mx-2 d-md-inline-flex">
84-
<LanguageSelector
85-
options={JSON.parse(getConfig().SITE_SUPPORTED_LANGUAGES)}
86-
compact
87-
authenticatedUser={authenticatedUser}
82+
<div className="d-flex align-items-center">
83+
{getConfig().ENABLE_HEADER_LANG_SELECTOR && (
84+
<div className="mx-2 d-md-inline-flex">
85+
<Responsive maxWidth={1200}>
86+
<LanguageSelector
87+
options={JSON.parse(getConfig().SITE_SUPPORTED_LANGUAGES)}
88+
compact
89+
authenticatedUser={authenticatedUser}
90+
/>
91+
</Responsive>
92+
<Responsive minWidth={1200}>
93+
<LanguageSelector
94+
options={JSON.parse(getConfig().SITE_SUPPORTED_LANGUAGES)}
95+
compact={false}
96+
authenticatedUser={authenticatedUser}
97+
/>
98+
</Responsive>
99+
</div>
100+
)}
101+
{showUserDropdown && authenticatedUser && (
102+
<AuthenticatedUserDropdown
103+
username={authenticatedUser.username}
88104
/>
89-
</div>
90-
)}
91-
{showUserDropdown && authenticatedUser && (
92-
<AuthenticatedUserDropdown
93-
username={authenticatedUser.username}
94-
/>
95-
)}
96-
{showUserDropdown && !authenticatedUser && (
97-
<AnonymousUserMenu />
98-
)}
105+
)}
106+
{showUserDropdown && !authenticatedUser && (
107+
<AnonymousUserMenu />
108+
)}
109+
</div>
99110
</div>
100111
</header>
101112
);

0 commit comments

Comments
 (0)