Improved the logger panel when the log context is very long

[javiereguiluz]

Q	A
Bug fix?	no
New feature?	yes
BC breaks?	no
Deprecations?	no
Tests pass?	yes
Fixed tickets	-
License	MIT
Doc PR	-

Problem

When the context of a log message is long, it's very hard to read it:

Solution

Add a toggle which displays the context properly formatted in several lines:

To avoid cluttering the logs, this toggle is only displayed for really long contexts. For example, these three short contexts don't display that link:

[HeahDude]

Nice feature ! Thanks

[stof]

👍

[linaori]

Very nice! Just today I fixed a log that put a massive array in the log message but adding it to the context wasn't much better in terms of readability (besides of being lighter of color).

[xabbuh]

👍

Status: Reviewed

[fabpot]

I find it weird to have a JSON representation for the inline log and a pseudo-PHP one for the expanded view.

[javiereguiluz]

I think @fabpot is right. I'd like to display the original JSON expanded, formatted and highlighted.

Would you agree with including a client-side highlighter in the profiler? (I'm thinking about highlight.js) We could reuse it to highlight SQL queries (removing our custom highlighter), other log files, etc.

[fabpot]

I would do the opposite and always display a PHP-like output, that's much easier.

[javiereguiluz]

I've updated the PR to always display pseudo-PHP:

Short contexts

Long contexts (expanded)

---

If you like, we could even hide the full context when is very long:

Before

After

[fabpot]

I like the second version.

[javiereguiluz]

Updated. Let's recap the new behavior:

Short context (<= 120 chars)

Full context is displayed and no link is added:

Long context (> 120 chars)

Only the first 120 chars are displayed and a Show full context link is added:

When clicked, the full context is shown formatted in several lines:

[fabpot]

👍

[fabpot]

Thank you @javiereguiluz.