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

Download favorite icon of entry url and add to the custom icon registry #30

Merged
merged 4 commits into from
Oct 6, 2016

Conversation

droidmonkey
Copy link
Member

No description provided.

@droidmonkey droidmonkey changed the title Download favorite icon of renty url and add to the custom icon registry Download favorite icon of entry url and add to the custom icon registry Oct 6, 2016
@droidmonkey droidmonkey merged commit c4b3f08 into develop Oct 6, 2016
@droidmonkey
Copy link
Member Author

droidmonkey commented Oct 6, 2016

I would still like to extend this functionality by adding a right click menu option to fetch the icon, maybe you can code that with this baseline as a start? I greatly simplified the functions.

Also you can wish all you want, but if you don't tell anyone you are working on something then no one will know. Recommend opening an issue next time to announce your intentions.

@droidmonkey droidmonkey deleted the feature/favicon branch October 6, 2016 03:11
@magkopian
Copy link
Contributor

magkopian commented Oct 6, 2016

I know, I should have opened an issue but I was a bit overexcited about that feature and I completely omitted it. And also, actually that was the very first time I was working with qt and I wasn't exactly sure if I would be able to do it. In fact, I'm a PHP developer but I wanted that feature so much so I decided to give it a try anyway, even though I didn't know much about c++ / qt applications and see how it goes. So, that's actually the major reason I didn't open an issue, I didn't want to get people excited about this and then just say sorry I can't do it.

Anyway, I just found it funny that we were working on the exact same feature the exact same time, while no one had the idea about implementing something similar it in the past, what are the odds. It was a fun experience though, it was interesting working on something desktop instead of web based for a change.

@droidmonkey
Copy link
Member Author

Hehe, it has actually been developed two other times on the original KeePassX repo, just never incorporated. I took one of the original pull requests and made it better and incorporated it in. I look forward to your future contributions!

@TheZ3ro TheZ3ro added this to the v2.1.0 milestone Nov 5, 2016
@Germano0 Germano0 mentioned this pull request Dec 15, 2021
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.

4 participants