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

Improvements #13

Merged
merged 22 commits into from Sep 9, 2014

Conversation

Projects
None yet
5 participants
@zerdei
Contributor

zerdei commented Feb 6, 2014

Hi,
I played around with your code, and made some changes.
It is quite usable now, so I replaced the good old Jd-gui to Luyten.
Try out my changes and pull/commit if you like.
Regards,
Zoltan Erdei

@PerfectSlayer

This comment has been minimized.

Show comment
Hide comment
@PerfectSlayer

PerfectSlayer May 25, 2014

Hey ! Your changes are very valuable ! 👍
Why are they still not merged ? Does the project is still alive ?

PerfectSlayer commented May 25, 2014

Hey ! Your changes are very valuable ! 👍
Why are they still not merged ? Does the project is still alive ?

@CalebFenton

This comment has been minimized.

Show comment
Hide comment
@CalebFenton

CalebFenton May 27, 2014

+1, would also like to see this merged.

CalebFenton commented May 27, 2014

+1, would also like to see this merged.

@PerfectSlayer

This comment has been minimized.

Show comment
Hide comment
@PerfectSlayer

PerfectSlayer May 27, 2014

Until it was merged, I started a fork on the @zerdei project.
Sad to see the original author is so few active.

PerfectSlayer commented May 27, 2014

Until it was merged, I started a fork on the @zerdei project.
Sad to see the original author is so few active.

@Quitten

This comment has been minimized.

Show comment
Hide comment
@Quitten

Quitten Aug 1, 2014

Contributor

👍 added a find all feature, what we can do if the author is inactive?

Contributor

Quitten commented Aug 1, 2014

👍 added a find all feature, what we can do if the author is inactive?

@PerfectSlayer

This comment has been minimized.

Show comment
Hide comment
@PerfectSlayer

PerfectSlayer Aug 1, 2014

I made myself a lot of change, including great performance improvement for Save all feature.
I started to rewrite the GUI but I have no more time with my current job…

PerfectSlayer commented Aug 1, 2014

I made myself a lot of change, including great performance improvement for Save all feature.
I started to rewrite the GUI but I have no more time with my current job…

@Quitten

This comment has been minimized.

Show comment
Hide comment
@Quitten

Quitten Aug 2, 2014

Contributor

we can open a new Luyten project by ourself based on the Luyten source code? i want to get merge our improvements to the real Luyten project.. what do you say?

Contributor

Quitten commented Aug 2, 2014

we can open a new Luyten project by ourself based on the Luyten source code? i want to get merge our improvements to the real Luyten project.. what do you say?

deathmarine pushed a commit that referenced this pull request Sep 9, 2014

@deathmarine deathmarine merged commit 7cb4944 into deathmarine:master Sep 9, 2014

@deathmarine

This comment has been minimized.

Show comment
Hide comment
@deathmarine

deathmarine Sep 9, 2014

Owner

Sorry guys... been distracted.

Owner

deathmarine commented Sep 9, 2014

Sorry guys... been distracted.

@Quitten

This comment has been minimized.

Show comment
Hide comment
@Quitten

Quitten Sep 11, 2014

Contributor

Hi, glad you came back, please get in more often, i will update my find all feature pull request in a few days, i will be glad if you will review it and merge it :)
regards.

Contributor

Quitten commented Sep 11, 2014

Hi, glad you came back, please get in more often, i will update my find all feature pull request in a few days, i will be glad if you will review it and merge it :)
regards.

deathmarine pushed a commit that referenced this pull request Sep 22, 2014

deathmarine
Merge pull request #13 from zerdei/master
Improvements
Fantastic Organization Skills
Thank zerdei
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment