Skip to content

Commit c71086c

Browse files
authored
Fix welcome modal width (#8459)
* Fix welcome modal width * Use correct modal
1 parent fe08a56 commit c71086c

4 files changed

Lines changed: 48 additions & 14 deletions

File tree

Gemfile.lock

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -657,6 +657,7 @@ GEM
657657

658658
PLATFORMS
659659
arm64-darwin-23
660+
arm64-darwin-24
660661
x64-mingw32
661662
x86_64-linux
662663

app/javascript/components/modals/seniority-survey-modal/BootcampAdvertismentView.tsx

Lines changed: 19 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,28 @@
1-
import React, { useContext } from 'react'
2-
import { GraphicalIcon, Icon } from '@/components/common'
1+
import React from 'react'
2+
import { Icon } from '@/components/common'
33
import { FormButton } from '@/components/common/FormButton'
44
import { ErrorBoundary, ErrorMessage } from '@/components/ErrorBoundary'
5-
import { SenioritySurveyModalContext } from './SenioritySurveyModal'
65
import { useAppTranslation } from '@/i18n/useAppTranslation'
76
import { Trans } from 'react-i18next'
7+
import { MutationStatus } from '@tanstack/react-query'
88

99
const DEFAULT_ERROR = new Error('Unable to dismiss modal')
1010

11-
export function BootcampAdvertismentView() {
12-
const { patchCloseModal, links } = useContext(SenioritySurveyModalContext)
11+
type BootcampAdvertismentViewProps = {
12+
patchCloseModal: {
13+
mutate: () => void
14+
status: string
15+
error: unknown
16+
}
17+
links: {
18+
codingFundamentalsCourse: string
19+
}
20+
}
21+
22+
export function BootcampAdvertismentView({
23+
patchCloseModal,
24+
links,
25+
}: BootcampAdvertismentViewProps) {
1326
const { t } = useAppTranslation('components/modals/seniority-survey-modal')
1427

1528
return (
@@ -86,7 +99,7 @@ export function BootcampAdvertismentView() {
8699
{t('bootcampAd.learnMore')}
87100
</a>
88101
<FormButton
89-
status={patchCloseModal.status}
102+
status={patchCloseModal.status as MutationStatus}
90103
className="btn-secondary btn-l w-[140px]"
91104
type="button"
92105
onClick={patchCloseModal.mutate}

app/javascript/components/modals/seniority-survey-modal/SenioritySurveyModal.tsx

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -115,11 +115,18 @@ export default function SenioritySurveyModal({
115115

116116
function Inner() {
117117
const { t } = useAppTranslation('components/modals/seniority-survey-modal')
118-
const { currentView } = useContext(SenioritySurveyModalContext)
118+
const { currentView, patchCloseModal, links } = useContext(
119+
SenioritySurveyModalContext
120+
)
119121
switch (currentView) {
120122
case 'initial':
121123
return <InitialView />
122124
case 'bootcamp-advertisment':
123-
return <BootcampAdvertismentView />
125+
return (
126+
<BootcampAdvertismentView
127+
patchCloseModal={patchCloseModal}
128+
links={links}
129+
/>
130+
)
124131
}
125132
}

app/javascript/components/modals/welcome-modal/WelcomeModal.tsx

Lines changed: 19 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { sendRequest } from '@/utils/send-request'
44
import { Modal, ModalProps } from '../Modal'
55
import { InitialView } from './InitialView'
66
import { SeniorView } from './DeveloperView'
7-
import { JuniorView } from './JuniorView'
7+
import { BootcampAdvertismentView } from '../seniority-survey-modal/BootcampAdvertismentView'
88

99
type ViewVariant = 'initial' | 'beginner' | 'developer'
1010

@@ -134,9 +134,16 @@ export default function WelcomeModal({
134134
<Modal
135135
cover={true}
136136
open={open}
137-
style={
138-
currentView === 'initial' ? { content: { maxWidth: '590px' } } : {}
139-
}
137+
style={{
138+
content: {
139+
maxWidth:
140+
currentView === 'initial'
141+
? '590px'
142+
: currentView === 'beginner'
143+
? '620px'
144+
: '1180px',
145+
},
146+
}}
140147
{...props}
141148
onClose={() => null}
142149
className="m-welcome"
@@ -148,12 +155,18 @@ export default function WelcomeModal({
148155
}
149156

150157
function Inner() {
151-
const { currentView } = useContext(WelcomeModalContext)
158+
const { currentView, patchCloseModal, links } =
159+
useContext(WelcomeModalContext)
152160
switch (currentView) {
153161
case 'initial':
154162
return <InitialView />
155163
case 'beginner':
156-
return <JuniorView />
164+
return (
165+
<BootcampAdvertismentView
166+
patchCloseModal={patchCloseModal}
167+
links={links}
168+
/>
169+
)
157170
case 'developer':
158171
return <SeniorView />
159172
default:

0 commit comments

Comments
 (0)