Skip to content

Conversation

rpspringuel
Copy link
Contributor

This is not ready for merge, but I wanted to put it out there for comment at this point.

This is not ready for merge, but I wanted to put it out there for comment at this point.
@rpspringuel rpspringuel mentioned this pull request Mar 23, 2018
end tell

try
set output to do shell script ("/bin/echo \"" & input & "\" | ~/scripts/syllabify")
Copy link
Contributor

Choose a reason for hiding this comment

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

there are a few invalid html chars here, & should be remplaced by &

@eroux
Copy link
Contributor

eroux commented Apr 14, 2018

this is an excellent idea! It doesn't look very simple but I guess with the TeXShop system it can't really be...

@rpspringuel
Copy link
Contributor Author

It's actually not so much TeXShop that is the problem, but the restrictions that Apple in general and AppleScript in particular places on GUI apps. These are for the purpose of "security" but they're damned inconvenient when it comes to customizing apps.

@rpspringuel rpspringuel changed the title First draft Syllabifier Integration into TeXShop Apr 18, 2018
@rpspringuel
Copy link
Contributor Author

This now incorporates #72 into it and thus should be merged only after that one.

@rpspringuel
Copy link
Contributor Author

I also want to look into doing something similar in TeXworks, but that will have to wait for the summer. It's exam crunch time now and I don't have the spare time to research its macro extension language.

@rpspringuel
Copy link
Contributor Author

Okay, it's been over a year since I touched this. Scripting in TeXworks turns out to be fairly different from scripting in TeXShop so I'm probably not going to adapt this to TeXworks anytime soon. As a result, I'm going to say this should be merged as is.

@henryso henryso merged commit 1e11cf3 into gregorio-project:master Oct 31, 2019
@rpspringuel rpspringuel deleted the syllabifier branch February 29, 2020 20:09
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