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

Kill live log feature in log window #6475

Closed
guruz opened this issue Apr 19, 2018 · 8 comments
Closed

Kill live log feature in log window #6475

guruz opened this issue Apr 19, 2018 · 8 comments
Assignees
Labels
Enhancement ReadyToTest QA, please validate the fix/enhancement
Milestone

Comments

@guruz
Copy link
Contributor

guruz commented Apr 19, 2018

#6435 (comment)

And yeah, the log window isn't very functional. Personally, I'd drop the live view and just keep the "capture logs" checkbox along with a button that can open the folder that contains the files.

@guruz guruz added this to the 2.6.0 milestone Apr 19, 2018
@ckamm
Copy link
Contributor

ckamm commented Apr 20, 2018

Thanks!

@ogoffart
Copy link
Contributor

ogoffart commented Dec 4, 2018

I think the log window is fine, this is supposed to be a debug only feature. Normal users should not open it.

@ckamm
Copy link
Contributor

ckamm commented Dec 17, 2018

I agree it's a debug feature. I think the live view and search is very had to use for debugging and we'd be better off dropping it and encouraging users to use log files. (via the permanent logging feature)

@michaelstingl
Copy link
Contributor

Bring the settings for permanent logging to the general settings? (Then kill the window)

@guruz
Copy link
Contributor Author

guruz commented Dec 17, 2018

Bring the settings for permanent logging to the general settings?

but please only behind the extended settings checkbox..

@guruz
Copy link
Contributor Author

guruz commented Dec 17, 2018

Hm no. This might bloat the dialog. Let's keep it the separate window but one could add a button to open the window behind the extended settings checkbox.

@guruz
Copy link
Contributor Author

guruz commented Dec 17, 2018

And yes, kill the live view.

@michaelstingl
Copy link
Contributor

@ChrisEdS suggested:

I desire an additional button that takes me directly to the appropriate folder. Let's say it's called "View" or "Saved Logs".

image

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement ReadyToTest QA, please validate the fix/enhancement
Projects
None yet
Development

No branches or pull requests

4 participants