Extend toolbar with POST 302 data about yii2-debug HOT 18 OPEN

There's already a dropdown:

from yii2-debug.

yes, I am familiar with that downdown, but a few clicks are needed to get there, at most of the time 4 clicks

1. click on the toolbar at the bottom
2. click at the profile menu
3. click at last 10
4. find and click at the POST

When debugging a POST, preferably, I would not have to click at all but see immediately if something was wrong.

from yii2-debug.

Well, it was wrong not in the current request.

from yii2-debug.

I am not saying the toolbar should be removed for the current request,
I am advocating to improve visibility for non-visible requests (redirects), which I do think makes sense.

from yii2-debug.

What about AJAX requests? These are non visible as well. And there could be 100 requests while the main one is active.

from yii2-debug.

reporting these could be handy as well, but in a separate solution. 100 rows would now be practical as you pointed, a dropdown would fit better.

About non-ajax redirects (which will usually will be 1) what do you think about adding an extra row in de toolbar? It also improves visibility on the fact that a REDIRECT has occurred.

from yii2-debug.

OK. It makes sense overall. So:

1. If current request was redirect, display an extra badge colored in response status of the original request.
2. When clicked, it will lead you to previous request opened in full debugger.

Correct?

from yii2-debug.

1. could be an extra badge, but I see more value in showing a complete row with the same data as the current GET request:

• less confusing as 1 row corresponds with 1 request (better from a conceptual perspective)
• more valuable information for the redirect (such as time/db/memory etc)
• IMHO we can live with an extra 40px

2. correct

from yii2-debug.

@samdark: why not implement both. Than the user can decide what to use.

from yii2-debug.

I have another usecase. I was doing some refactoring in some of my own core components and after a while I came to the conclusion that my POSTs got 5 X slower.

I can not easily be notified about this with the current toolbar. I could have when supplied with another row about the POST request, supplying me with figures about memory & total time.

from yii2-debug.

This pic is to show you my intentions. The summary bar of the previous request is only shown when 302 POSTs occur.

from yii2-debug.

@dynasource affirmative. Intention understood. Overall it should be useful.

from yii2-debug.

from yii2-debug.

• if I understand your pic correctly, I should hide double stuff like the yii logo? Will do that with opacity:0 ok?

from yii2-debug.

Well, it doesn't make sense to display it twice. Not sure how to handle it in the best way though...

from yii2-debug.

the current output is like this:

I am using this for a while now and I am very happy with it.
Is it valuable enough to implement? If not, please tell me, than I will start to implement my own version/hook.

from yii2-debug.

Yes, I think I'll merge it. Still not sure about visual part of it. It looks a bit misleading so probably we need a way to hint that it was a redirect and that's why there are 2 rows. Something like the following:

from yii2-debug.

true, despite the double logos etc, I liked #38 (comment) better because of design consistency.

With respect to the hint for the redirect, its quite clear with the status 302. But you have to get used to it in the beginning, thats true.

from yii2-debug.