Skip to content
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

Timestamps #14

Open
ErikKrause opened this issue Nov 7, 2017 · 5 comments
Open

Timestamps #14

ErikKrause opened this issue Nov 7, 2017 · 5 comments

Comments

@ErikKrause
Copy link

Timestamps with milliseconds, like in the devtools console, would be nice.

@cadorn
Copy link
Member

cadorn commented Nov 10, 2017

I don't see them. Screenshot? I presume they should show the server time?

@ErikKrause
Copy link
Author

ErikKrause commented Nov 10, 2017

zwischenablage01
You must disable devtools.webconsole.new-frontend-enabled in FF 57 in order to see this. The output is by my chromelogger fork.

@cadorn
Copy link
Member

cadorn commented Nov 11, 2017

  1. Is the time showing the server time?
  2. What timezone is the time? Server timezone or local desktop? We can make it configurable but would be good to use same default.

@ErikKrause
Copy link
Author

I guess this is a function the devtools console provides, since it is also displayed for CSS, JS etc. events. Hence I assume it is local time. But server time would be better of course, just to see timings between subsequent calls.

@cadorn
Copy link
Member

cadorn commented Nov 12, 2017

Yeah, I agree, server time would be better. I don't see client time being useful as all log times would be equal to time of request.

I filed an issue for FirePHPCore here: firephp/firephp-core#46

I see two views for the console once we have server timing info available:

  1. Grouped by request/sub-request and then sorted by index (as is the case now)
  2. Sorted by time across all requests so you can see timing of server events as different requests were processed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants