Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
API page: https://api.drupal.org/api/drupal/core%21includes%21theme.inc/group/cont...
Enter a descriptive title (above) relating to Content markers, then describe the problem you have found:
This topic mentions theme_mark(). This does not exist in Drupal 8. It should instead refer to mark.html.twig.
Comment | File | Size | Author |
---|---|---|---|
#6 | 2296775-content-markers_1.patch | 468 bytes | gauravkhambhala |
#4 | 2296775-content-markers.patch | 463 bytes | gauravkhambhala |
Comments
Comment #1
gauravkhambhala CreditAttribution: gauravkhambhala commentedComment #2
gauravkhambhala CreditAttribution: gauravkhambhala commented@jhodgdon Could you point out how to refer mark.html.twig in the documentation?
Comment #3
jhodgdonYes: Just put the file name into the documentation, such as:
This something something uses the mark.html.twig file something something.
Comment #4
gauravkhambhala CreditAttribution: gauravkhambhala commentedThank you @jhodgdon for the tips. Added @see reference as well. Here is the patch.
Comment #5
jhodgdonOK... We probably don't really need the @see lines, because both the function and the Twig file are mentioned in the previous line?
If you feel the @see lines are a good idea, there needs to be a space between the text and those @see lines.
So, either add a blank line or just take them out... your choice, either way is fine! Thanks!
Comment #6
gauravkhambhala CreditAttribution: gauravkhambhala commentedAdded line in between. Please review.
Comment #7
jhodgdonLooks good to me, thanks!
Comment #8
alexpottCommitted 6f9cead and pushed to 8.0.x. Thanks!