Page MenuHome

Status bar: Click on report message to open Info Editor window
ClosedPublic

Authored by Valentin (Poulpator) on Tue, Aug 13, 2:29 AM.

Details

Summary

This patch add the ability to click on a report message to show the Info Editor window as described in T68448

Diff Detail

Repository
rB Blender

Event Timeline

Great!

However, we cannot commit this until the Info Editor actually displays the notifications. Otherwise it's rather pointless. But we can keep this handy when we get to that point.

I think the info editor already displays all reports that appear in the status bar?

I think the info editor already displays all reports that appear in the status bar?

You are right:

Ok then, in that case this could be added then I guess - although it'd be nicer if it was easier to hide the very loud operator list.

The implementation looks good to me.

The only nitpick I have is about the name "Info Editor", which we don't use in the UI currently. It's just "Info" in the menu, and "Info Editor" is a bit strange since you are not actually editing the information.

Maybe "Info Log" is better?

Brecht Van Lommel (brecht) requested changes to this revision.Wed, Aug 14, 5:07 PM

Marking as request changes for the naming.

It would be good to support showing just the reports, but would not consider it a blocker. The operator log also provides a bit of context to the reports that can be helpful.

This revision now requires changes to proceed.Wed, Aug 14, 5:07 PM

Change name to "Blender Info Log"

When filter by report type would be implement in Info Editor, we could change the context of the "Info Log" to the specific report type.

I'd change "info editor" to "info log" everywhere, not just for the window title.

Brecht Van Lommel (brecht) requested changes to this revision.Wed, Aug 14, 6:22 PM
This revision now requires changes to proceed.Wed, Aug 14, 6:22 PM

Sorry didn't understand in the first place, the diff is now updated, let me know if there is something else.

This revision is now accepted and ready to land.Fri, Aug 16, 2:12 PM