-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Google Fonts picker - remove need for google api #3050
Conversation
Build for latest commit 0f0f685 is at https://pr3050.build.csb.dev/s/new. |
import Color from 'color'; | ||
|
||
const svg = props => |
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.
For later: oh noes, would be cool to drop this in a file so it's discoverable?
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.
Honestly I don't think we will ever used this again, I could have done in css now that I think of it
packages/app/src/app/pages/Sandbox/Editor/Workspace/Dependencies/AddFont/FontPicker/elements.ts
Show resolved
Hide resolved
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.
looks good! ship it
|
Yeah! Something is wrong with the build, I am gonna talk to bogdan |
Merge with master should fix it!
…On Thu, 21 Nov 2019 at 13:23, Sara Vieira ***@***.***> wrote:
Yeah! Something is wrong with the build, I am gonna talk to bogdan
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#3050?email_source=notifications&email_token=AAEPKCCKGU3AAI5ROS5IPK3QUZVRTA5CNFSM4JPWMXQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEZ4QTQ#issuecomment-557041742>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEPKCAZKV64RHBU5JZA4T3QUZVRTANCNFSM4JPWMXQA>
.
|
Google api was needed because of the display of the font in his style and with that removed there is no need to use an api key