-
Notifications
You must be signed in to change notification settings - Fork 984
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
Implement Information box and ens banner view #13753
Conversation
Jenkins BuildsClick to see older builds (22)
|
506c84c
to
3a4c442
Compare
3a4c442
to
ee36298
Compare
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.
nice! great job!
785d7b0
to
4f5d383
Compare
@@ -1,26 +1,25 @@ | |||
(ns quo2.foundations.typography) | |||
|
|||
;; Formulat for letter spacing from figma %: 16/{figma-percentage)*100 |
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.
this can't be the formula for letter-spacing, as it is inversely proportional,
when there should be more spacing it will give less, when there should be less like 0, it will give infinite
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.
[UPDATED] @Parveshdhull Figma can also express the letter spacing in em
rather than percentages. The body text having a spacing of -0.003em
cc @pedro-et
There's a formula provided for the Inter font here: https://rsms.me/inter/dynmetrics/ (formula for letter-spacing which is 'tracking'). As far as I can tell we haven't deviated from this formula and you should be able to use it to set the letter-spacing. I'll leave it to @pedro-et to confirm this though
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.
hi @hesterbruikman, Thank you very much for the link, I will explore the program/formula and see how it can be used.
968a8f3
to
4802dd0
Compare
4802dd0
to
e6fa742
Compare
94% of end-end tests have passed
Failed tests (5)Click to expandClass TestPairingSyncMultipleDevicesMerged:
Class TestEnsStickersMultipleDevicesMerged:
Passed tests (82)Click to expandClass TestRestoreOneDeviceMerged:
Class TestContactBlockMigrateKeycardMultipleSharedDevices:
Class TestKeycardTxOneDeviceMerged:
Class TestPairingSyncMultipleDevicesMerged:
Class TestCommandsMultipleDevicesMerged:
Class TestSendTxDeviceMerged:
Class TestOneToOneChatMultipleSharedDevices:
Class TestPublicChatMultipleDeviceMerged:
Class TestWalletManagementDeviceMerged:
Class TestGroupChatMultipleDeviceMerged:
Class TestPublicChatBrowserOneDeviceMerged:
Class TestOnboardingOneDeviceMerged:
Class TestEnsStickersMultipleDevicesMerged:
|
20% of end-end tests have passed
Failed tests (4)Click to expandClass TestEnsStickersMultipleDevicesMerged:
Passed tests (1)Click to expandClass TestPairingSyncMultipleDevicesMerged:
|
0% of end-end tests have passed
Failed tests (4)Click to expandClass TestEnsStickersMultipleDevicesMerged:
|
@Parveshdhull thanks for your great work!
One request: can you add an accessibility-id for this button, please? We'll need it for e2e fixes. After this, PR can be merged. |
sure will do, thank you |
e6fa742
to
92c5096
Compare
92c5096
to
bf3a5f2
Compare
Signed-off-by: Parvesh Monu <parvesh.dhullmonu@gmail.com>
bf3a5f2
to
f1f9fa1
Compare
fixes: #13749
fixes: #13743
Summary
PR implements Info Message and Information Box component, and also adds ENS Banner for release 1.20
status: ready