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/controls template esnext #12

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

intelijens
Copy link

@intelijens intelijens commented Jun 13, 2020

Description

Add in new template for Create Block package, using Gutenberg Examples - 04 Controls ESNext
Example block written with ESNext standard and JSX support for GUI Controls– build step required.

How has this been tested?

  1. cd gutenberg
  2. npx wp-create-block -t controls-esnext
  3. Quick option: default for all settings; Personalized option: Client's choice
  4. run build (optional? not needed.)
  5. Finder -> open Gutenberg folder directory (leave folder open to clone/move, Step 7)
  6. Deploy a local/remote WordPress site
  7. Clone/move generated example-04-controls-esnext plugin folder (from Step 5) into your local/remote wp-content/plugins folder
  8. WordPress Dashboard -> plugins -> activate Example: Controls (ESNext)
  9. Block will appear as Example: Controls (ESNext) in editor under Layout; add the block
  10. Verify in browser

Screenshots

Step 8
Screen Shot 2020-06-19 at 12 24 40 PM
Step 9
Screen Shot 2020-06-19 at 12 25 56 PM
Screen Shot 2020-06-20 at 1 32 02 AM
Step 10
Screen Shot 2020-06-20 at 1 32 15 AM

Types of changes

Add - new example template

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR.

Commit Log:

  • add/controls-template-esnext - initial commit; clone exnext default sample
  • add/controls-template-esnext - update sample from wp/gutenberg master
  • add/controls-template-esnext - update to match https://github.com/WordPress/gutenberg-examples/tree/master/04-controls-esnext sample
  • add/controls-template-esnext - reverted edits from commit 9010591; removed unneeded import js; add back in the scss to the template block folder; prevent missing css error; all of this will likely change with gutenberg and create block update; adjusted background colors for editor view and client view to easily distinguish block is active
  • add/controls-template-esnext - reverted mistakened edit of esnext template

@intelijens intelijens linked an issue Jun 13, 2020 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Jun 13, 2020

Size Change: 0 B

Total Size: 1.13 MB

ℹ️ View Unchanged
Filename Size Change
build/a11y/index.js 1.14 kB 0 B
build/annotations/index.js 3.62 kB 0 B
build/api-fetch/index.js 3.4 kB 0 B
build/autop/index.js 2.82 kB 0 B
build/blob/index.js 620 B 0 B
build/block-directory/index.js 7.37 kB 0 B
build/block-directory/style-rtl.css 941 B 0 B
build/block-directory/style.css 942 B 0 B
build/block-editor/index.js 109 kB 0 B
build/block-editor/style-rtl.css 10.7 kB 0 B
build/block-editor/style.css 10.7 kB 0 B
build/block-library/editor-rtl.css 7.59 kB 0 B
build/block-library/editor.css 7.6 kB 0 B
build/block-library/index.js 130 kB 0 B
build/block-library/style-rtl.css 8.04 kB 0 B
build/block-library/style.css 8.04 kB 0 B
build/block-library/theme-rtl.css 730 B 0 B
build/block-library/theme.css 732 B 0 B
build/block-serialization-default-parser/index.js 1.88 kB 0 B
build/block-serialization-spec-parser/index.js 3.1 kB 0 B
build/blocks/index.js 48.2 kB 0 B
build/components/index.js 198 kB 0 B
build/components/style-rtl.css 15.9 kB 0 B
build/components/style.css 15.9 kB 0 B
build/compose/index.js 9.65 kB 0 B
build/core-data/index.js 11.4 kB 0 B
build/data-controls/index.js 1.29 kB 0 B
build/data/index.js 8.44 kB 0 B
build/date/index.js 5.47 kB 0 B
build/deprecated/index.js 772 B 0 B
build/dom-ready/index.js 569 B 0 B
build/dom/index.js 3.19 kB 0 B
build/edit-navigation/index.js 9.87 kB 0 B
build/edit-navigation/style-rtl.css 1.02 kB 0 B
build/edit-navigation/style.css 1.02 kB 0 B
build/edit-post/index.js 303 kB 0 B
build/edit-post/style-rtl.css 5.51 kB 0 B
build/edit-post/style.css 5.5 kB 0 B
build/edit-site/index.js 16.6 kB 0 B
build/edit-site/style-rtl.css 3.03 kB 0 B
build/edit-site/style.css 3.03 kB 0 B
build/edit-widgets/index.js 9.32 kB 0 B
build/edit-widgets/style-rtl.css 2.42 kB 0 B
build/edit-widgets/style.css 2.42 kB 0 B
build/editor/editor-styles-rtl.css 537 B 0 B
build/editor/editor-styles.css 539 B 0 B
build/editor/index.js 44.8 kB 0 B
build/editor/style-rtl.css 3.85 kB 0 B
build/editor/style.css 3.85 kB 0 B
build/element/index.js 4.65 kB 0 B
build/escape-html/index.js 733 B 0 B
build/format-library/index.js 7.73 kB 0 B
build/format-library/style-rtl.css 547 B 0 B
build/format-library/style.css 548 B 0 B
build/hooks/index.js 2.13 kB 0 B
build/html-entities/index.js 622 B 0 B
build/i18n/index.js 3.56 kB 0 B
build/is-shallow-equal/index.js 710 B 0 B
build/keyboard-shortcuts/index.js 2.51 kB 0 B
build/keycodes/index.js 1.94 kB 0 B
build/list-reusable-blocks/index.js 3.13 kB 0 B
build/list-reusable-blocks/style-rtl.css 450 B 0 B
build/list-reusable-blocks/style.css 451 B 0 B
build/media-utils/index.js 5.29 kB 0 B
build/notices/index.js 1.79 kB 0 B
build/nux/index.js 3.4 kB 0 B
build/nux/style-rtl.css 663 B 0 B
build/nux/style.css 660 B 0 B
build/plugins/index.js 2.56 kB 0 B
build/primitives/index.js 1.5 kB 0 B
build/priority-queue/index.js 789 B 0 B
build/redux-routine/index.js 2.85 kB 0 B
build/rich-text/index.js 14 kB 0 B
build/server-side-render/index.js 2.68 kB 0 B
build/shortcode/index.js 1.7 kB 0 B
build/token-list/index.js 1.28 kB 0 B
build/url/index.js 4.06 kB 0 B
build/viewport/index.js 1.85 kB 0 B
build/warning/index.js 1.14 kB 0 B
build/wordcount/index.js 1.17 kB 0 B

compressed-size-action

@intelijens intelijens mentioned this pull request Jun 13, 2020
6 tasks
*
* @see https://www.npmjs.com/package/@wordpress/scripts#using-css
*/
import './style.scss';
Copy link
Author

@intelijens intelijens Jun 13, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fabiankaegy can we discuss this import during our meet up, I noticed some odd behavior when I left it included, causing my block to NOT show up in the editor, even when plugin is activated.

@brezocordero
Copy link

After activating the plugin on a site I get this error:
ErrorsAfterActivatingPlugin

@intelijens
Copy link
Author

intelijens commented Jun 19, 2020

Thank you for pointing out that error @brezocordero, I will try to recreate and perform additional testing. Also, thank you for the recommendation of suggesting completion of the initial documentation, i.e., description, how has this been tested, screenshots.

…moved unneeded import js; add back in the scss to the template block folder; prevent missing css error; all of this will likely change with gutenberg and create block update; adjusted background colors for editor view and client view to easily distinguish block is active
@intelijens
Copy link
Author

Screen Shot 2020-06-19 at 11 16 44 PM

@brezocordero found the cause, i tried to get too fancy pulling in those new updates again from that second commit, I should have KISS method, Keep It Simple S̶t̶u̶p̶i̶d̶ Silly. I didn't notice the console log error before my PR, my apologies. I uncovered my mistakes by trying the other templates from the start again.

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.

3 participants