search

m-bobrowicz / php-integrator-symbol-viewer

Atom package providing side panel with class information: constants, properties and methods with go-to functionality provided by php-integrator-base package.
GNU General Public License v3.0
20 stars 11 forks source link

command to toggle panel's visibility #8

Closed m-bobrowicz closed 8 years ago

Arcesilas commented 8 years ago

:+1:

Seems a very nice package, we just need to know how to use it. I've tried many possibilities in the command palette, but nothing to display the panel.

m-bobrowicz commented 8 years ago

Hi!

This is the issue for a feature that is not yet implemented, that is the command is not available right now. If properly installed, the panel should be visible in any PHP file. Here are some ideas what might be wrong: 1) php-integrator-base is not installed 2) php-integrator-base is not working properly (see FAQ)

I guess if there are no error notifications one of the clues should help.

Regrards, tocjent

Arcesilas commented 8 years ago

Hi,

Thanks for your answer. php-integrator base installed and working correctly (the command that checks configuration displays a nice green message). Using [https://atom.io/packages/symbols-tree-view](Symbols Tree View) for now, it does the job, even if I'd like the display to be a little more configurable (or just more natural: order classes, interfaces, traits, variables, functions,...)

I keep an eye on php-integrator though.

I'd be glad to help by providing some information about configuration, Atom version, debug traces if there are some, etc.

I guess I'd open a new ticket about that, to report the real issue.

Arcesilas commented 8 years ago

No ticket, since I don't know why, default timezone was actually not set (I was sure it was!). You could, maybe, check for this in the configuration test ?

Works now, but this feature would be great, since one may not always want to have symbols listed (it takes some place too).

Thanks!

Regards

m-bobrowicz commented 8 years ago

php-integrator-base already has a config check, so I think the ticket belongs there. If I have some free time maybe I'll investigate it there.

Toggling visibility is on the top of my priorities list and will be part of the next release, which will be coming probably at the end of the week.