Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[WebProfilerBundle] Make the IP address in the profiler header clickable#19815
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
fabpot commentedSep 6, 2016
I know that we already had some discussion about this topic. ping@javiereguiluz |
xabbuh commentedOct 13, 2016
What is the actual use case for this change? |
jameshalsall commentedOct 13, 2016
This is to make it easier to jump to results for the current IP, which brings it inline with the behaviour of clicking on the URL. |
xabbuh commentedOct 13, 2016
Let's answer two questions: How common is it that you have profiles with different IP addresses which would justify a need for this feature? And would it be confusing for all other users that the IP address is clickable? |
jameshalsall commentedOct 13, 2016
|
javiereguiluz commentedNov 7, 2016
👍 I like this proposal because it changes nothing for most users and it improves the productivity of those using this. @jameshalsall I'd like to have a minor change before merge: let's make the new link look like the previous text. In line 494 of profiler.css.twig, please update the selector to this: #summarydl.metadata,#summarydl.metadataa {/* <-- add this line */margin:5px00;color:rgba(255,255,255,0.75);} |
…ble to view requests by IP
bd45598 to52606a1Comparejameshalsall commentedNov 9, 2016
This has now been updated. |
nicolas-grekas left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
👍
fabpot commentedFeb 14, 2017
Thank you@jameshalsall. |
…r header clickable (jameshalsall)This PR was merged into the 3.3-dev branch.Discussion----------[WebProfilerBundle] Make the IP address in the profiler header clickable| Q | A || --- | --- || Branch? | master || Bug fix? | no || New feature? | yes || BC breaks? | no || Deprecations? | no || Tests pass? | yes || License | MIT |Commits-------52606a1 [WebProfilerBundle] Make the IP address in the profiler header clickable to view requests by IP
…profiler header clickable (jameshalsall)This PR was merged into the 3.3-dev branch.Discussion----------[WebProfilerBundle] Make the IP address in the profiler header clickable| Q | A || --- | --- || Branch? | master || Bug fix? | no || New feature? | yes || BC breaks? | no || Deprecations? | no || Tests pass? | yes || License | MIT |Commits-------52606a1 [WebProfilerBundle] Make the IP address in the profiler header clickable to view requests by IP
Uh oh!
There was an error while loading.Please reload this page.