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

Support favorites nodes #30

Closed
markiewb opened this issue Feb 24, 2013 · 15 comments
Closed

Support favorites nodes #30

markiewb opened this issue Feb 24, 2013 · 15 comments
Assignees
Milestone

Comments

@markiewb
Copy link
Collaborator

Currently it is not possible to open a console for the selected root-node in the favorites window. The action is not enabled.

EXPECTED: Support datashadow
Taken from my own plugin:

    private FileObject getFileObjectWithShadowSupport(DataObject dataObject) {
        if (dataObject instanceof DataShadow) {
            DataShadow dataShadow = (DataShadow) dataObject;
            return dataShadow.getOriginal().getPrimaryFile();
        }
        return dataObject.getPrimaryFile();
    }
@dzsessona
Copy link
Owner

Hi markiewb. Your code works fine, but there is a problem: the action is enabled with the cookie DataObject. At the moment i believe you can enable disable the action choosing one of:

-) Project
-) Open cookie
-) EditCookie
-) EditorCookie
-) DataObject

So the action in that case would be disabled. One thing i have done though is the following: you can open favourite root folders by using the "open terminal (or filesystem) in" ... action. I know there are a few more clicks but I think it could work, Here is a screenshot of what i have done, tell me what you think. Or if you know how to write a custom action enabler....

projects

@ghost ghost assigned dzsessona Feb 27, 2013
@markiewb
Copy link
Collaborator Author

Or if you know how to write a custom action enabler....

Alternative a) Use a CookieAction

Alternative b) Extend from AbstractAction

I know this will be some refactoring within your code, but afterwards it is more flexible then. I hope that will help you.

@markiewb
Copy link
Collaborator Author

markiewb commented Mar 8, 2013

Do you need any support?

@dzsessona
Copy link
Owner

Hi markieweb, thank you! Well, if you want, i cannot find not even 5 minutes to work on it. Hopefully next week i will not be so busy.

@markiewb
Copy link
Collaborator Author

markiewb commented May 2, 2013

@kinkadzs : Any news here?

@markiewb
Copy link
Collaborator Author

@kinkadzs : How about merging my pull request? Did you managed to review the patch? A new version for the plugin center would be very nice.

@dzsessona
Copy link
Owner

@markiewb i did review it but still for 7.3 friday 25 october will be done for 7.4 and will publish the plugin! :-)

@markiewb
Copy link
Collaborator Author

@kinkadzs Thank you!

@dzsessona
Copy link
Owner

working on it now....

@dzsessona
Copy link
Owner

@markiewb hi, i have just uploaded the 7.4 version, I merged your pull request but the root of favorites nodes still have the actions disabled... do you mind having a quick look at it? Now i should be able to work on this plugin a bit more btw.... And thanks, I mentioned you on netbeans center.

@markiewb
Copy link
Collaborator Author

@kinkadzs I will have a look within the next days. IIRC my patched worked times ago. I will figure it out and provide a new pull request if neccessary

@dzsessona
Copy link
Owner

@markiewb That's great! Thanks

@markiewb
Copy link
Collaborator Author

@kinkadzs : Please review #30 (reference)
The actions now work for favorite root nodes too, but I now the toolbar icons are gone. Can you figure that out?

@dzsessona
Copy link
Owner

cool, will check that on the weekend.

@markiewb markiewb added this to the 1.1.0 milestone Mar 17, 2016
@markiewb markiewb assigned markiewb and unassigned dzsessona Mar 17, 2016
markiewb added a commit that referenced this issue Mar 17, 2016
Introduced AbstractFileContextAwareAction, which encapsulates the context sensitivity
@markiewb
Copy link
Collaborator Author

Fixed in 1.1.0

It was easier than I originally thought

markiewb added a commit that referenced this issue Mar 17, 2016
Introduced AbstractFileContextAwareAction, which encapsulates the context sensitivity
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants