Skip to content

Commit

Permalink
Merge pull request #5730 from espoon-voltti/fix-mobile-settings-navba…
Browse files Browse the repository at this point in the history
…r-spinning

Korjataan työntekijän mobiilin navigaatiopalkki asetukset sivulla
  • Loading branch information
Joosakur authored Sep 30, 2024
2 parents ddeeaf9 + 327f9b4 commit 7b9f88f
Showing 1 changed file with 30 additions and 24 deletions.
54 changes: 30 additions & 24 deletions frontend/src/employee-mobile-frontend/settings/SettingsPage.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import BottomNavbar from '../common/BottomNavbar'
import TopBar from '../common/TopBar'
import { useTranslation } from '../common/i18n'
import { toUnitOrGroup } from '../common/unit-or-group'
import { MessageContextProvider } from '../messages/state'
import { RememberContext } from '../remember'
import { unitInfoQuery } from '../units/queries'

Expand All @@ -38,29 +39,34 @@ export const SettingsPage = React.memo(function SettingsPage({
const { user: userResponse } = useContext(UserContext)

return (
<ContentArea opaque paddingVertical="zero" paddingHorizontal="zero">
{renderResult(combine(userResponse, unitInfoResponse), ([user, unit]) => (
<>
<TopBar
title={unit.name}
unitId={unitId}
onBack={
user && user.unitIds.length > 1
? () => navigate('/units')
: undefined
}
/>
<ContentArea
opaque={false}
paddingVertical="zero"
paddingHorizontal="s"
>
<H1>{i18n.common.settings}</H1>
<NotificationSettings unitId={unitId} />
</ContentArea>
</>
))}
<BottomNavbar selected="settings" unitOrGroup={unitOrGroup} />
</ContentArea>
<MessageContextProvider unitOrGroup={unitOrGroup}>
<ContentArea opaque paddingVertical="zero" paddingHorizontal="zero">
{renderResult(
combine(userResponse, unitInfoResponse),
([user, unit]) => (
<>
<TopBar
title={unit.name}
unitId={unitId}
onBack={
user && user.unitIds.length > 1
? () => navigate('/units')
: undefined
}
/>
<ContentArea
opaque={false}
paddingVertical="zero"
paddingHorizontal="s"
>
<H1>{i18n.common.settings}</H1>
<NotificationSettings unitId={unitId} />
</ContentArea>
</>
)
)}
<BottomNavbar selected="settings" unitOrGroup={unitOrGroup} />
</ContentArea>
</MessageContextProvider>
)
})

0 comments on commit 7b9f88f

Please sign in to comment.