Problem/Motivation

  1. Minor typo issues or language quality improvements
  2. In most places in the code, it's stated Configuration Split Setting, while it's not recommended to start the word setting(s) with an uppercase since it's not part of the module name nor is it consistent in how other modules handle menu link names (see screenshot below).

Configuration Split Settings

Proposed resolution

  1. Walk through entire module/code to check language.
  2. Find and replace 'Configuration Split Setting' to 'Configuration Split setting'.
CommentFileSizeAuthor
#4 interdiff.txt253 bytesnovitsh
#4 general_typo_issues-3003865-4.patch8.04 KBnovitsh
#3 general_typo_issues-3003865-3.patch7.99 KBnovitsh
config split screencap.PNG18.62 KBnovitsh

Comments

Novitsh created an issue. See original summary.

novitsh’s picture

novitsh commented
Issue summary: View changes
novitsh’s picture

novitsh commented
Status: Active » Needs review
StatusFileSize
new general_typo_issues-3003865-3.patch7.99 KB

I think this patch covers it all?

novitsh’s picture

novitsh commented
StatusFileSize
new general_typo_issues-3003865-4.patch8.04 KB
new interdiff.txt253 bytes

Was missing a full stop at the end of a sentence. Added this now. Interdiff attached.

alonaoneill’s picture

alonaoneill commented
Status: Needs review » Reviewed & tested by the community

Looks good to me. All typos and Capitalizations are fixed.
Marking RTBCed
Thanks for working on the module!

  • bircher committed d18eace on 8.x-1.x 
    Issue #3003865 by Novitsh, alonaoneill: General typo issues
bircher’s picture

bircher
Location 🇨🇿
commented
Status: Reviewed & tested by the community » Fixed

Thanks for the contribution.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.