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

Support of echo with chords inside #205

Merged
merged 1 commit into from Feb 22, 2016
Merged

Support of echo with chords inside #205

merged 1 commit into from Feb 22, 2016

Conversation

oliverpool
Copy link
Contributor

Je risque d'avoir besoin d'un support de \echo{} dans des fichiers chordpro pour un projet perso, voici donc une PR pour proposer une implémentation.

Il faut notamment permettre les accords au sein de ces echo.

Je propose la paire {start_echo}, {end_echo} sans le of, car ce ne sont pas des paragraphes (comme chorus / bridge).

Qu'en pensez-vous ?
Est-ce que la syntaxe vous convient ? Est-ce que l'implémentation semble correcte ?

@paternal
Copy link
Contributor

Du coup, ça ferme #119, non ?

Ça me parait bien. Qu'est-ce que ça donne si les {se}…{ee} ne sont pas sur la même ligne ? L'erreur est compréhensible ?

@oliverpool
Copy link
Contributor Author

Du coup, ça ferme #119, non ?

Je savais bien qu'on en avait déjà discuté: oui, c'est parfaitement cela !

Qu'est-ce que ça donne si les {se}…{ee} ne sont pas sur la même ligne ? L'erreur est compréhensible ?

C'est une erreur fatale (comme un {end_of_chorus} absent).

Je pense qu'il y a des améliorations plus génériques à faire sur lorsque des erreurs sont détectée (plutôt dans une autre issue, je pense)

paternal added a commit that referenced this pull request Feb 22, 2016
Support of echo with chords inside

Closes #119.
@paternal paternal merged commit d03b42b into master Feb 22, 2016
@paternal paternal deleted the echo_support branch February 22, 2016 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants