HelpHub: Support theme update and associated plugin

[Clorith]

HelpHub builds on the support theme, but with some custom post types and widgets to customize the landing page of wordpress.org/support, and organize content.

The patch includes the following theme changes:

• A generic archive template, when more article types are planned for future phases we will likely introduce different archive views.
• The current forums front page is moved to the archive-forum template.
• A new (conditional) front page is introduced, it uses widgets and relies on the link block widget in the patch. If the widgets are undefined, the fallback is to use the forums archive like today.
• Generic code style cleanup.

The following plugin is introduced during the initial release:

• Adds a support-helphub plugin, in the same spirit as support-forums
• Custom post types for various articles
• A widget for declaring link blocks for the front page.

Enabling Gutenberg on WordPress.org/support is also a request to go along with this. The post types we'll be working with for now are REST enabled for this purpose.
Gutenberg has been checked to not cause any issues with bbPress, so that shouldn't be a concern at this time.

[Clorith]

I had forgotten to remove the WPORG fallbacks in HelpHub_first_iteration.patch​, please disregard it.

[ocean90]

Some quick feedback:

• It's look like there are a few theme files with code styling changes. Those should be removed from the patch to keep it clean.
• dirname( __FILE__ ) => __DIR__
• support-helphub/inc/helphub-front-page-blocks/helphub-front-page-blocks.php shouldn't have a plugin header.
• /support-helphub/inc/helphub-front-page-blocks/includes/class-support-helphub-front-page-blocks-widget.php shouldn't have the PhpStorm header.
• No need for helphub-post-types/index.php.
• support-helphub/inc/helphub-post-types/readme.txt can be removed. Also the plugin/file headers since it's no longer its own plugin.
• Do we really need jQuery UI CSS Framework 1.12.1?

[Clorith]

Thanks for the quick look!

HelpHub_first_iteration.3.patch​ removes all the PHPCS lines that aren't directly related to new/modified code, and addresses all the other points you made.

The jQuery UI styles are because of the WordPress Version CPT using jQuery UI in the backend, if there's a fixed one for use in meta that should obviously be used instead of the direct styles here.

[Clorith]

3485.patch​ is good to go.

Mostly the same as the previous patch, but introduces the custom editor role.

The HElpHub Editor role provides the default editor roles, with added capabilities to modify menus, as used by widget blocks, and the ability to view the customizer to ensure a good UI on the front end.

[netweb]

Patch 3485.2.patch​ is a minor refresh of 3485.patch​ with some merge conflicts resolved.

Further work ongoing...

[netweb]

This patch 3485.3-scss-css.patch​ accompanies the ​https://github.com/Kenshino/HelpHub/pull/221 PR

[dd32]

In 7283:

HelpHub: Add the plugin directory for the new HelpHub plugin.

See #3485.

[dd32]

In 7306:

Support Hub: Add Support Hub plugin

Props atachibana, bravokeyl, carl-alberto, clorith, felipeelia, hideokamoto, hlashbrooke, joyously, justingreerbbi, kenshino, mapk, mariovalney, mirucon, miyauchi, netweb, torounit, webdevmattcrom, zzap

See #3485.

[netweb]

In 7354:

Support Hub: Update Support Hub to latest changes via GitHub

References:

• ​https://github.com/Kenshino/HelpHub/commit/18f1f61
• ​https://github.com/Kenshino/HelpHub/commit/8e40f35
• ​https://github.com/Kenshino/HelpHub/commit/83e3919

Props clorith, kenshino, netweb.
See #3485.

[netweb]

In 7373:

Support Hub: Update Support Hub to latest changes via GitHub

Reference: ​https://github.com/Kenshino/HelpHub/commit/ed34600

Props milana_cap.
See #3485.

[netweb]

In 7813:

Support Forum: Updated bbPress templates

See #3485.

[SergeyBiryukov]

In 7818:

Support Forums: Revert [7813], overzealous escaping breaks legitimate markup.

See #3485.

[netweb]

In 7835:

Support Forum: Updated bbPress templates and styles

• This commit adds the bare minimum changes from ​https://github.com/Kenshino/HelpHub to implement the new Support Hub features for the Support site at ​https://wordpress.org/support
• Coding standards for existing code will follow in future commits

See #3485.

[SergeyBiryukov]

@netweb: [7835] unnecessary limits the width on existing pages and adds extra underlines on the forums, see 3485.page.png​ and 3485.forums.png​.

Could we limit these changes to HelpHub pages only, e.g. with .single-helphub_article instead of .single and .page-_some_helphub_class_ instead of .page?

[coffee2code]

In 7836:

Support Theme: Limit HelpHub-specific styles to HelpHub sections.

Props SergeyBiryukov, coffee2code.
See #3485.

[coffee2code]

[7836] seems to address the issues Sergey brought up. I didn't spot any other problems, so this may be ready for deploy. (Which should be noted is holding up deploy on any other potential w.org changes.)

[Clorith]

I've attached 3485.HelpHub_Manager.patch​ which introduces a new HelpHub Manager role, allowing manager users to add HelpHub Editors using the default profiles on wordpress.org/support/users/<username>.

This allows for more flexibility in who can be given the capabilities to manage HelpHub contributors, and also avoids the need for messing with core role and capability management.

Keymasters have the regular profile edit link which gives access to the same role changer, as such the capabilities for the base edit form is tied to the manage_helphub capability only (meaning HelpHub Managers and network admins will be able to see it).

[netweb]

Replying to coffee2code:

[7836] seems to address the issues Sergey brought up. I didn't spot any other problems, so this may be ready for deploy. (Which should be noted is holding up deploy on any other potential w.org changes.)

Looks good to me, thanks, deploying sounds good to me 👌

---

Replying to Clorith:

I've attached 3485.HelpHub_Manager.patch​ which introduces a new HelpHub Manager role, allowing manager users to add HelpHub Editors using the default profiles on wordpress.org/support/users/<username>.

Let's move this to a ticket of its own, primarily to close out this ticket and make further iterations in individual tickets please.

[coffee2code]

This has been deployed (yesterday, with the related CSS cache bust just now).

---

Replying to netweb:

Replying to Clorith:

I've attached 3485.HelpHub_Manager.patch​ which introduces a new HelpHub Manager role, allowing manager users to add HelpHub Editors using the default profiles on wordpress.org/support/users/<username>.

Let's move this to a ticket of its own, primarily to close out this ticket and make further iterations in individual tickets please.

Clorith has opened this particular item as #3918.