mirror/php-flarum
1
0
Fork 0
mirror of https://github.com/flarum/core.git synced 2025-07-12 20:36:28 +02:00
Code Issues Releases Wiki Activity

Fix email confirmation alert

Browse Source 
Currently, the controls are on a new line due to the container div. We want to wrap ALL children of the alert, including the controls, in the container div.

We need to split it into a separate class so that we can add modify the alert vnode AFTER the alert component's `view` logic has been applied.
This commit is contained in:
Alexander Skvortsov
2020-10-03 18:47:27 -04:00
parent 27d562f3fc
commit f8a0d9459a
1 changed files with 10 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
js/src/forum/utils/alertEmailConfirmation.js
View File

@ -52,13 +52,18 @@ export default function alertEmailConfirmation(app) {
} }
} }
class ContainedAlert extends Alert {
view(vnode) {
const vdom = super.view(vnode);
return { ...vdom, children: [<div className="container">{vdom.children}</div>] };
}
}
m.mount($('<div/>').insertBefore('#content')[0], { m.mount($('<div/>').insertBefore('#content')[0], {
view: () => ( view: () => (
<Alert dismissible={false} controls={[<ResendButton />]}> <ContainedAlert dismissible={false} controls={[<ResendButton />]}>
<div className="container"> {app.translator.trans('core.forum.user_email_confirmation.alert_message', { email: <strong>{user.email()}</strong> })}
{app.translator.trans('core.forum.user_email_confirmation.alert_message', { email: <strong>{user.email()}</strong> })} </ContainedAlert>
</div>
</Alert>
), ),
}); });
} }