Skip to content
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

Add lock images to static repo. #9

Merged
merged 1 commit into from
Mar 3, 2016
Merged

Add lock images to static repo. #9

merged 1 commit into from
Mar 3, 2016

Conversation

danfinlay
Copy link
Contributor

Having the ui in one project but the static resources in this one is a good sign that either:

  • We should have the UI in this project.
  • We should find a way to keep the static resources in the UI repo.

Sorry that they're pngs. They should be svgs, but Pixelmator apparently doesn't support svg export and I didn't realize until I'd already made this nice lock as a proof of concept.

Having the ui in one project but the static resources in this one is a good sign that either:

 - We should have the UI in this project.
 - We should find a way to keep the static resources in the UI repo.
@kumavis
Copy link
Member

kumavis commented Mar 3, 2016

yeah you are correct - this is a bit of a pain point. lets think about the best flow here. maybe breaking it out into its own react module and then doing some preprocessomg to inline the file makes the most sense...

kumavis added a commit that referenced this pull request Mar 3, 2016
Add lock images to static repo.
@kumavis kumavis merged commit ffdf913 into master Mar 3, 2016
@kumavis kumavis mentioned this pull request Mar 8, 2016
2 tasks
@kumavis kumavis deleted the AddLockImages branch May 24, 2016 20:46
@kumavis kumavis restored the AddLockImages branch May 24, 2016 20:46
@kumavis kumavis deleted the AddLockImages branch May 24, 2016 20:46
kumavis added a commit that referenced this pull request Feb 3, 2017
danfinlay added a commit that referenced this pull request Sep 19, 2019
meppsilon pushed a commit to meppsilon/metamask-extension that referenced this pull request Dec 1, 2021
aloisklink referenced this pull request in aloisklink/metamask-extension Jan 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants