You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It is a bit confusing to have the text-direction field as it is not needed if the language field is set to one of the RTL languages.
It seems that the text-direction is now derived from the language field of the book section of the configuration.
According to the list of public mdbooks there is only one book using the separate text-direction field. I asked them to verify that the language-field is enough for them. persian-rust/book#6
Of course there are probably many other RTL books out there, some public some private.
Proposed Solution
I'd suggest to add a warning to mdbook if it encounters the use of the text-direction field. In the report mention that the language field should be enough and ask them to report here if for some reason they need the text-direction field.
Then, if there is no objection, a few months later remove this field from mdbook.
Notes
Current version mdbook v0.4.48
The text was updated successfully, but these errors were encountered:
Problem
It is a bit confusing to have the
text-direction
field as it is not needed if thelanguage
field is set to one of the RTL languages.It seems that the text-direction is now derived from the
language
field of thebook
section of the configuration.According to the list of public mdbooks there is only one book using the separate
text-direction
field. I asked them to verify that the language-field is enough for them. persian-rust/book#6Of course there are probably many other RTL books out there, some public some private.
Proposed Solution
I'd suggest to add a warning to
mdbook
if it encounters the use of thetext-direction
field. In the report mention that thelanguage
field should be enough and ask them to report here if for some reason they need thetext-direction
field.Then, if there is no objection, a few months later remove this field from
mdbook
.Notes
Current version
mdbook v0.4.48
The text was updated successfully, but these errors were encountered: