[sims1253]

• length(buttons > 1) should be length(buttons) > 1; as written it is always >= 1 (truthy), so the else-branch is dead and a single-button call prints "the NA button to quit".
• In the dead else-branch, buttons[1] is a second argument to cat, not to gettextf, so even when fixed it would print the literal %s.
• After pmatch, embeddings is integer; 2L == "modify" coerces to "2" == "modify", always FALSE, so the prohibition on a "modify" mouse embedding is never enforced.

Found with https://github.com/sims1253/ry :)

[dmurdoch]

Thanks, those all look like good fixes. I'll accept after testing...