Code Editor: Error disables the Update File button.

[paaljoachim]

When there is an error in the code the Update File button is disabled.
It would be better to find another approach.

Such as giving a warning about saving the file etc... Leaving it up to the user to decide on saving the file.

[westonruter]

There could be an “I understand” override checkbox similar to when supplying an insecure password. Or rather, instead of having a separate checkbox the error notification itself could simply be made dismissible, and once dismissed then the ability to save would be restored. This could be integrated into Custom CSS, Custom HTML widget, and file editor.

[westonruter]

Bumping priority to high for visibility and alignment with 4.9 goals, and given proximity to beta 1 deadline.

[melchoyce]

Let's disable save locking for CSS errors — the only way you're going to whitescreen your site is if you display:none everything, haha.

Working on an override design for php/js files.

[melchoyce]

update-despite-errors.png​ introduces a checkbox to toggle on saving.

Some notes:

• When you click the checkbox, the disabled button becomes enabled.
• Let's save this setting by default, so the next time someone makes an error the checkbox is already selected.
• On smaller screens, the checkbox will wrap underneath the notice. The update button will wrap underneath that, so it goes Notice > Checkbox > Button.
• I've changed "save" in the notice to "update this file," to match the label on the primary button.
• The editor's been moved down a bit to give it some space between it and the header.
• Likewise, I've moved down the template files in the right side of the screen so it lines up with the top of the editor.
• I've also moved the documentation to the top of the screen, so it's not grouped with the update settings.

[westonruter]

In 41721:

File Editors: Introduce sandboxed live editing of PHP files with rollbacks for both themes and plugins.

• Edits to active plugins which cause PHP fatal errors will no longer auto-deactivate the plugin. Supersedes #39766.
• Introduce sandboxed PHP file edits for active themes, preventing accidental whitescreening of a user's site when introducing a fatal error.
• After writing a change to a PHP file for an active theme or plugin, perform loopback requests on the file editor admin screens and the homepage to check for fatal errors. If a fatal error is encountered, roll back the edited file and display the error to the user to fix and try again.
• Introduce a secure way to scrape PHP fatal errors from a site via wp_start_scraping_edited_file_errors() and wp_finalize_scraping_edited_file_errors().
• Moves file modifications from theme-editor.php and plugin-editor.php to common wp_edit_theme_plugin_file() function.
• Refactor themes and plugin editors to submit file changes via Ajax instead of doing full page refreshes when JS is available.
• Use get method for theme/plugin dropdowns.
• Improve styling of plugin editors, including width of plugin/theme dropdowns.
• Improve notices API for theme/plugin editor JS component.
• Strip common base directory from plugin file list. See #24048.
• Factor out functions to list editable file types in wp_get_theme_file_editable_extensions() and wp_get_plugin_file_editable_extensions().
• Scroll to line in editor that has linting error when attempting to save. See #41886.
• Add checkbox to dismiss lint errors to proceed with saving. See #41887.
• Only style the Update File button as disabled instead of actually disabling it for accessibility reasons.
• Ensure that value from CodeMirror is used instead of textarea when CodeMirror is present.
• Add "Are you sure?" check when leaving editor when there are unsaved changes.

Supersedes [41560].
See #39766, #24048, #41886.
Props westonruter, Clorith, melchoyce, johnbillion, jjj, jdgrimes, azaozz.
Fixes #21622, #41887.

[westonruter]

In 42171:

Customize: Allow notifications for linting errors in code editor control (for Additional CSS) to be overridden to allow saving.

Implements the same override that was implemented in [41721] for the theme/plugin editors.

See #41897, #41887.
Fixes #42528.

[westonruter]

In 42172:

Customize: Allow notifications for linting errors in code editor control (for Additional CSS) to be overridden to allow saving.

Implements the same override that was implemented in [41721] for the theme/plugin editors.

See #41897, #41887.
Fixes #42528 for 4.9.