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 KeePassDX icon #443

Merged
merged 1 commit into from
Oct 8, 2020
Merged

Add KeePassDX icon #443

merged 1 commit into from
Oct 8, 2020

Conversation

admvx
Copy link
Collaborator

@admvx admvx commented Oct 8, 2020

Trickier one! 😅 Addresses #374

  • New file which is under 1,024 bytes
  • Edit Readme
  • Add reference image
  • Edit reference page
  • Android Image

@edent
Copy link
Owner

edent commented Oct 8, 2020

Amazing use of feDropShadow! I don't think we've had one of those before.

I'm granting you commit access to the Repo. This is per the advice of The Pull Request Hack.

Please use this privilege wisely.

@edent edent merged commit 534625b into edent:master Oct 8, 2020
@admvx
Copy link
Collaborator Author

admvx commented Oct 8, 2020

Thank you @edent! I'll absolutely take the responsibility seriously – and I admire the philosophy of the linked post.

I must admit I had no cause to use feDropShadow before either 😁 Thanks again

@admvx admvx deleted the keypassdx branch October 8, 2020 19:25
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.

None yet

2 participants