-
Notifications
You must be signed in to change notification settings - Fork 330
Enhancement/#11812 - Implement new designs for the TourTooltip component
#11934
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
base: develop
Are you sure you want to change the base?
Changes from 6 commits
28216a8
79eceda
149b779
7cf0069
02481cb
fba95fd
6ece239
60a499e
6e420ba
ad8de9b
63fe862
c6e015f
7ffe840
d7ee0ac
1f4c165
7298718
5272605
c6b6dd2
7c628cd
4c5b748
73984f3
5888b8e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -220,3 +220,65 @@ | |
| margin-top: 42px !important; | ||
| top: 0 !important; | ||
| } | ||
|
|
||
| // Setup Flow Refresh variant styles. | ||
| .googlesitekit-tour-tooltip--setupFlowRefresh { | ||
|
|
||
| .googlesitekit-tooltip-title { | ||
| font-family: $f-primary; | ||
| font-size: $fs-title-lg; | ||
| font-weight: $fw-medium; | ||
| letter-spacing: $ls-xxs; | ||
| line-height: $lh-title-lg; | ||
| margin-bottom: 8px; | ||
| } | ||
|
|
||
| .googlesitekit-tooltip-content { | ||
| font-family: $f-primary; | ||
| font-size: $fs-body-md; | ||
| font-weight: $fw-normal; | ||
| letter-spacing: $ls-s; | ||
| line-height: $lh-body-md; | ||
| } | ||
nfmohit marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
|
||
| .googlesitekit-tooltip-body { | ||
| max-width: 332px; | ||
| padding: 32px $grid-gap-phone 0; | ||
| } | ||
|
|
||
| .googlesitekit-tooltip-close { | ||
| right: $grid-gap-phone; | ||
| top: $grid-gap-phone; | ||
| } | ||
|
|
||
| .googlesitekit-tooltip-button { | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It looks like we're also missing hover and focus styles for both the primary and tertiary buttons. We should clarify them with Sigal (if necessary) and add them. |
||
| border-radius: $br-lg; | ||
| line-height: 20px; | ||
|
||
|
|
||
| &:focus { | ||
| outline: none; | ||
| } | ||
|
|
||
| &:focus:not(:active) { | ||
| outline: 2px solid $c-interactive-inverse-focus; | ||
| } | ||
| } | ||
nfmohit marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| .googlesitekit-tooltip-button--primary { | ||
| background-color: $c-site-kit-sk-300; | ||
| padding: 6px $grid-gap-phone; | ||
| } | ||
|
|
||
| .googlesitekit-tooltip-steps { | ||
| font-family: $f-primary; | ||
| font-size: $fs-label-md; | ||
| font-weight: $fw-medium; | ||
| letter-spacing: $ls-xs; | ||
| line-height: $lh-label-md; | ||
| margin: 0; | ||
| } | ||
nfmohit marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| .googlesitekit-tooltip-actions { | ||
| padding: $grid-gap-phone; | ||
| } | ||
| } | ||



Uh oh!
There was an error while loading. Please reload this page.