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

Tutorials: Suggestions on advanced/incipits tutorial #263

Open
mfeustle opened this issue Jul 22, 2021 · 0 comments · May be fixed by #645
Open

Tutorials: Suggestions on advanced/incipits tutorial #263

mfeustle opened this issue Jul 22, 2021 · 0 comments · May be fixed by #645

Comments

@mfeustle
Copy link
Contributor

mfeustle commented Jul 22, 2021

The current tutorial is a very effective summation of just about everything covered in the basic tutorials. For discoverability, I wonder if the title could also reflect that to those looking for practice solidifying what they have learned.

One small issue on the "first note" step is the presentation of the example with the closing </ tag. For consistency, it may be worth using the "usual" empty element tag.

I would suggest a few clarifications on the final step of the tutorial, which introduces slurs and the use of xml:id: When I first attempted the tutorial, I initially read the "1" in the "d1e" identifier as a lower-case "L." I imagine others have gotten tripped up there.

More guidance could also be present on the callouts of the xml:id identifiers in the final rendering of the slur [slur staff="1" curvedir="above" startid="#d1e4614" endid="#d1e4615"/]. It is not clear from the current accompanying text how or where to use the "#" symbol, and depends on the user's perseverance in trial and error perhaps more than we would like.

@musicEnfanthen musicEnfanthen changed the title Suggestions on advanced/incipits tutorial Tutorials: Suggestions on advanced/incipits tutorial Feb 11, 2022
@musicEnfanthen musicEnfanthen linked a pull request May 20, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants