Skip to content

Conversation

asherber
Copy link
Member

I basically hooked into the class that injects the HashURL so that it also injects RTFNotes. And the design is a little more modular now, so we can more easily inject other elements in the future, if needed.

I confirmed locally that dist files built in this branch differ from master only by the presence of RTFNotes (where appropriate) and the lack of whitespace mangling in plugindef.

I'm happy for any suggestions or questions.

@asherber
Copy link
Member Author

(I'll submit a separate PR for the library function I'm working on to display RTFNotes with the new text editor control.)

@asherber
Copy link
Member Author

Oh, I forgot to add pandoc to the GitHub runner. That'll be the next commit.

@rpatters1
Copy link
Collaborator

This PR can't be merged at the moment.

@asherber
Copy link
Member Author

Fixed

Copy link
Collaborator

@rpatters1 rpatters1 left a comment

Choose a reason for hiding this comment

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

looks good to me

@rpatters1 rpatters1 merged commit f636ff8 into finale-lua:master Jan 31, 2024
@asherber asherber deleted the generate-rtf-notes branch February 5, 2024 01:10
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