#8026 closed defect (bug) (fixed)
2.7 Admin Validation Fixes
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 2.7 | Priority: | normal |
Severity: | normal | Version: | 2.7 |
Component: | Administration | Keywords: | has-patch dev-reviewed |
Focuses: | Cc: |
Description
A generic ticket for validation fix patches.
Attachments (2)
Change History (14)
#4
@
17 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Fix validation errors introduced in [9464] (onClick
) plus do a CDATA on the inline JS.
This ticket was mentioned in PR #8131 on WordPress/wordpress-develop by @flixos90.
5 months ago
#6
This is an exploration for using [cross-document view transitions]() in a WordPress theme, specifically the Twenty Fifteen theme, similar to #8026.
The difference from that PR is that this one here only transitions specifically the post title and featured image between an archive page and the post's own page, while the other PR transitions the entire main content of the page with slide-in / slide-out transition.
A benefit of the approach explored here is that it could possibly be applied in the same way for almost any WordPress theme, as it only relies on elements commonly used across WordPress themes (both block themes and classic themes), and it avoids more opinionated transitions.
Trac ticket:
@flixos90 commented on PR #8131:
4 months ago
#7
My assumption is that the WordPress Playground catches an error somewhere, preventing the code from executing properly and locking things up.
When this happens, nothing shows up in my browser console. FWIW I experience this bug outside of the WordPress Playground, on my local development environment with the same code, so it's not specific to the WordPress Playground. There's no try catch
block anywhere in the view-transitions.js
file, so I'm not sure what's happening. I'm still experiencing this problem consistently.
4 months ago
#8
FWIW I experience this bug outside of the WordPress Playground, on my local development environment with the same code, so it's not specific to the WordPress Playground.
Does your local dev env also run PHP using WASM, or is it actually a standalone env using a real PHP server? I’m asking because notice that CPU for the tab spikes to 100% when it hangs, which makes me think there is more at play here:
Also, could you check how Safari behaves with your local setup?
@flixos90 commented on PR #8131:
4 months ago
#9
Does your local dev env also run PHP using WASM, or is it actually a standalone env using a real PHP server? I’m asking because notice that CPU for the tab spikes to 100% when it hangs, which makes me think there is more at play here:
No, my local env is a PHP server inside Docker, that's why I think it's unrelated to WordPress Playground / WASM.
Also, could you check how Safari behaves with your local setup?
I'm on Chrome OS, so can't easily check that. But maybe someone else from my team can.
@adamsilverstein commented on PR #8131:
4 months ago
#10
This looks great, nice work! 🎉
A benefit of the approach explored here is that it could possibly be applied in the same way for almost any WordPress theme
This point makes me wonder if we could build this capability into core as an API themes could opt into easily, for example by opting into theme support (add_theme_support).
@flixos90 commented on PR #8131:
4 months ago
#11
@adamsilverstein
This point makes me wonder if we could build this capability into core as an API themes could opt into easily, for example by opting into theme support (add_theme_support).
That's exactly what I have in mind, planning to explore that next week!
@flixos90 commented on PR #8131:
4 months ago
#12
@adamsilverstein See https://github.com/WordPress/wordpress-develop/pull/8370
8026.patch does not result in 100% validation due to things like
autocomplete="off"
, duplicate nonce ID's, etc. but besides those kinds of things, everything should be mostly valid.I was unable to validate the dashboard due to a UTF-8 issue coming from the "Other WordPress News" widget. Not sure how to fix that.
Navigation bar has duplicate IDs if a plugin needs updating or a comment is in moderation. Will leave that to ryan or someone else who's already been handling the menu.