Please avoid ternary syntax in JSX – see here

Can you clarify why we have two components that seem to be do the same thing, with a lot of duplicated code?

Can one of these be deprecated?

If not, can we factor them out so the code is shared? I recall at some point sharing feedback about adding a colon character (:) at the end of this sentence... it was only applied to the other component, which is a class of problem that should be addressed by DRY ("Don't Repeat Yourself") principles.

Same comment here regarding ternary in JSX

Can you clarify why we have two components that seem to be do the same thing, with a lot of duplicated code?

QRCodeLogin will be removed after we launch signed device lists.

function LoginFormWrapper(): React.Node {
  const usingRestoreFlow = useIsRestoreFlowEnabled();
  if (!usingRestoreFlow) {
    return <LegacyLoginForm />;
  }
  return <LoginForm />;
}

It is a part of LegacyLoginForm that can be deleted after usingRestoreFlow becomes true.

Since QRCodeLogin will be removed soon anyway, I'm not deduplicating this.

https://phab.comm.dev/D14381 added a colon