Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gutenberg and _s #1248

Open
samikeijonen opened this issue Dec 29, 2017 · 30 comments
Open

Gutenberg and _s #1248

samikeijonen opened this issue Dec 29, 2017 · 30 comments
Labels
Help wanted Extra attention is needed Second Opinion

Comments

@samikeijonen
Copy link
Contributor

samikeijonen commented Dec 29, 2017

I'm opening this general issue for Gutenberg in _s. I'm pretty sure _s wants to be 100% ready for Gutenberg, whatever that means :) Because it can mean that

  • let Core handle all the blocks styling etc.
  • _s gives base ideas how themers can work with blocks, styling in the editor and front end etc.
@mor10
Copy link
Contributor

mor10 commented Dec 29, 2017

Only practical example I've seen of this so far is from @BinaryMoon: BinaryMoon/granule@777cd29

@caraya
Copy link

caraya commented Dec 29, 2017

If you're incorporating Gutenberg it should be opt-in. Let's not forget that WordPress as a CMS is used for a variety of content, not all of which requires blocks.

@mor10
Copy link
Contributor

mor10 commented Dec 29, 2017

@caraya Gutenberg is going to be a core feature in WordPress in a few months. Gutenberg support is necessary to allow the theme authors full control over the theme.

@caraya
Copy link

caraya commented Dec 30, 2017

@mor10 either through plugins or via some sort of theme customization people who don't want them will remove Gutenberg support. I believe it's better if the removal happens through theme customization, it saves people from having to install yet another Wordpress plugin

Am I missing something?

@mor10
Copy link
Contributor

mor10 commented Dec 30, 2017

In the short term, developers can declare lack of Gutenberg support in plugins to force WordPress to fall back on the classic editor. That's a stop-gap measure until plugins have time to update. Blocking Gutenberg at theme level goes against how WordPress will work. Once Gutenberg evolves beyond the editor, which is the next step, blocking it will no longer be a viable option.

@samikeijonen
Copy link
Contributor Author

samikeijonen commented Jan 6, 2018

Here are Gutenberg docs for themes. Mainly about add_theme_support() at the moment.

@crunnells
Copy link
Contributor

The Theme Team here at Automattic is looking at adding Gutenberg support to _s in the near future.

@philiparthurmoore
Copy link
Collaborator

@crunnells Where's the patch for this? I saw the label has been removed. We could probably remove Second Opinion. It seems unlikely that keeping as much support as possible for Gutenburg out of _s would be a bad idea. :-)

@tessak22
Copy link

Has this been completed with the release of 5.0 just around the corner?

@samikeijonen
Copy link
Contributor Author

@tessak22 No it hasn't.

@tessak22
Copy link

@samikeijonen is there a way I can dig in and help?

@samikeijonen
Copy link
Contributor Author

@tessak22 I'm not sure about status of _s, and what kind of support it's maintainers want to go with.

One helpful thing would be to dig in what have been done in default themes, and what parts of that could be used in _s.

@primedime
Copy link

Just checking in on this. I see on the main page it says Build Failed. Is _s still safe to use moving forward?

I have a new project I am starting and wanted to use _s but unsure if it is Gutenburg ready. Thanks!

@samikeijonen
Copy link
Contributor Author

It's not Gutenberg ready in that sense but other than that safe to use.

@caraya
Copy link

caraya commented Jan 15, 2019

@tessak22 the twenty nineteen theme was built primarily around Gutenberg support and think that might be the best starting theme to dig in. I'm doing the same :)

@jaclyntan
Copy link

So far the best solution I've come up with is to convert the compiled block css from gutenberg to scss. Does anyone know if there is a gutenberg update coming for _s anytime soon??

I dug around in twentynineteen (https://github.com/WordPress/twentynineteen/blob/master/sass/blocks/_blocks.scss) but it's philosophy doesn't fit with _s IMO. Too many specific/opinionated variables and breakpoints. _s doesn't even support breakpoints at this moment, and I'm not sure it should in the future.

The problem is, with the nature of gutenberg there are a lot of new variables and things to take into account such as breakpoints and columns....and I have a block.scss I am happy to share, but as I said, it's either _s accepts responsive styles, or I strip out the responsive styles altogether.

@loorlab
Copy link

loorlab commented Apr 2, 2019

+1

@x5hwuk7nc6jsudcc
Copy link

x5hwuk7nc6jsudcc commented May 23, 2019

The Theme Team here at Automattic is looking at adding Gutenberg support to _s in the near future.

@crunnells, this comment was more than 12 months ago. Do we have an update? It would be great to have sample Gutenberg blocks out-of-the-box for _s.

@robwent
Copy link

robwent commented Jul 1, 2019

I can probably help with this if anyone knows what's needed?

Seems like Gutenberg has its own classes for things like columns, so should just be a case of copying them from twentynineteen or converting them to sass?

Edit: Should have read @jaclyntan's post first. Would like to see your block.scss if possible.

@devsrealm
Copy link

Any update for Gutenberg yet?

@jaclyntan
Copy link

The blocks.scss I am using at the moment is just a compiled version of this file:
https://github.com/WordPress/twentynineteen/blob/master/sass/blocks/_blocks.scss. I've been using it in projects very minimally since I have yet to use GB substantially. I don't think it is a good reference or addition to _s as there are media queries and random units mixed in throughout.

If anyone wants to tackle formatting twentynineteen's scss for _s please have a go. I think we all really need a gutenberg update at this point and something that is formatted with respect to _s' philosophy is definitely needed imo, especially if WP want more devs to support gutenberg........

@steveo2001
Copy link

Can anyone give an update on current state of underscores?

@philiparthurmoore
Copy link
Collaborator

Pretty much abandoned. #1369

@robwent
Copy link

robwent commented Oct 9, 2019

I don't read that as being abandoned.
#1369 (comment)

@philiparthurmoore
Copy link
Collaborator

@robwent A simple look at open issues, pull requests, and general activity around the project will clear up any confusion.

"Abandoned" could easily be called "not a priority anymore", for now.

If/when it is I'm sure everyone will see more activity around the repo. Until then, yeah, I'd focus on default themes or some of .com's more recent themes for new ideas or ways of coding themes.

@Ismail-elkorchi Ismail-elkorchi added the Help wanted Extra attention is needed label Apr 18, 2020
@crunnells crunnells removed their assignment Apr 23, 2020
@kangabell
Copy link

Would it be helpful to add in some empty placeholder rules to address the different block selectors available? Something like:

.wp-block-group {
}

.wp-block-image {
}

.wp-block-button {
}

// ... etc.

Similar to some other parts of _S where there are placeholders in the sass, e.g. https://github.com/Automattic/_s/blob/master/sass/navigation/_menus.scss

That way there are no unnecessary "opinions" inserted about what the theme dev should do with these blocks, but they're there for us to add to as we go along.

@ianstewart
Copy link
Contributor

new ideas or ways of coding themes

This is pretty neat. https://github.com/WordPress/theme-experiments/

@Ismail-elkorchi
Copy link
Contributor

Ismail-elkorchi commented May 2, 2020

Would it be helpful to add in some empty placeholder rules to address the different block selectors available?

I don't have yet a strong opinion about the best way to approach gutenberg integration for _s, but I'm sure that we should tackle #1215 first, as it will help us generate the frontend and the editor stylesheets from the same code base.

@Ismail-elkorchi
Copy link
Contributor

Oups ! I clicked the wrong button. 😅

@gsarig
Copy link

gsarig commented Jun 8, 2020

The minimum that could be added, in my opinion, is support for editor-styles, like for example:

add_theme_support( 'editor-styles' );
add_editor_style( '/css/editor-styles.css' );

Personally, I put all gutenberg-related code in a special "gutenberg/gutenberg.php" file, since I usually need some more adjustments like additional alignments for core blocks, blacklist or whitelist blocks etc. and it helps keeping it better organized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help wanted Extra attention is needed Second Opinion
Projects
None yet
Development

No branches or pull requests