While evaluating this module I noticed a few grammar issues. Also, the language could be more inclusive.
click => select ... Not all users can "click" .

Patch to come, thanks!!

CommentFileSizeAuthor
#4 quick_node_block-readme-3030337-4.patch2.26 KBsdxt
#3 review-typo.png171.07 KBsdxt
#2 quick_node_block-readme-3030337-2.patch2.27 KBvolkswagenchick

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Assigned: volkswagenchick » Unassigned
Status: Active » Needs review
StatusFileSize
new quick_node_block-readme-3030337-2.patch2.27 KB

I uploaded a patch that corrects the grammar and changed some of the languages to be more inclusive.
I also took the time to reformat the file to meet d.o documentation standards.

sdxt’s picture

sdxt commented
Assigned: Unassigned » sdxt
Status: Needs review » Needs work
StatusFileSize
new review-typo.png171.07 KB

Patch applies cleanly and looks good but I noticed one typo in the patch, please find attached screenshot for the same.
I will be adding patch for that.

sdxt’s picture

sdxt commented
Issue summary: View changes
Status: Needs work » Needs review
StatusFileSize
new quick_node_block-readme-3030337-4.patch2.26 KB

Here is the updated patch.

sdxt’s picture

sdxt commented
Assigned: sdxt » Unassigned

  • saesa committed c85e038 on 8.x-1.x 
    Issue #3030337 by sdxt, volkswagenchick: Readme - typos and inclusive...
saesa’s picture

saesa commented
Status: Needs review » Fixed

Uploaded the patch to version 8-x-1.x with some changes. Thanks.

Status: Fixed » Closed (fixed)

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