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

Disallowed/locked blocks trigger browser console warning #70

Closed
marijnvdwerf opened this issue Feb 6, 2024 · 1 comment
Closed

Disallowed/locked blocks trigger browser console warning #70

marijnvdwerf opened this issue Feb 6, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@marijnvdwerf
Copy link
Contributor

Describe the bug
When opening a post that has illegal blocks, the following warning appeared:

Warning

Unsupported style property background-color. Did you mean backgroundColor?

Seems to be related to this line:

<div style={ { opacity: 0.6, 'background-color': '#eee', border: '2px dashed #999' } }>

To Reproduce

  • Add a block, disallow it in governance.json, reload the post

Expected behavior
No warnings in console

Actual behavior
Warnings in console

Version of the plugin
1.0.6

Additional context

react-dom.js?ver=18.2.0:73 Warning: Unsupported style property background-color. Did you mean backgroundColor?
    at div
    at div
    at Disabled (https://testsite.lndo.site/wp-includes/js/dist/components.js?ver=9a812780592a1d78379b:61076:3)
    at https://testsite.lndo.site/wp-content/plugins/vip-governance/build/index.js?ver=289fda8f4f8dd70bdba8:1:3203
    at FilteredComponentRenderer (https://testsite.lndo.site/wp-includes/js/dist/components.js?ver=9a812780592a1d78379b:75975:9)
    at EditWithGeneratedProps (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:16549:5)
    at BlockEdit (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:16604:5)
    at BlockCrashBoundary (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:16932:5)
    at BlockListBlock (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:18190:10)
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:15699:5
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:32182:5
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:33391:5
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:34958:21
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:37068:112
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:37911:67
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:38389:5
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:38705:5
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:38779:5
    at FilteredComponentRenderer (https://testsite.lndo.site/wp-includes/js/dist/components.js?ver=9a812780592a1d78379b:75975:9)
    at https://testsite.lndo.site/wp-includes/js/dist/compose.js?ver=228e7d7fccaae67c220c:3232:10
    at https://testsite.lndo.site/wp-includes/js/dist/data.js?ver=2b5bb06caaeb5048ed96:4436:25
    at https://testsite.lndo.site/wp-includes/js/dist/data.js?ver=2b5bb06caaeb5048ed96:4286:22
    at shouldComponentUpdate (https://testsite.lndo.site/wp-includes/js/dist/compose.js?ver=228e7d7fccaae67c220c:3268:12)
    at Items (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:28529:3)
    at BlockListItems
    at div
    at Root (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:28447:3)
    at BlockList
    at RecursionProvider (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:56320:3)
    at div
    at WritingFlow (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:20399:3)
    at ExperimentalBlockCanvas (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:41821:3)
    at div
    at MotionComponent (https://testsite.lndo.site/wp-includes/js/dist/components.js?ver=9a812780592a1d78379b:18531:46)
    at div
    at MotionComponent (https://testsite.lndo.site/wp-includes/js/dist/components.js?ver=9a812780592a1d78379b:18531:46)
    at div
    at BlockTools (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:55870:3)
    at VisualEditor (https://testsite.lndo.site/wp-includes/js/dist/edit-post.js?ver=57aa460a64bb0e4ae963:3929:3)
    at div
    at NavigableRegion (https://testsite.lndo.site/wp-includes/js/dist/edit-post.js?ver=57aa460a64bb0e4ae963:1391:3)
    at div
    at div
    at div
    at InterfaceSkeleton (https://testsite.lndo.site/wp-includes/js/dist/edit-post.js?ver=57aa460a64bb0e4ae963:1457:3)
    at Layout (https://testsite.lndo.site/wp-includes/js/dist/edit-post.js?ver=57aa460a64bb0e4ae963:8679:63)
    at ErrorBoundary (https://testsite.lndo.site/wp-includes/js/dist/editor.js?ver=09660a848fa8379476ab:5911:5)
    at BlockRefsProvider (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:17776:3)
    at Provider (https://testsite.lndo.site/wp-includes/js/dist/components.js?ver=9a812780592a1d78379b:36533:3)
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:19222:5
    at https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:18775:5
    at WithRegistryProvider(Component)
    at BlockContextProvider (https://testsite.lndo.site/wp-includes/js/dist/block-editor.js?ver=0930e041f64667e446db:16486:3)
    at EntityProvider (https://testsite.lndo.site/wp-includes/js/dist/core-data.js?ver=77370c9a15a7db2ae084:6359:3)
    at EntityProvider (https://testsite.lndo.site/wp-includes/js/dist/core-data.js?ver=77370c9a15a7db2ae084:6359:3)
    at https://testsite.lndo.site/wp-includes/js/dist/editor.js?ver=09660a848fa8379476ab:12496:3
    at https://testsite.lndo.site/wp-includes/js/dist/editor.js?ver=09660a848fa8379476ab:12081:5
    at WithRegistryProvider(Component)
    at SlotFillProvider (https://testsite.lndo.site/wp-includes/js/dist/components.js?ver=9a812780592a1d78379b:36376:3)
    at provider_SlotFillProvider (https://testsite.lndo.site/wp-includes/js/dist/components.js?ver=9a812780592a1d78379b:36398:5)
    at Provider (https://testsite.lndo.site/wp-includes/js/dist/components.js?ver=9a812780592a1d78379b:36533:3)
    at Editor (https://testsite.lndo.site/wp-includes/js/dist/edit-post.js?ver=57aa460a64bb0e4ae963:9253:3)
@alecgeatches
Copy link
Contributor

@marijnvdwerf The fix for these warnings in #71 was included in the latest release: https://github.com/Automattic/vip-governance-plugin/releases/tag/1.0.7. Thank you for reporting!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants