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

Output extended #7420

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ChrisMarsh82
Copy link
Contributor

Updated output window.
Separated all logs into OutputGroup/OutputElemenet through the output logger
Create wrappers for these logs (PageGroup/PageElement) so they can be used in the output pages

@ChrisMarsh82
Copy link
Contributor Author

@Patowhiz here are the new changes to the output window we discussed. It is still very messy but should be able to get the idea.

@lloyddewit
Copy link
Contributor

@ChrisMarsh82 there are some merge conflicts. Is it still the intention to merge this PR? please could you clarify the status? thanks

@ChrisMarsh82
Copy link
Contributor Author

@lloyddewit This is only a draft. I only committed to show Patrick the changes I was working on. I will make sure all conflicts are resolved before creating a pull request

@rdstern
Copy link
Collaborator

rdstern commented Jun 1, 2023

@ChrisMarsh82 should this be closed?

@rdstern
Copy link
Collaborator

rdstern commented Jun 15, 2024

@ChrisMarsh82 what would you like for this one? Should it just wait here - it is doing no harm, or should someone take over, or should we close it?

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

Successfully merging this pull request may close these issues.

3 participants