Problem/Motivation

In #2913321: Proposal to experiment with React for building Drupal’s administrative UIs, #2675320: A list of interactions that could benefit from having a smart frontend framework, #2920196: Which capabilities should modules have in a new JS based admin UI, and related issues, we're exploring the use of a JavaScript framework for Drupal core. This initiative has the support of the JavaScript subsystem maintainers, the frontend framework managers, Dries, etc.

Proposed resolution

@drpal and @justafish already seem to be coordinating things, so let's invite them to be the initiative coordinators.

@lauriii and the other JavaScript subsystem maintainers would be technical advisors to the initiative, but they already have separate roles in the governance for this.

Remaining tasks

  • I've called it the "JavaScript Framework Initiative" but feel free to propose something better.
  • @drpal and @justafish can confirm here whether they in fact want this role. :) https://www.drupal.org/contribute/core/maintainers#initiative
  • The product and framework managers probably will want to +1 this as a strategic initiative.
  • Dries appoints and signs off on initiatives and their coordinators so it's his decision ultimately.

Comments

xjm created an issue. See original summary.

xjm’s picture

Status: Active » Needs review
Issue tags: +Needs product manager review, +Needs frontend framework manager review, +Needs framework manager review
StatusFileSize
new623 bytes
GrandmaGlassesRopeMan’s picture

➕1️⃣ for @justafish

Additionally, I'm good with this, and accept the responsibility. ✌️

star-szr’s picture

+1!

justafish’s picture

👍 🎉

GrandmaGlassesRopeMan’s picture

+++ b/core/MAINTAINERS.txt
@@ -479,6 +479,10 @@ API-first Initiative
+JavaScript Framework Initiative

Is it possible to generalize this, maybe? Something like Drupal JavaScript Initiatives?

xjm’s picture

Assigned: Unassigned » dries

Seems like a good Dries question. Dries, thoughts on what we should call it?

effulgentsia’s picture

FWIW, my $0.02 is that for initiatives, more specific/focused is better. So I propose renaming to "React initiative" (since that's the chosen library) or "Reactive admin UIs initiative" (if we want to focus on the product benefit rather than the technology).

For reflecting other JS work in Drupal, we already have a "JavaScript" subsystem in MAINTAINERS.txt. But if we want initiatives for certain efforts (e.g., ES6), what about proposing those as separate initiatives?

xjm’s picture

I'd actually say we shouldn't make it just about React. Other initiatives are very broad with broad goals (Worfklow Initiative, not "Content Moderation Initiative" or "Workspaces Initiative", etc.)

"JavaScript Modernization Initiative" might be another option but I defer to the people organizing it. :)

effulgentsia’s picture

+1 to "JavaScript Modernization Initiative". I'm also happy to defer the final choice of name to @drpal, @justafish, and @Dries.

dawehner’s picture

@xjm asked me to explain why I dislike the term "React initiative".

a) React is not a self-goal, as in, the value is not in react, its in having a better user experience, better integration with modern JS etc.
b) The hard problems will not be in using react, it will probably be in the integration with PHP, and the general problem space of having modules inside the more full control JS world. Other problems will be defining a component library for example. Nothing is intrinsic a react problem.

justafish’s picture

+1 for JavaScript Modernisation Initiative

GrandmaGlassesRopeMan’s picture

@justafish s or z? Either way, ➕1️⃣ from me.

effulgentsia’s picture

Title: Add the JS framework initiative to MAINTAINERS.txt » Add the JS modernization initiative to MAINTAINERS.txt
Issue tags: -Needs framework manager review
StatusFileSize
new627 bytes

Initiative name change per #12, but with a "z".

Also, signing off on this as framework manager.

Version: 8.5.x-dev » 8.6.x-dev

Drupal 8.5.0-alpha1 will be released the week of January 17, 2018, which means new developments and disruptive changes should now be targeted against the 8.6.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

webchick’s picture

@drpal asked me to join the team as an initiative coordinator, to handle communication-style tasks in order to free up he and @justafish and others to rock out on JS, and just wanted to leave a note here that I'm both honoured to be asked and happy to play this role. :)

dries’s picture

This was blocked on having an initiative coordinator (based on conversations with all core committers, drpl and justafish). Big +1 for the drpl + justafish + webchick combination. I'm excited to see this move forward! Feel free to commit and updated version of this patch.

xjm’s picture

Assigned: dries » Unassigned
Status: Needs review » Needs work

Thanks @Dries!

I think we should update the patch to add @webchick?

ApacheEx’s picture

Status: Needs work » Needs review
StatusFileSize
new688 bytes
new434 bytes

Here it is.
I've added @webchick

webchick’s picture

Thank you, @ApacheEx! :)

From a product management POV, I guess one thing we want to ensure isn't lost is that the end goal of this initiative isn't to modernize Drupal's JavaScript, nor is it to ship Drupal with React. It's rather to do those things in service of utilizing the kinds of interactive interactions provided by a JS library like React for the purpose of making Drupal administration easier to use.

However, as we were discussing in UX meeting today, we don't yet have a real clear idea of what that "end game" looks like. #2902399: Redesign the Admin UI is the closest place we have for a brainstorming place on this, so we can continue to discuss over there in the coming weeks.

This should not hold up the initiative from kicking off, though, since there's quite a lot of "foundational" things on the plate listed at #2926656: [plan] Modernize Drupal's JavaScript. Just a general plea to keep the end goal in mind throughout. :)

Not marking RTBC yet, since framework manager review tag is still on it, but removing ours.

effulgentsia’s picture

@Cottser +1'd this in #4, so I think we can remove the "Needs frontend framework manager review" tag now.

webchick’s picture

Status: Needs review » Reviewed & tested by the community

Well all righty then!

nod_’s picture

Issue tags: +JavaScript

haha did not see that one go through.

xjm’s picture

Hooray!

I reordered it to be alphabetical (by last name) to match the rest of the file:

 JavaScript Modernization Initiative
+- Angela Byron 'webchick' https://www.drupal.org/u/webchick
 - Matthew Grill 'drpal' https://www.drupal.org/u/drpal
 - Sally Young 'justafish' https://www.drupal.org/u/justafish
-- Angela Byron 'webchick' https://www.drupal.org/u/webchick

Saving issue credit.

xjm’s picture

Version: 8.6.x-dev » 8.5.x-dev

Committed and pushed to 8.6.x. Thanks!

We'll want to backport this to 8.5.x. as well, so leaving RTBC for that after the commit freeze.

  • xjm committed 810aab4 on 8.6.x
    Issue #2921367 by ApacheEx, xjm, effulgentsia, drpal, webchick,...
xjm’s picture

I also added @drpal and @justafish to g.d.o/core so that they can post announcements about the initiative.

xjm’s picture

I also added @justafish to https://groups.drupal.org/core so she can post announcements etc. about the initiative as needed. (drpal was already a member, and she and drpal both already had "maintain issues" permission for core.)

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 19: 2921367-19.patch, failed testing. View results

xjm’s picture

Status: Needs work » Reviewed & tested by the community

Silly testbot, of course it doesn't apply to 8.6.x anymore. :)

  • xjm committed aa591cd on 8.5.x
    Issue #2921367 by ApacheEx, xjm, effulgentsia, drpal, webchick,...
xjm’s picture

Status: Reviewed & tested by the community » Fixed

Backported to 8.5.x as well. Thanks!

Status: Fixed » Closed (fixed)

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