Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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…
Comment | File | Size | Author |
---|---|---|---|
#20 | 1387964.png | 137.45 KB | mohangupta |
#19 | People-miss-the-first-section-of-the-help-overview-page-issue-number-1387964-19.patch | 1.26 KB | galooph |
#19 | 1387964.png | 106.28 KB | galooph |
#17 | issue-1387964-before.png | 134.99 KB | alexanansi |
#17 | issue-1387964-after.png | 84.04 KB | alexanansi |
Comments
Comment #1
yoroy CreditAttribution: yoroy commentedText 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:
After:
Comment #2
yoroy CreditAttribution: yoroy commentedGrr, wrong patch, ignore the one above.
Comment #4
yoroy CreditAttribution: yoroy commentedComment #5
yoroy CreditAttribution: yoroy commentedMaybe this one…
Comment #6
kscheirer#5: 1387964-help-page.patch queued for re-testing.
Comment #8
themic8 CreditAttribution: themic8 commentedComment #9
themic8 CreditAttribution: themic8 commentedText 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.
Comment #11
themic8 CreditAttribution: themic8 commented#9: People-miss-the-first-section-of-the-help-overview-page-issue-number-1387964.patch queued for re-testing.
Comment #13
themic8 CreditAttribution: themic8 commented#9: People-miss-the-first-section-of-the-help-overview-page-issue-number-1387964.patch queued for re-testing.
Comment #15
themic8 CreditAttribution: themic8 commentedTest 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.
Comment #16
realitylooprerolled, although the heading isn't displaying for me on local dev, test pass so it would be great to get some eyes on this.
Comment #17
alexanansi CreditAttribution: alexanansi commentedMy 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
Comment #18
realityloopThat look like it needs work then
Comment #19
galooph CreditAttribution: galooph commentedAdded 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>';
Comment #20
mohangupta CreditAttribution: mohangupta commentedI have tested this. See attached screenshot
Comment #21
realityloopI also checked that help tests are still passing this is ready to commit
Comment #22
lostkangaroo CreditAttribution: lostkangaroo commentedAdded this issue to #1908570: [meta] Update or create hook_help() texts for D8 core modules
Comment #23
alexpottCommitted 1146ebd and pushed to 8.x. Thanks!