Skip to content

[WebProfilerBundle] Improved page for logs #13324

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

Closed
wants to merge 1 commit into from

Conversation

hason
Copy link
Contributor

@hason hason commented Jan 8, 2015

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

Before
logs-before
After
logs-after

@rvanlaak
Copy link
Contributor

rvanlaak commented Jan 8, 2015

👍 nice

Maybe use the new dump() to output the context?

@aitboudad
Copy link
Contributor

👍

@fabpot
Copy link
Member

fabpot commented Jan 16, 2015

👍

{{ log.priorityName }}
{%- endif -%}
</td>
<td>{{ log.channel is defined ? log.channel }}</td>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about removing the column when we don't have information about the channel?

@timglabisch
Copy link

looks good but needs a lot of space. i would like to see stats about rendering performance on large logs.

@fabpot
Copy link
Member

fabpot commented May 16, 2015

Can you rebase this one?

@hason
Copy link
Contributor Author

hason commented May 16, 2015

@fabpot Rebased.

@fabpot
Copy link
Member

fabpot commented May 17, 2015

Thank you @hason.

fabpot added a commit that referenced this pull request May 17, 2015
This PR was submitted for the 2.7 branch but it was merged into the 2.8 branch instead (closes #13324).

Discussion
----------

[WebProfilerBundle] Improved page for logs

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

*Before*
![logs-before](https://cloud.githubusercontent.com/assets/288535/5661713/3942c530-972f-11e4-89cb-2e185a6d07ac.png)
*After*
![logs-after](https://cloud.githubusercontent.com/assets/288535/5661681/d02b50da-972e-11e4-96e8-0572428ba82f.png)

Commits
-------

5fdc650 [WebProfilerBundle] Improved page for logs
@fabpot fabpot closed this May 17, 2015
@hason hason deleted the log branch May 18, 2015 09:25
@fabpot fabpot mentioned this pull request Nov 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants