Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Redesigned the Symfony Profiler #15523

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.

Sign up for GitHub

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 42 commits into from
Closed

Redesigned the Symfony Profiler #15523

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
42 commits
Select commit Hold shift + click to select a range
b267347
Redesigned most of the Symfony Profiler panels
javiereguiluz Aug 12, 2015
4e66c45
Added some missing SVG icons
javiereguiluz Aug 12, 2015
b7f604c
Highlight the line of code which caused the exception
javiereguiluz Aug 12, 2015
3cdf546
CSS tweaks to fix some reported issues
javiereguiluz Aug 12, 2015
0a58431
A new round of tweaks and fixes
javiereguiluz Aug 13, 2015
9dc561c
Redesigned the "debug/dump" panel and other minor tweaks
javiereguiluz Aug 13, 2015
10f1fbb
Replaced the CSS-based tabs by proper JavaScript-based tabs
javiereguiluz Aug 14, 2015
a5f367c
First version of the redesigned Form panel
javiereguiluz Aug 17, 2015
6ca6b2f
Added the new icons, made the sidebar narrower, added badges in the
javiereguiluz Aug 19, 2015
22841fc
More design tweaks
javiereguiluz Aug 21, 2015
7cbced0
Refactored the "summary" section and redesigned all the profiler erro…
javiereguiluz Aug 21, 2015
50a60d5
Refactored the JavaScript code that creates the tabs
javiereguiluz Aug 21, 2015
656739e
Added a "toggle" utility
javiereguiluz Aug 21, 2015
b3aa953
Refactored the JavaScript code, used toggles in the Dump panel and
javiereguiluz Aug 23, 2015
376f1be
Finished the redesign of the Symfony Profiler
javiereguiluz Aug 24, 2015
3f6f747
Tweaks as per Stof's review and comments
javiereguiluz Aug 25, 2015
ab133e1
Minor tweaks for the Form panel
javiereguiluz Aug 25, 2015
904c618
Allow a toggle to define if its related content is displayed by default
javiereguiluz Aug 25, 2015
8a2a7cd
Replaced a randomly-generated ID by the real log index
javiereguiluz Aug 25, 2015
4e81feb
Lots of minor tweaks
javiereguiluz Aug 25, 2015
688e3f7
Minor JavaScript changes
javiereguiluz Aug 25, 2015
e47339b
Fixed the title of all panels and fixed some JavaScript issues
javiereguiluz Aug 26, 2015
3ac9100
Refactored some JavaScript code
javiereguiluz Aug 26, 2015
edb5e5e
Improved the vertical spacing of tabbed navigation
javiereguiluz Aug 26, 2015
4139c6b
Minor JavaScript change
javiereguiluz Aug 26, 2015
14b7cf5
More CSS tweaks to polish profiler panels
javiereguiluz Aug 26, 2015
a1d2e24
Redesigned the search results page
javiereguiluz Aug 27, 2015
489fbd6
Improved the timeline graphics and allowed panels to define their pag…
javiereguiluz Aug 27, 2015
437a67b
Simplified the management of the profiler panel titles
javiereguiluz Aug 28, 2015
72a64e3
Added some metrics about sub-requests in the performance panel
javiereguiluz Aug 28, 2015
f79894f
Fixed some CSS issues in Firefox
javiereguiluz Sep 1, 2015
2b55b9b
Allow to disable tabs, tweaked tables layout and other minor tweaks
javiereguiluz Sep 1, 2015
79f7297
Display the listener priority in the events panel
javiereguiluz Sep 3, 2015
50d5f59
Add a check to avoid errors for special pages (exceptions for example)
javiereguiluz Sep 3, 2015
14e725f
Fixed some test for the TraceableEventDispatcher changes
javiereguiluz Sep 3, 2015
b254b69
Sort uncalled events by priority
javiereguiluz Sep 3, 2015
67ef234
Grouped listeners by event in the events panel
javiereguiluz Sep 3, 2015
f971509
Fixed an HTML markup issue for the new events panel
javiereguiluz Sep 3, 2015
ac6f79a
Adjusted the logger panel after the grouping of related deprecations
javiereguiluz Sep 7, 2015
ac63334
Tweaked the logger panel stack trace contents
javiereguiluz Sep 7, 2015
a808fa5
Tweaked the toolbar for Silex 2
javiereguiluz Sep 7, 2015
73809a8
Removed the app.request... call to make it compatible with Silex
javiereguiluz Sep 7, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Lots of minor tweaks
  • Loading branch information
@javiereguiluz
javiereguiluz committed Sep 7, 2015
commit 4e81feb6ffb423676ca946ea95c520d610513cfa
8 changes: 6 additions & 2 deletions 8 src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/logger.html.twig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -117,12 +117,13 @@

{% macro render_table(logs, show_level = false, is_deprecation = false) %}
{% import _self as helper %}
Copy link
Member

Choose a reason for hiding this comment

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

I would prefer to keep using the {% from %} tag to import the needed macro, as done previously

Copy link
Member Author

Choose a reason for hiding this comment

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

I've made all the changes suggested in your other comments, but I'm struggling with this one.

First I don't really know why {% from ... %} is recommended instead of {% import .... Second, I though that perfixing all utility macros with the helper. prefix would be better for code legibility. But of course this could be a bad idea :) I just need some more comments about this. Thanks.

{% set channel_is_defined = (logs|first).channel is defined %}

<table class="auto-layout logs">
<thead>
<tr>
<th>{{ show_level ? 'Level' : 'Time' }}</th>
<th>Channel</th>
{% if channel_is_defined %}<th>Channel</th>{% endif %}
<th>Message</th>
</tr>
</thead>
Expand All @@ -141,7 +142,10 @@
<span class="text-muted nowrap newline">{{ log.timestamp|date('H:i:s') }}</span>
</td>

<td class="font-normal text-small text-bold nowrap">{{ log.channel }}</td>
{% if channel_is_defined %}
<td class="font-normal text-small text-bold nowrap">{{ log.channel }}</td>
{% endif %}

<td class="font-normal">{{ helper.render_log_message(loop.index, log, is_deprecation) }}</td>
</tr>
{% endfor %}
Expand Down
2 changes: 1 addition & 1 deletion 2 src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/memory.html.twig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

<div class="sf-toolbar-info-piece">
<b>PHP memory limit</b>
<span>{{ collector.memoryLimit == -1 ? 'Unlimited' : '%.0f MB'|format(collector.memoryLimit / 1024 / 1024)|escape }}</span>
<span>{{ collector.memoryLimit == -1 ? 'Unlimited' : '%.0f MB'|format(collector.memoryLimit / 1024 / 1024) }}</span>
</div>
{% endset %}

Expand Down
2 changes: 1 addition & 1 deletion 2 src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/request.html.twig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@
{% endif %}

<h3>Request Headers</h3>
{{ include('@WebProfiler/Profiler/bag.html.twig', { bag: collector.requestheaders }, with_context = false) }}
{{ include('@WebProfiler/Profiler/bag.html.twig', { bag: collector.requestheaders, labels: ['Header', 'Value'] }, with_context = false) }}

<h3>Request Content</h3>

Expand Down
46 changes: 23 additions & 23 deletions 46 src/Symfony/Bundle/WebProfilerBundle/Resources/views/Collector/time.html.twig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,34 +48,34 @@
{% endblock %}

{% block panel %}
{% if collector.events is empty %}
<div class="empty">
<p>No timing events have been recorded. Are you sure that debugging is enabled in the kernel?</p>
<h2>Performance metrics</h2>

<div class="metrics">
<div class="metric">
<span class="value">{{ '%.0f'|format(collector.duration) }} <span class="unit">ms</span></span>
<span class="label">Total execution time</span>
</div>
{% else %}
<h2>Performance metrics</h2>

<div class="metrics">
<div class="metric">
<span class="value">{{ '%.0f'|format(collector.duration) }} <span class="unit">ms</span></span>
<span class="label">Total execution time</span>
</div>
<div class="metric">
<span class="value">{{ '%.0f'|format(collector.inittime) }} <span class="unit">ms</span></span>
<span class="label">Symfony initialization</span>
</div>

{% if profile.collectors.memory %}
<div class="metric">
<span class="value">{{ '%.0f'|format(collector.inittime) }} <span class="unit">ms</span></span>
<span class="label">Symfony initialization</span>
<span class="value">{{ '%.2f'|format(profile.collectors.memory.memory / 1024 / 1024) }} <span class="unit">MB</span></span>
<span class="label">Peak memory usage</span>
</div>
{% endif %}
</div>

{% if profile.collectors.memory %}
<div class="metric">
<span class="value">{{ '%.2f'|format(profile.collectors.memory.memory / 1024 / 1024) }} <span class="unit">MB</span></span>
<span class="label">Peak memory usage</span>
</div>
{% endif %}
</div>

<h2>Execution timeline</h2>
<h2>Execution timeline</h2>

{% if collector.events is empty %}
<div class="empty">
<p>No timing events have been recorded. Are you sure that debugging is enabled in the kernel?</p>
</div>
{% else %}
{{ block('panelContent') }}
{% endif %}
{% endblock %}
Expand All @@ -84,8 +84,8 @@
<form id="timeline-control" action="" method="get">
<input type="hidden" name="panel" value="time">
<label for="threshold">Threshold</label>
Copy link
Member

Choose a reason for hiding this comment

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

for references an id. Your input has no id.

<input type="number" size="3" name="threshold" value="3" min="0"> ms
<span class="help">(timeline only displays events slower than this threshold)</span>
<input type="number" size="3" name="threshold" id="threshold" value="3" min="0"> ms
<span class="help">(timeline only displays events with a duration longer than this threshold)</span>
</form>

{% if profile.parent %}
Expand Down
4 changes: 2 additions & 2 deletions 4 src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/search.html.twig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
<select name="method" id="method">
<option value="">Any</option>
{% for m in ['DELETE', 'GET', 'HEAD', 'PATCH', 'POST', 'PUT'] %}
<option {{ m == method ? 'selected="selected"' : '' }}>{{ m }}</option>
<option {{ m == method ? 'selected="selected"' }}>{{ m }}</option>
{% endfor %}
</select>
</div>
Expand Down Expand Up @@ -39,7 +39,7 @@
<label for="limit">Results</label>
<select name="limit" id="limit">
{% for l in [10, 50, 100] %}
<option {{ l == limit ? 'selected="selected"' : '' }}>{{ l }}</option>
<option {{ l == limit ? 'selected="selected"' }}>{{ l }}</option>
{% endfor %}
</select>
</div>
Expand Down
Morty Proxy This is a proxified and sanitized view of the page, visit original site.