Implement busy-status integration

[ghost]

Hello

I've recently begun requiring intentions for php-integrator-refactoring to allow the user to select contextual commands from the list. This works very well and I've now been wondering if it would be a good idea to have some kind of placeholder (i.e. "No intentions available") if none of the providers return any intentions (or some way of taking a look at and possibly updating the final combined list from all providers).

Thanks in advance

[steelbrain]

We could have a notification but the IDE that this package was inspired from (PhpStorm) doesn't have do this. Can you please recommend a place to show this type of message?

[ghost]

It's a bit of a visual hint to the user. For example, suppose I have the following code:

public function test()
{

}

... where there is only an intent on the name of the method. If the user presses alt-enter on any other part of the code, no intentions will show up (which is of course, intended behavior). Showing the message here gives a form of "satisfaction" to the user that what he is trying to do did indeed do something (the intentions were requested) as opposed to "Did it do nothing at all, does the shortcut not work or are there simply no intentions available in this context?".

[steelbrain]

Another thing I could think of is that we could keep the spinner of busy-signal spinning while the providers are doing their thing and stop it once it's done. Spinner moving and then stopping but no list popping up would mean that there's no intentions at that particular location.

We could show a notification but it would be too noisy

[ghost]

That would also be a good idea. I agree that too much noise is best avoided. As such, atom notifications are absolutely overkill. It would be possible to show it inside the intentions menu itself (as if it were some sort of intention), but then care should be taken that it doesn't hog the cursor or it can become a nuisance there as well.

Another idea would be to have some other kind of indication, such as the cursor shaking or a quickly fading out message of some kind, but that is probably a little too much ;-).

[steelbrain]

Great, I'll be implementing busy-signal and will see how it goes

[steelbrain]

Hey sorry for lack of update but busy-signal is done and published, I'll be integrating it into this package soon