When this package is enabled it overrides the outline view for PHP based files. Sometimes this is ok, when the PHP file is mostly HTML with inlined PHP (i.e. WordPress template files).
However, it outright breaks PHP files that are only PHP. So when working on PHP package development ide-html would have to be disabled. For PHP classes such as SslCertificate.php it just shows a single item ? and overrides ide-php or orther php parsers for Atom.
Expected Behavior
For PHP files that are mostly HTML with inline PHP, it would be best for the outline to just show something like <?php indicating that there's a block of php code. Basing the depth on the surrounding HTML.
For PHP files that are only PHP code - ideally the outline will just come from the ide-php or ide-seranata package and not be affected by the this one.
Visual Examples:
The real outline of an all PHP walker-nav file:
And a mock up of the mostly HTML template file:
Note: Image adjusted to 'fix' the current issue being described.
Actual Behavior
See images here:
This an all PHP file, same one as above:
And this is from the common header.php file for wordpress themes:
The only issue being that the PHP tags included add artficial depth to the a.brand-text tag. And that they could potentially be better represented by <?php rather than ?.
Possible Fix
Make PHP an optional extension to outline for - or improve interop with other ide-* parsers though that may not be as realistic. I'm not very familiar with Atom APIs so I'm not sure if such interop is possible.
Steps to Reproduce
Have ide-html enabled.
Open a .php file
Try to use outline.
Context
In order to properly debug real PHP files of any type I have to disable ide-html. Even in the PHP packages I maintain such as mallardduck/ssl-certificate.
Description
When this package is enabled it overrides the outline view for PHP based files. Sometimes this is ok, when the PHP file is mostly HTML with inlined PHP (i.e. WordPress template files).
However, it outright breaks PHP files that are only PHP. So when working on PHP package development
ide-html
would have to be disabled. For PHP classes such as SslCertificate.php it just shows a single item?
and overrideside-php
or orther php parsers for Atom.Expected Behavior
For PHP files that are mostly HTML with inline PHP, it would be best for the outline to just show something like
<?php
indicating that there's a block of php code. Basing the depth on the surrounding HTML.For PHP files that are only PHP code - ideally the outline will just come from the
ide-php
oride-seranata
package and not be affected by the this one.Visual Examples: The real outline of an all PHP walker-nav file:![walker-nav-real-outline](https://user-images.githubusercontent.com/619938/51284163-953d2500-19b8-11e9-9c90-2242b666efcc.png)
And a mock up of the mostly HTML template file:
Note: Image adjusted to 'fix' the current issue being described.
Actual Behavior
See images here:
This an all PHP file, same one as above:![walker-nav-outline](https://user-images.githubusercontent.com/619938/51284055-2e1f7080-19b8-11e9-8ec5-80edc75a8b5b.png)
And this is from the common
The only issue being that the PHP tags included add artficial depth to the a.brand-text tag. And that they could potentially be better represented by
header.php
file for wordpress themes:<?php
rather than?
.Possible Fix
Make PHP an optional extension to outline for - or improve interop with other
ide-*
parsers though that may not be as realistic. I'm not very familiar with Atom APIs so I'm not sure if such interop is possible.Steps to Reproduce
.php
fileContext
In order to properly debug real PHP files of any type I have to disable ide-html. Even in the PHP packages I maintain such as mallardduck/ssl-certificate.
Your Environment