#63821: Proposal: Drop support for IE conditional scripts and styles
-------------------------------------------------+-------------------------
Reporter: jonsurrell | Owner: joedolson
Type: enhancement | Status: closed
Priority: normal | Milestone: 6.9
Component: General | Version: 4.8
Severity: normal | Resolution: fixed
Keywords: has-patch has-unit-tests needs-dev- | Focuses:
note needs-testing has-test-info reviewing | javascript, css
-------------------------------------------------+-------------------------
Comment (by edge22):
Just wanted to chime in here about the deprecation notice.
As we saw with the `_load_textdomain_just_in_time` change in 6.8, a lot of
users have `WP_DEBUG` enabled without realizing it. That update caused a
wave of confusion and support requests from everyday users who suddenly
saw warnings in their logs or even on the frontend:
https://wordpress.org/search/_load_textdomain_just_in_time/
In my opinion, `_load_textdomain_just_in_time` was a necessary notice - it
revealed a real problem that could lead to incorrect translations.
In this case, though, the IE conditional change doesn't seem to expose the
same kind of breakage risk. The notice could end up creating noise and
unnecessary confusion for end-users without providing meaningful value to
developers.
Maybe it's better handled silently (or logged only when `SCRIPT_DEBUG` is
enabled), instead of surfacing as a `_doing_it_wrong()` notice.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/63821#comment:44>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform