Skip to content

Conversation

@wergilX
Copy link

@wergilX wergilX commented Oct 26, 2025

Describe your changes

Createed button in console for copy logs to clipboard

Issue ticket number and link if any

#2305

Checklist for finalizing the PR

  • I have performed a self-review of my code
  • I have added tests for new features and bugfixes
  • I have added documentation for new features
  • If it is a modifying the libf3d API, I have updated bindings
  • If it is a modifying the .github/workflows/versions.json, I have updated docker_timestamp

Continuous integration

Please write a comment to run CI, eg: \ci fast.
See here for more info.

{
std::string text;
clip::get_text(text);
return text.c_str();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it legal? the text variable is destroyed when exiting the function and the pointer isn't valid anymore.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For copy button we don't need this function, but i reworked anyway.

Copy link
Member

@Meakk Meakk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some remarks

@Meakk
Copy link
Member

Meakk commented Oct 26, 2025

Is it possible to make the logs selectable, so we can copy a part of the logs only by pressing CTRL+C?

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.

2 participants