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

fix: Propagate Djot attributes to the blockquote implementation #139

Merged
merged 1 commit into from
Dec 30, 2024

Conversation

Omikhleia
Copy link
Owner

So the latter might implement a different behavior depending on them. As of now, SILE's default blockquote (from SILE 0.15.0) and resilient's style-aware blockquote don't support options, so this would not do anything yet, but it paves the way for more customization or alternate implementations.
N.B. It's also what we were already doing for the captioned quotes (via the epigraph environment, which does support several options), so it also fixes a small API discrepancy...

Rationale:
See discussion #138

So the latter might implement a different behavior depending on them.
As of now, SILE's default blockquote (from SILE 0.15.0) and resilient's
style-aware blockquote don't support options, so this would not do anything
yet, but it paves the way for more customization or alternate
implementations.
N.B. It's also what we were already doing for the captioned quotes
(via the epigraph environment, which does support several options), so
it also fixes a small API discrepancy...
@Omikhleia Omikhleia self-assigned this Dec 30, 2024
@Omikhleia Omikhleia added the bug Something isn't working label Dec 30, 2024
@Omikhleia Omikhleia added this to the 2.3.1 milestone Dec 30, 2024
@Omikhleia
Copy link
Owner Author

Merging, so as to experiment it with the --dev rock.

@Omikhleia Omikhleia merged commit a9a6839 into main Dec 30, 2024
2 checks passed
@Omikhleia Omikhleia deleted the fix-propagate-attr-blockquote branch December 30, 2024 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant