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

PHP: Multi-line strings can break function list #91

Closed
Hirse opened this issue Apr 24, 2017 · 2 comments
Closed

PHP: Multi-line strings can break function list #91

Hirse opened this issue Apr 24, 2017 · 2 comments
Labels
Milestone

Comments

@Hirse
Copy link
Owner

Hirse commented Apr 24, 2017

Details about your environment

  • Brackets Version: 1.9
  • Brackets Outline List Version: 1.1.0

What did you do? Please include the actual source code causing the issue.

<?php

# de wijzigingen opslaan
function ajaxEditAccount() {
    # als er een wachtwoord is ingvuld, kun je dit ook wijzigen
    if (!empty($context['account']['password'])) {
        $html .=
            '<li>
            <h4>Wachtwoord wijzigen</h4>
            </li>

            <li>
            <label>Nieuw wachtwoord</label>
            <input type="password" name="new_password1" value="" class="text-input" />
            </li>

            <li>
            <label>Bevestig nieuw wachtwoord</label>
            <input type="password" name="new_password2" value="" class="text-input" />
            </li>

            <li>
            <h4>Huidig wachtwoord</h4>
            </li>

            <li>
            <label>Wachtwoord</label>
            <input type="password" name="password" value="" class="text-input" />
            </li>';
    }
}

What did you expect to happen? How did you expect the Outline to look?

  • ajaxEditAccount()

What actually happened? How did the Outline actually look?

  • ajaxEditAccount()
  • value
  • value
  • li

@sQuarecoW Please see this issue for your reported problem.

@Hirse Hirse added the bug label Apr 24, 2017
@Hirse Hirse added this to the v1.2.0 milestone Apr 24, 2017
@Hirse Hirse closed this as completed in 8f2948e Apr 24, 2017
@sQuarecoW
Copy link

Nice find!

@sQuarecoW
Copy link

Yes, I can confirm it is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants