Problem/Motivation

The Paragraphs UI sometimes writes "paragraph" / "paragraphs" and sometimes "Paragraph" / "Paragraphs".

We should be clear that we mean our concept of "Paragraphs" - not the linguistic definition of a paragraph.
Some people already complained that Paragraphs should be renamed...

Proposed resolution

Captitalise all use of the concept "Paragraph" / "Paragraphs" in the UI.

(Also that said, end users still can experience a reconfigured Paragraph that hides the term completely...)

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#8 capitalise-paragraph-text-in-the-ui-2901996-8.patch36.3 KBkarthikkumarbodu
#6 capitalise-paragraph-text-in-the-ui-2901996-6.patch35.7 KBkarthikkumarbodu
#3 capitalise-paragraph-text-in-the-ui-2901996-3.patch36.95 KBkarthikkumarbodu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

miro_dietiker created an issue. See original summary.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Issue tags: +Novice

Promoting for novices.

Lots of replaces though. :-)

karthikkumarbodu’s picture

karthikkumarbodu CreditAttribution: karthikkumarbodu at Axelerant commented
FileSize
capitalise-paragraph-text-in-the-ui-2901996-3.patch36.95 KB

Replaced "paragraph" / "paragraphs" with "Paragraph" / "Paragraphs" in the UI and in some comment sections.

karthikkumarbodu’s picture

karthikkumarbodu CreditAttribution: karthikkumarbodu at Axelerant commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 3: capitalise-paragraph-text-in-the-ui-2901996-3.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

karthikkumarbodu’s picture

karthikkumarbodu CreditAttribution: karthikkumarbodu at Axelerant commented
Assigned: Unassigned » karthikkumarbodu
Status: Needs work » Needs review
FileSize
capitalise-paragraph-text-in-the-ui-2901996-6.patch35.7 KB

Fixed failed tests

Status: Needs review » Needs work

The last submitted patch, 6: capitalise-paragraph-text-in-the-ui-2901996-6.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

karthikkumarbodu’s picture

karthikkumarbodu CreditAttribution: karthikkumarbodu at Axelerant commented
Status: Needs work » Needs review
FileSize
capitalise-paragraph-text-in-the-ui-2901996-8.patch36.3 KB

Fixed failed test case.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Needs review » Fixed

Reads nice. Great consistency improvement.

karthikkumarbodu’s picture

karthikkumarbodu CreditAttribution: karthikkumarbodu at Axelerant commented

@miro_dietiker Thanks for reviewing the patch.

Status: Fixed » Closed (fixed)

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