Feat/block chart edit

[preda-bogdan]

Summary

Added a button inside Gutenberg to Edit the Chart from a single location. Will open a Media pop-up Modal, the same that is used outside of the editor. Added a new filter visualizer_legacy_block_edit to allow users to switch to the legacy behavior. Eg.

add_filter( 'visualizer_legacy_block_edit', '__return_true' );

Hide previous controls from the block. Added E2E test.

Will affect the visual aspect of the product

YES

Screenshots

Edit Chart


Media Pop-up


Test instructions

1. All tests should pass.
2. On a fresh instance add a chart inside the Gutenberg editor.
3. Check that the Edit Chart button is present and previous controls are not available.
4. Check that. the button will open the Media pop-up and allows you to edit the current chart.

Check before Pull Request is ready:

• [x] I have written a test and included it in this PR
• [x] I have run all tests and they pass
• [x] The code passes when running the PHP CodeSniffer
• [x] Code meets WordPress Coding Standards for PHP, HTML, CSS and JS
• [x] Security and Sanitization requirements have been followed
• [x] I have assigned a reviewer or two to review this PR (if you're not sure who to assign, we can do this step for you)

Closes Codeinwp/visualizer-pro#440.

[pirate-bot]

Plugin build for 48f6d40d43edfeacfda3c343ea871d6f11aeb9a8 is ready :bellhop_bell:!

• Download build

[irinelenache]

@preda-bogdan Tested and found a single issue:

• [x] If you edit the chart using the pop-up, there will be a console error after saving and the changes will be visible after refreshing the editor https://vertis.d.pr/v/wsi8xB

Can be checked here:

 Admin area URL: https://block-chart.s3-tastewp.com/wp-admin 
 Username: irinel 
 Password: v9M2uK9vP2Q 

[Soare-Robert-Daniel]

• [ ] If you edit the chart using the pop-up, there will be a console error after saving, and the changes will be visible after refreshing the editor https://vertis.d.pr/v/wsi8xB

@preda-bogdan you can add:

window.send_to_editor = () => {
    view.close();
};

I found the same issue yesterday.

[preda-bogdan]

@Soare-Robert-Daniel thank you, but we also need to notify the editor to re-render the block.

[preda-bogdan]

@irinelenache The chart should update upon the window closing and no error should be displayed in the console.

Let me know if everything is working as expected now. Thank you!

[irinelenache]

@preda-bogdan The issue is fixed now, thank you 🚀