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

NotWorking: Attempt at navigatoreContent extension #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nickl-
Copy link

@nickl- nickl- commented Nov 14, 2017

As per issues #25 I attempted to create a navigatorContent extension for browsing jar files.

The implementation is not perfect but it should at least do something. If nothing else I would've been pleased with some sort of exception but nada...

Not sure what to try next?!?!?

I included the an attribute I found to set the decompiler association by default, this does seem to work but it could also just be cache that is allowing this to association to be set, again I cannot be sure.

Submitting this for fresh eyes who may be able to spot something I may have done wrong or something that I missing as I cannot think of anything else to try. It simply does nothing.

@RobertZenz
Copy link
Collaborator

Thank you very much for working on this, but I might not have the time to look at this for the next few days, but will do.

@nickl-
Copy link
Author

nickl- commented Nov 16, 2017

No worries, take your time.

As mentioned it is not perfect but it should at least do something. Perhaps my testing procedure is faulty I tried via Plugins -> Launch eclipse.... and installing a new plugin from the compiled source folder but neither would produce any changes. Unless I can see that I am changing something I have no idea how to go forward with this.

Thought I'd at least submit what I've done maybe you can see something I am not.

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