"Umbraco" should not be lowercased

[iOvergaard]

In the text "Anonymized site ID, umbraco version, and packages installed." it should be "Umbraco".

Please check that Umbraco is written with a capital U throughout all the translation files and keys.

Originally posted by @bjarnef in https://github.com/umbraco/Umbraco-CMS/issues/12739#issuecomment-1202370184

[bjarnef]

Another minor thing is that we have this text:

<i>We might change what we send on the Detailed level in the future. If so, it will be listed above.
<br>By choosing "Detailed" you agree to current and future anonymized information being collected.</i>

The <i> has no semantic meaning so general I think it would be better to use <em> and <strong> in favor of <i> and <b>. The <i> element has historically been used for font icons (probably because of the i).

so I would change that text to this instead:

<em>We might change what we send on the Detailed level in the future. If so, it will be listed above.
<br>By choosing "Detailed" you agree to current and future anonymized information being collected.</em>

[MayurDeore]

I'm working on this.

[bjarnef]

@MayurDeore the changes hasn't been merged into v10/contrib branch yet, so maybe it should be merged first to adjust this.

[MayurDeore]

@bjarnef yes I hadn`t checked that before, thanks for letting me know.

[iOvergaard]

@MayurDeore > Thank you for your interest. The relevant code has now been merged with the main branch making this issue relevant to fix. Are you still up for it?

[bjarnef]

@iOvergaard it still seem the recent changes in v10/dev not has been merged to v10/contrib branch, e.g. changes in "detailedLevelDescription" from your PR.

https://github.com/umbraco/Umbraco-CMS/blob/v10/dev/src/Umbraco.Core/EmbeddedResources/Lang/en_us.xml

https://github.com/umbraco/Umbraco-CMS/blob/v10/contrib/src/Umbraco.Core/EmbeddedResources/Lang/en_us.xml

[MayurDeore]

@iOvergaard Yes, I'm eager to get started. But not everything has merged yet, according to @bjarnef comment.

[nul800sebastiaan]

I've just done the merge now! 👍

[MayurDeore]

Great, I'm going to start working on this.

[MayurDeore]

@bjarnef Can I also modify the other translation files that contain the <i> and <b> tags?

Another minor thing is that we have this text:

<i>We might change what we send on the Detailed level in the future. If so, it will be listed above.
<br>By choosing "Detailed" you agree to current and future anonymized information being collected.</i>

The <i> has no semantic meaning so general I think it would be better to use \<em\> and \<strong\> in favor of <i> and <b>. The <i> element has historically been used for font icons (probably because of the i).

so I would change that text to this instead:

<em>We might change what we send on the Detailed level in the future. If so, it will be listed above.
<br>By choosing "Detailed" you agree to current and future anonymized information being collected.</em>

[bjarnef]

@MayurDeore yes, I think that would be fine.