minor updates to comments for clarity

This commit is contained in:
rr-bw 2024-09-07 13:03:15 -07:00
parent e190ca48c7
commit 4f893e0769
No known key found for this signature in database
GPG Key ID: 3FA13C3ADEE51D5D
2 changed files with 10 additions and 5 deletions

View File

@ -32,7 +32,7 @@
</button> </button>
</div> </div>
<!-- Login with Passkey link --> <!-- Link to Login with Passkey page -->
<div class="tw-mb-3 tw-flex tw-flex-col tw-items-center tw-justify-center"> <div class="tw-mb-3 tw-flex tw-flex-col tw-items-center tw-justify-center">
<p class="tw-mb-3">{{ "or" | i18n }}</p> <p class="tw-mb-3">{{ "or" | i18n }}</p>
<a <a
@ -48,6 +48,7 @@
<hr /> <hr />
<!-- Link to Create Account page -->
<p class="tw-m-0 tw-text-sm"> <p class="tw-m-0 tw-text-sm">
{{ "newAroundHere" | i18n }} {{ "newAroundHere" | i18n }}
<!-- Two notes: <!-- Two notes:
@ -71,11 +72,14 @@
------------------------------------> ------------------------------------>
<ng-container *ngIf="validatedEmail"> <ng-container *ngIf="validatedEmail">
<div class="tw-mb-6 tw-h-28"> <div class="tw-mb-6 tw-h-28">
<!-- Master Password input -->
<bit-form-field class="!tw-mb-1"> <bit-form-field class="!tw-mb-1">
<bit-label>{{ "masterPass" | i18n }}</bit-label> <bit-label>{{ "masterPass" | i18n }}</bit-label>
<input type="password" bitInput #masterPasswordInput formControlName="masterPassword" /> <input type="password" bitInput #masterPasswordInput formControlName="masterPassword" />
<button type="button" bitSuffix bitIconButton bitPasswordInputToggle></button> <button type="button" bitSuffix bitIconButton bitPasswordInputToggle></button>
</bit-form-field> </bit-form-field>
<!-- Link to Password Hint page -->
<a <a
class="tw-mt-2" class="tw-mt-2"
routerLink="/hint" routerLink="/hint"
@ -85,10 +89,12 @@
> >
</div> </div>
<!-- Captcha iframe -->
<div [hidden]="!showCaptcha()"> <div [hidden]="!showCaptcha()">
<iframe id="hcaptcha_iframe" height="80" sandbox="allow-scripts allow-same-origin"></iframe> <iframe id="hcaptcha_iframe" height="80" sandbox="allow-scripts allow-same-origin"></iframe>
</div> </div>
<!-- Button to Login with Master Password -->
<div class="tw-mb-3 tw-flex tw-space-x-4"> <div class="tw-mb-3 tw-flex tw-space-x-4">
<button bitButton buttonType="primary" bitFormButton type="submit" [block]="true"> <button bitButton buttonType="primary" bitFormButton type="submit" [block]="true">
<span> {{ "loginWithMasterPassword" | i18n }} </span> <span> {{ "loginWithMasterPassword" | i18n }} </span>

View File

@ -364,7 +364,7 @@ export class LoginComponentV2 implements OnInit, OnDestroy {
private async webOnInit(): Promise<void> { private async webOnInit(): Promise<void> {
this.activatedRoute.queryParams.pipe(first(), takeUntil(this.destroy$)).subscribe((qParams) => { this.activatedRoute.queryParams.pipe(first(), takeUntil(this.destroy$)).subscribe((qParams) => {
// If there is an query parameter called 'org', set previousUrl to `/create-organization?org=paramValue` // If there is a parameter called 'org', set previousUrl to `/create-organization?org=<paramValue>`
if (qParams.org != null) { if (qParams.org != null) {
const route = this.router.createUrlTree(["create-organization"], { const route = this.router.createUrlTree(["create-organization"], {
queryParams: { plan: qParams.org }, queryParams: { plan: qParams.org },
@ -373,9 +373,8 @@ export class LoginComponentV2 implements OnInit, OnDestroy {
} }
/** /**
* If there is a query parameter called 'sponsorshipToken', that means they are coming * If there is a parameter called 'sponsorshipToken', they are coming from an email for sponsoring a families organization.
* from an email for sponsoring a families organization. If so, then set the prevousUrl * Therefore set the prevousUrl to `/setup/families-for-enterprise?token=<paramValue>`
* to `/setup/families-for-enterprise?token=paramValue`
*/ */
if (qParams.sponsorshipToken != null) { if (qParams.sponsorshipToken != null) {
const route = this.router.createUrlTree(["setup/families-for-enterprise"], { const route = this.router.createUrlTree(["setup/families-for-enterprise"], {