Spinning this off from #1167350: Action links were ignored , quoting comment #1:

Help section: Participants missed the links and focused their attention on the busy information part (bottom half) of the page, leaving them overwhelmed.

I think it would help to have a heading above the first section. As it is now, the eye catches the big bold 'Help topics' heading and scans down from there. Patch forthcoming…

Files: 
CommentFileSizeAuthor
#20 1387964.png137.45 KBmohangupta
#19 People-miss-the-first-section-of-the-help-overview-page-issue-number-1387964-19.patch1.26 KBgalooph
PASSED: [[SimpleTest]]: [MySQL] 58,837 pass(es).
[ View ]
#19 1387964.png106.28 KBgalooph
#17 issue-1387964-before.png134.99 KBalexanansi
#17 issue-1387964-after.png84.04 KBalexanansi
#16 People-miss-the-first-section-of-the-help-overview-page-issue-number-1387964-16.patch790 bytesrealityloop
PASSED: [[SimpleTest]]: [MySQL] 57,779 pass(es).
[ View ]
#9 People-miss-the-first-section-of-the-help-overview-page-issue-number-1387964.patch1.91 KBthemic8
FAILED: [[SimpleTest]]: [MySQL] 57,261 pass(es), 1 fail(s), and 0 exception(s).
[ View ]
#5 1387964-help-page.patch2.61 KByoroy
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1387964-help-page_0.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#1 help-page-before.png142.98 KByoroy
#1 Help-page-after.png141.77 KByoroy
#1 1387964-help-page.patch43.22 KByoroy
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1387964-help-page.patch. This may be a -p0 (old style) patch, which is no longer supported by the testbots.
[ View ]

Comments

yoroy’s picture

Status:Active» Needs review
StatusFileSize
new43.22 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1387964-help-page.patch. This may be a -p0 (old style) patch, which is no longer supported by the testbots.
[ View ]
new141.77 KB
new142.98 KB

Text changes:
The paragraph "Follow these steps to set up and start using your website:" is replaced with a h2 "Getting started".
The paragraph "Help is available on the following items:" below the h2 "Help topics" is removed.

Before:
screenshot help start pageAfter:
screenshot help start page with patch applied

yoroy’s picture

Grr, wrong patch, ignore the one above.

Status:Needs review» Needs work

The last submitted patch, 1387964-help-page.patch, failed testing.

yoroy’s picture

Status:Needs work» Active
yoroy’s picture

Status:Active» Needs review
StatusFileSize
new2.61 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 1387964-help-page_0.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Maybe this one…

kscheirer’s picture

#5: 1387964-help-page.patch queued for re-testing.

Status:Needs review» Needs work

The last submitted patch, 1387964-help-page.patch, failed testing.

themic8’s picture

Assigned:Unassigned» themic8
themic8’s picture

Assigned:themic8» Unassigned
Status:Needs work» Needs review
StatusFileSize
new1.91 KB
FAILED: [[SimpleTest]]: [MySQL] 57,261 pass(es), 1 fail(s), and 0 exception(s).
[ View ]

Text changes:
The paragraph "Follow these steps to set up and start using your website:" is replaced with a h2 "Getting started".
The paragraph "Help is available on the following items:" below the h2 "Help topics" is removed.

Status:Needs review» Needs work
themic8’s picture

Status:Needs work» Needs review

Status:Needs review» Needs work
themic8’s picture

Status:Needs work» Needs review

Status:Needs review» Needs work
themic8’s picture

Test Fails due to this change request: The paragraph "Help is available on the following items:" below the h2 "Help topics" is removed. The test is looking for Help is available on the following items and it has been removed.

realityloop’s picture

Status:Needs work» Needs review
StatusFileSize
new790 bytes
PASSED: [[SimpleTest]]: [MySQL] 57,779 pass(es).
[ View ]

rerolled, although the heading isn't displaying for me on local dev, test pass so it would be great to get some eyes on this.

alexanansi’s picture

StatusFileSize
new84.04 KB
new134.99 KB

My first contribution to testing :) I tested @realityloop's patch and the whole paragraph disappeared for me, instead of the header changing, see the attached images

realityloop’s picture

Status:Needs review» Needs work

That look like it needs work then

galooph’s picture

Status:Needs work» Needs review
StatusFileSize
new106.28 KB
new1.26 KB
PASSED: [[SimpleTest]]: [MySQL] 58,837 pass(es).
[ View ]

Added a missing '.'

$output = '<p>' . t('Follow these steps to set up and start using your website:') . '</p>';
becomes
$output .= '<p>' . t('Follow these steps to set up and start using your website:') . '</p>';

mohangupta’s picture

Status:Needs review» Reviewed & tested by the community
StatusFileSize
new137.45 KB

I have tested this. See attached screenshot

realityloop’s picture

I also checked that help tests are still passing this is ready to commit

lostkangaroo’s picture

alexpott’s picture

Status:Reviewed & tested by the community» Fixed

Committed 1146ebd and pushed to 8.x. Thanks!

Status:Fixed» Closed (fixed)

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