search

neo4j-contrib / neoclipse

Graph Database Tool
218 stars 136 forks source link

Give explanations for disabled actions in context menu. #30

Closed jotomo closed 12 years ago

jotomo commented 12 years ago

When right clicking a node in the graph view and only a single node is selected, the "Create a relationship between two existing nodes" submenu reads "(disabled)" and is greyed out. It's not immediately obvious (at least not to everyone) that the action needs two selected nodes. As the action is greyed out and disabled, the text "(disabled)" is redundant and can be put to better use. This patch replaces the text with "Select two nodes to create a relationship between them", which gives a clear indication why the action is greyed out. The other actions are changed accordingly.

Hey @nawroth, according to git log you're the main developer for Neoclipse, so I'm pinging you on this. Patch is tested and working. Let me know if you think this is a good idea.

peterneubauer commented 12 years ago

Looks nice! @nrkkalyan you wanna pull it in?

nrkkalyan commented 12 years ago

Ya. Will do. Looks cool

Sent from my HTC

----- Reply message ----- From: "Peter Neubauer" reply@reply.github.com To: "Radhakrishna Kalyan" nrkkalyan@gmail.com Subject: [neoclipse] Give explanations for disabled actions in context menu. (#30) Date: Mon, May 21, 2012 10:07

Looks nice! @nrkkalyan you wanna pull it in?

Reply to this email directly or view it on GitHub: https://github.com/neo4j/neoclipse/pull/30#issuecomment-5817276

nawroth commented 12 years ago

@jotomo As you can see others are more active here nowadays :-)

Great fix btw!

peterneubauer commented 12 years ago

Yes, and keep it coming @jotomo!

On Mon, May 21, 2012 at 12:05 PM, Anders Nawroth reply@reply.github.com wrote:

@jotomo As you can see others are more active here nowadays :-)

Great fix btw!

Reply to this email directly or view it on GitHub: https://github.com/neo4j/neoclipse/pull/30#issuecomment-5819127