Contextual Help Tabs Cleanup

[dougwrites]

For all the help tabs not covered by ticket #15314 (Network Admin).

Two screens are missing text currently except for generic links to the Codex and Support Forums. These screens are "Edit Media" and "Link Categories." I have draft text for these and will put that into comments below.

Also, how much time is left, if any, to make small modifications in wording to existing help screens for the sake of describing new/changed features or to be less confusing? If that window is still open, I have a list of about twenty such items.

Finally, did we want to clean up the Codex links so that they are all in the format of "Documentation on X Y" instead of having 15 still "X Y Documentation"?

[dougwrites]

draft help text for "Edit Media" screen:

This screen allows you to edit five fields for metadata in a file within the media library.

For images only, you can click on Edit Image under the thumbnail to expand out an inline image editor with icons for cropping, rotating, or flipping the image as well as for undoing and redoing. The boxes on the right give you more options for scaling the image, for cropping it, and for cropping the thumbnail in a different way than you crop the original image. You can click on Help in those boxes to get more information. Note that you crop the image by clicking on it (the Crop icon is already selected) and dragging the cropping frame to select the desired part. Then click Save to retain the cropping.

Remember to click Update Media to save metadata entered or changed.

Edit Media in Codex ​http://codex.wordpress.org/Media_Add_New_SubPanel#Edit_Media

[nacin]

Go for it. String freeze won't be for another month. Sooner the better.

[dougwrites]

draft help text for "Link Categories" screen:

You can create groups of links by using link categories. Link category names must be unique and link categories are separate from the categories you use for posts.

You can delete link categories in the Bulk Action pulldown, but that action does not delete the links within the category. Instead, it moves them to the default link category.

For more information:

Documentation on Link Categories: ​http://codex.wordpress.org/Links_Link_Categories_SubPanel

[zeo]

Finally, did we want to clean up the Codex links so that they are all in the format of "Documentation on X Y" instead of having 15 still "X Y Documentation"?

See this old patch that need refresh, ​http://core.trac.wordpress.org/attachment/ticket/13736/13736.diff

[dougwrites]

Two features added (admin bar, dev blog) and doc on

[dougwrites]

doc on xy rather than xy doc for 3 codex links

[dougwrites]

change to "Documentation on Media Library"

[dougwrites]

added contextual help to file where none was before; apparently it had defaulted to generic links previously

[dougwrites]

Adds a "The" and a doc on to the widget help screen

[dougwrites]

Can anyone help me figure out where to place the code for the help screen for "Link Categories?" Changeset 15820 moved edit-link-categories.php into edt-tags.php and I'm having trouble figuring how that file calls help screens. Said file does call a help tab for the Posts > Category screen.

[nacin]

In edit-tags.php, we build the contextual help sentence by sentence, based on the taxonomy we're dealing with. In trunk, this starts around line 161 and ends at 196.

[dougwrites]

So to build it this way to call three help screens instead of two, would the following syntax be on the right track?:

if ( 'category' == $taxonomy

'link_category' == $taxonomy

'post_tag' == $taxonomy ) {

if ( 'category' == $taxonomy )
elseif ( 'link_category' == $taxonomy )
else

[nacin]

Yep. (Your formatting came through correctly in the email. You can surround code with {{{ and '}}}`.)

Though at that point we'd probably do if ( in_array( $taxonomy, array( 'category', 'link_category', 'post_tag' ) ) ), but that's something for me to worry about :-)

[ryan]

(In [16326]) Contextual help updates. Props dougwrites. see #15346

[markmcwilliams]

removes the few spaces that entered the url text of r16326 and adds some spacing

[dougwrites]

So that file can call 3 help screens (cats, tags, link cats) instead of 2.

[dougwrites]

Sorry, uploaded the wrong file version of edit-tags. In the .patch, there is still a problem with lines 164-165 and 172 (partially redundant); the elseif prevents the link cat help from calling line 167, which is for another screen. I'm pretty sure there's a more elegant way to do this...

[ryan]

(In [16333]) Whitespace cleanup in contextual help. Props markmcwilliams. see #15346

[zeo]

Replying to dougwrites:

Sorry, uploaded the wrong file version of edit-tags. In the .patch, there is still a problem with lines 164-165 and 172 (partially redundant); the elseif prevents the link cat help from calling line 167, which is for another screen. I'm pretty sure there's a more elegant way to do this...

You might wanna double check and refresh your patch for unnecessary whitespace. See markmcwilliams patch.

[dougwrites]

improved spacing and logic; still doesn't fix hiccup in lines 164-165; does output 3 correct help screens

[zeo]

@dougwrites,

Post Tags: ​http://codex.wordpress.org/Posts_Post_Tags_SubPanel
Categories: ​http://codex.wordpress.org/Posts_Categories_SubPanel

Please refresh your edit-tags2.patch with the new link.

Also it's best to create patch or diff from WP root dir.

[dougwrites]

3 help screens for tags. Revised version of same patch to include more current Codex links; also more directly from nightly build

[dougwrites]

Zeo: thanks for the catch on the Codex links. I deleted the file locally and re-did the SVN Update from the nightly build, which I believe fixes your patch creation concern.

Obviously, my hope is for edit-tags3.patch (or a revision fixing the hiccup still in 164-5) to be committed instead of edit-tags.patch or edit-tags2.patch, which should be ignored now.

[zeo]

@dougwrites, line 170 should be $help .= and not $help =

If not mistaken, it seems the help content for link categories seems incomplete. Perhaps asking Jane Wells or the wp-docs team for 2nd oppinion or more content?

[dougwrites]

Updated Codex links for 2 screens: Posts and Pages

[dougwrites]

Expanded one sentence and added a second Codex link

[dougwrites]

Added a phrase and updated Codex links for Users screen help tab

[dougwrites]

Added a phrase and updated Codex links for Users screen help tab

[dougwrites]

For 4 screens in Pages and Post (Edit same as Add New): add 2 Post features, fixes Codex links

[dougwrites]

Added text for Press This and Cats/Tags Coverter; Codex link changed to Doc on XY

[dougwrites]

Changed Codex link to Doc on XY rather than XY Doc

[dougwrites]

change in Codex link

[dougwrites]

Are the filters still coming back for Tools>Export in this cycle? I'm looking at tickets #14750, #15197, #15219, and most of the mentions of their temporary removal are about 3 weeks old. When the filters are restored, we'll need to put back the help tab text with some tweaks and explanation of the new UI for filters.

[dougwrites]

Note that the Formats part but not all of AddEditPostAddEditPagehelp.patch has been surpassed by ticket #15376. I'd still like to see a mention of the full screen toggle and a rewording of the Codex link; that page really does discuss Add, Editing, and Writing neatly bundled together.

[dougwrites]

clearer wording for Admin Bar and Seperator Lines

[dougwrites]

And the dashboard needs yet more fixing. There's a "the the" where a "the" will suffice, and we probably want to look at the paragraph that keeps saying "modules" instead of "boxes." My sense is that I shouldn't keep lobbing more tiny patches at the same index.php file until a core person has had time to deal with it. Guidance?

[PeteMall]

Can you submit this as a single patch? I'm going to look at the contextual help tickets with Jane tomorrow.

[dougwrites]

@PeteMall: this isn't everything in a single patch; sorry. These 14 are fairly routine; I have another 5 files with more complex problems/questions/issues. I'll list those in a comment below.

[dougwrites]

The five files I think need even more scrutiny are edit-tags.php (possible problems in the PHP logic, see edit-tags3.patch above) and edit-form-advanced (all kinds of stuff) as well as themes.php, menu.php, and options-permalink.php (smaller questions I still don't have answers for).

[zeo]

@dougwrites did u even read my previous reply? I already gave u the fix.

[dougwrites]

@zeo: I believe you're referring to comment 15 in this ticket... I did read and appreciate it. Sorry for not saying so. The ongoing problem, even when I tested the change of

$help .= 

and not

 $help =

is that I cannot work the

if, ifelse

logic to call more than one paragraph on the Link Categories help tab. Thus I had to put the exact same content that was in two paragraphs in 3.0 into one big paragraph for this version. I suspect there are even more problems still there. If anyone wants to write another patch which solves all this for edit-tags.php, that would be great.

[dougwrites]

Extensive cleanup for edit-form-advanced.php; separated from other help tab patches because this file/screen also has tickets #15376 and #15638 working on improving the help text about Formats (not addressed by this patch). Now includes changes in Screen Options (nod to WP-Testers).

[zeo]

Replying to dougwrites:

logic to call more than one paragraph on the Link Categories help tab. Thus I had to put the exact same content that was in two paragraphs in 3.0 into one big paragraph for this version. I suspect there are even more problems still there. If anyone wants to write another patch which solves all this for edit-tags.php, that would be great.

It would be easier if you put the text in .txt. It's easier to see what's exactly u trying to accomplish under Categories, Post Tags, Link Categories help tab. Example: ​http://core.trac.wordpress.org/attachment/ticket/13467/Media%20section.txt

[dougwrites]

Second try to upload same patch for 3 files; put it in just before the upgrade this morning and it disappeared. At this point, would it help to put all the diffs for all 18 files into a single patch?

[nacin]

Yes. One patch with everything. There's no way we can reasonably work with 18 different patches, when I don't know if any of these succeed each other.

Please diff from the root, not from within wp-admin.

[dougwrites]

Good, working on it right now.

[dougwrites]

@Nacin: This should cover everything. I hope the diff from root is done the right way.

[nacin]

(In [16980]) Help text edits. props dougwrites, fixes #15346.

[nacin]

Thanks Doug. Any deviations were via an editing session with Jane.

[zeo]

Minor fix to edit-tags.php

[zeo]

Documentation on x y

[zeo]

Reopen. Please review 2 of my latest patch.

[zeo]

Update: edit-tags thingy was fixed in #15845

[nacin]

(In [17014]) Help fixes. props zeo, fixes #15346.

[SergeyBiryukov]

Minor typo fix

[SergeyBiryukov]

Fixed typo, see the patch above.

[nacin]

(In [17039]) Fix one of my typos. props SergeyBiryukov, fixes #15346.

[nacin]

Please be sure to re-open if you find any other typos.

[nacin]

(In [17054]) Simplify this string. props jane, see #15346, see #15829.