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

Copy Mod List as text #6577

Closed
baconpaul opened this issue Aug 27, 2022 · 1 comment · Fixed by #6773
Closed

Copy Mod List as text #6577

baconpaul opened this issue Aug 27, 2022 · 1 comment · Fixed by #6773
Labels
Accessibility Issues related to making Surge more accessible Feature Request New feature request
Milestone

Comments

@baconpaul
Copy link
Collaborator

Talking on accessibility heop this is a nice way to help colleagues debug patches

@baconpaul baconpaul added Feature Request New feature request Accessibility Issues related to making Surge more accessible labels Aug 27, 2022
@baconpaul baconpaul added this to the Surge XT 1.1.2 milestone Aug 27, 2022
@mkruselj mkruselj changed the title mod list copy as text Copy Mod List as text Sep 1, 2022
@baconpaul
Copy link
Collaborator Author

baconpaul commented Oct 7, 2022

I should have made it clearer sorry!
But here's the details, all in ModulationEditor.cpp
There's a class ModulationListContents which does al the filtering. It maintains at any moment std::vector dataRows and the Datum contains everything you need to build the list; It also as a DataRowEditor which turns it into strings and paints it

So the issue really is something like

  1. Add std::string datumToString(const Datum &d) and code it
  2. Add a 'copy' button to the side
  3. Do a
std::ostringstream oss
foreach( d : dataRows ) 
   oss << datumToString(d) << "\n";
juce::ClipboardGoobits::copy(oss.str());

baconpaul added a commit to baconpaul/surge that referenced this issue Jan 8, 2023
puts a human readable version of the modlist into the clipboard
for sharing with other humans.

Closes surge-synthesizer#6577
baconpaul added a commit that referenced this issue Jan 8, 2023
puts a human readable version of the modlist into the clipboard
for sharing with other humans.

Closes #6577
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility Issues related to making Surge more accessible Feature Request New feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants