developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

#536 closed enhancement (fixed)

<script /> tags don't end with a newline

Reported by: SooW Owned by: laurentj
Priority: lowest Milestone: Jelix 1.0.4
Component: jelix:core response Version: 1.0.2
Severity: trivial Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

When I use addCSSLink(), XHTML output is well indented and each tag is shown on its own line. When I use addJSLink(), all tags are output on a single line. Adding a "\n" slightly improves readibility of the XHTML source.

I've attached the diff to fix it.

Attachments (1)

536.diff (669 bytes) - added by SooW 12 years ago.
The diff that fix core/response/jResponseHtml.class.php

Download all attachments as: .zip

Change History (5)

Changed 12 years ago by SooW

The diff that fix core/response/jResponseHtml.class.php

comment:1 Changed 12 years ago by laurentj

  • Milestone set to Jelix 1.0.4
  • review set to review+
  • Status changed from new to assigned

comment:2 Changed 12 years ago by bastnic

Soow, tu peux me dire ton vrai prénom/nom stp :p. C'est pour le commit.

comment:3 Changed 12 years ago by SooW

Warren Seine. Pour l'email, c'est warren.seine@….

Enfin, c'est faible comme modif' ;).

comment:4 Changed 12 years ago by bastnic

  • Resolution set to fixed
  • Status changed from assigned to closed

Faible ou pas, c'est commité ;).

Merci pour ce patch (même si tu avais oublié de prendre en compte les js IE).

Note: See TracTickets for help on using tickets.