Closed (fixed)
Project:
User Guide
Version:
8.x-6.0
Component:
User Guide content
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
7 Mar 2019 at 14:30 UTC
Updated:
10 May 2019 at 08:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
jhodgdonThanks for this issue too! We'll get it fixed soon.
Comment #3
jhodgdonHow's this patch?
Comment #4
gillesbailleux@jhodgdon: because I am being asked to become Acquia Certified Site Builder, I have to read the Drupal 8 User Guide. It is a true pleasure to take time to discover the work of the Documentation team, either for its content but also for the challenge that it represents: writing technological guidelines for an audience which is presumed to be non technical is rather a matter of communication than a question of technical expertise. The Documentation team has fortunatley members with both qualities, for instance Boris Doesborg (whom I have not seen for ages) who has achieved a tremendous work.
Back to this issue: because the tags are automatically added, the use of the future sense is preferred to the conditional tense. I suggest the following sentence: "adding its editor button will automatically update the allowed tags".
Comment #5
jhodgdonGood idea, thanks! Here's a new patch. I didn't make an interdiff, as it's a 2-line patch anyway.
Comment #6
eojthebraveLooks good to me. Thanks @gillesbailleux for point out the issue, and @jhodgdon for helping to get it fixed.
I'll commit this now. And, we'll want to notify the translation teams about this change.
Comment #17
jhodgdonI'm sending information to the group email list now.
Comment #18
gillesbailleux@jhodgdon: just a word to tell you that I passed the certification and that I shall continue to contribute to the User Guide.