Skip to content

Conversation

@Goober5000
Copy link
Contributor

This adds an option to the sound handling so that an in-mission looping sound is played for as long as the text is scrolling, and then stops when the text finishes scrolling. Right now beep_5.wav is used as a placeholder sound effect; textdraw.wav could also be used, but it would have to be copied into the in-mission sound section in sounds.tbl.

Also adds an important detail to sexp help, removes trailing whitespace in the script, and fixes tab-indentation.

(Unfortunately the whitespace changes in Scroll:Draw make it appear that there are more changes than there really are. The main non-whitespace changes are the addition of the line.Handle management and the removal of the finalString variable.)

This corresponds to the old PR 3:
AxemP/AxemFS2Scripts#3

This adds an option to the sound handling so that an in-mission looping sound is played for as long as the text is scrolling, and then stops when the text finishes scrolling.  Right now beep_5.wav is used as a placeholder sound effect; textdraw.wav could also be used, but it would have to be copied into the in-mission sound section in sounds.tbl.

Also adds an important detail to sexp help, removes trailing whitespace in the script, and fixes tab-indentation.

(Unfortunately the whitespace changes in `Scroll:Draw` make it appear that there are more changes than there really are.  The main non-whitespace changes are the addition of the `line.Handle` management and the removal of the `finalString` variable.)

This corresponds to the old PR 3:
AxemP/AxemFS2Scripts#3
@Goober5000 Goober5000 added documentation Improvements or additions to documentation enhancement New feature or request labels Jun 22, 2022
@BMagnu BMagnu merged commit e58f053 into FSO-Scripters:master Jul 7, 2022
@Goober5000 Goober5000 deleted the scrollwrite_sounds branch August 18, 2022 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants