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

Add way for additional styles to be added to editor. #514

Closed
wants to merge 1 commit into from

Conversation

BE-Webdesign
Copy link
Contributor

Fixes #422. Added an action so that someone could hook into where
gutenberg styles are loaded and bring in an editor style of their own.
I took this approach, as there is not a super clean way to mimic
add_editor_style() without adding another global variable to
WordPress, without PHP 5.3+, that I could find.

Fixes #422.  Added an action so that someone could hook into where
gutenberg styles are loaded and bring in an editor style of their own.
I took this approach, as there is not a super clean way to mimic
`add_editor_style()` without adding another global variable to
WordPress, without PHP 5.3+, that I could find.
@nylen
Copy link
Member

nylen commented May 15, 2017

Sorry, I've been meaning to reply to this for a while, but I think we will need a more sophisticated approach here, potentially including styles for each block (defined by the block? by the theme? I'm not sure yet). See also: #422 (comment)

@BE-Webdesign
Copy link
Contributor Author

Sounds good, so this will be tied in with the Server Side API for registering blocks?

@nylen
Copy link
Member

nylen commented May 18, 2017

so this will be tied in with the Server Side API for registering blocks?

Yes, though it's not yet clear to me how it will work. Something like a combination of add_theme_support and registering a stylesheet maybe?

@nylen nylen deleted the editor-style branch May 18, 2017 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants