-
Notifications
You must be signed in to change notification settings - Fork 135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clean up and clarify authz landing #1570
Conversation
This improves the "authorization" landing page (i.e. when authenticating to a dapp) in a few ways: * The dapps teaser/header is removed as it caused issues on small windows (CTA not in view) * The copy is clarified * The alt origin dropdown is changed to appear upon clicking the dapp domain
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thx
"to_continue_to": "to connect to", | ||
"is_alternative_of": "is an alternative domain of", | ||
"auth_same_identity": "and you will be authenticated to both with the same identity.", | ||
|
||
"first_time_create": "Securely connect to dapps on the Internet Computer", | ||
"first_time_create": "Securely connect with Internet Identity", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
asked my self if it should be phrased "Securely connect with an Internet Identity" so just sharing, just in case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, I think it depends whether the product is also still called Internet Identity...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Product did consider it, then went for the current wording
@@ -1990,7 +1990,7 @@ a:hover, | |||
} | |||
|
|||
.c-chasm__arrow { | |||
margin: auto; | |||
margin-left: calc(1.25 * var(--rs-card-bezel)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
side note:
@meodai 1.25 * var(--rs-card-bezel)
comes few times in the CSS file, four exactly, should a variable be extracted?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, definitely better wording than before.
"to_continue_to": "to connect to", | ||
"is_alternative_of": "is an alternative domain of", | ||
"auth_same_identity": "and you will be authenticated to both with the same identity.", | ||
|
||
"first_time_create": "Securely connect to dapps on the Internet Computer", | ||
"first_time_create": "Securely connect with Internet Identity", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, I think it depends whether the product is also still called Internet Identity...
This improves the "authorization" landing page (i.e. when authenticating to a dapp) in a few ways: