Fixes for 7.2 #667
Open
Fixes for 7.2 #667
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was
linked to
issues
Jan 15, 2026
daobrien
requested changes
Jan 15, 2026
Collaborator
daobrien
left a comment
daobrien
left a comment
There was a problem hiding this comment.
Mostly ok but a few things need updating.
| <listitem> | ||
| <para> | ||
| DRaaS (Disaster Recovery-as-a-Service) | ||
| AIaaS (Artificial Intelligence-as-a-Service) |
Collaborator
There was a problem hiding this comment.
This actually contradicts the previous para: "When spelling it out, do not capitalize the term". You might like to update that para to include "unless part of a proper noun" or similar.
We also do not need all these examples of aaS terms. We need basic guidance and a list of any exceptions, and that's all. Otherwise this list is going to get very long over time.
en-US/A.xml
Outdated
| <listitem> | ||
| <para> | ||
| Avoid use of an acronym if it could stand for more than one term in a single asset. for example, if you are writing content that discusses both Cloud-as-a-Service and Containers-as-a-Service. | ||
| Avoid using an acronym if it could stand for more than one term in a single asset, for example, if you are writing content that discusses both Cloud-as-a-Service and Containers-as-a-Service. |
Collaborator
There was a problem hiding this comment.
Suggested change
| Avoid using an acronym if it could stand for more than one term in a single asset, for example, if you are writing content that discusses both Cloud-as-a-Service and Containers-as-a-Service. | |
| Avoid using an acronym if it could stand for more than one term in a single asset. For example, if you are writing content that discusses both Cloud-as-a-Service and Containers-as-a-Service. |
The original version was more correct IMO.
en-US/Design.xml
Outdated
| <title>Writing Effective Titles</title> | ||
| <para> | ||
| Use a title that represents the content. | ||
| Use a title that represents the content. |
Collaborator
There was a problem hiding this comment.
Personally I prefer "Use titles that represent..." but that's just me.
en-US/Design.xml
Outdated
| </para> | ||
| <para> | ||
| Avoid a title that consists of only one word. | ||
| Avoid a title that consists of only one word. |
en-US/Design.xml
Outdated
| <para> | ||
| Do not use manual line indentation for the second and subsequent lines of commands. | ||
| For some output formats, no control is possible over where lines would break, and any indentation might appear as extra spaces in inelegant places. | ||
| For some output formats, no control is possible over where lines would break, and any indentation might appear as extra spaces in inelegant places. |
en-US/Design.xml
Outdated
| <title>Omitting Part of Output</title> | ||
| <para> | ||
| For the sake of brevity, do not show all output to the user in all cases, but only those parts of any output that are relevant to the context that is described. | ||
| For the sake of brevity, do not show all output to the user in all cases, but only those parts of any output that are relevant to the context that is described. |
Collaborator
There was a problem hiding this comment.
Suggested change
| For the sake of brevity, do not show all output to the user in all cases, but only those parts of any output that are relevant to the context that is described. | |
| For the sake of brevity, do not show all output to the user in all cases. | |
| Instead, show only those parts of any output that are relevant to the context that is described. |
en-US/Language.xml
Outdated
| <row> | ||
| <entry> The system menu allows you to switch network and VPN connections on or off. </entry> | ||
| <entry> From the system menu, you can switch network and VPN connections on or off. </entry> | ||
| <entry> The <guibutton>system</guibutton> menu allows you to switch network and VPN connections on or off. </entry> |
Collaborator
There was a problem hiding this comment.
Shouldn't you be using <guimenu>?
en-US/Language.xml
Outdated
| <entry> The system menu allows you to switch network and VPN connections on or off. </entry> | ||
| <entry> From the system menu, you can switch network and VPN connections on or off. </entry> | ||
| <entry> The <guibutton>system</guibutton> menu allows you to switch network and VPN connections on or off. </entry> | ||
| <entry> From the <guibutton>system</guibutton> menu, you can switch network and VPN connections on or off. </entry> |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.