Make WordPress Core

Opened 17 years ago

Closed 17 years ago

Last modified 4 months ago

#8026 closed defect (bug) (fixed)

2.7 Admin Validation Fixes

Reported by: viper007bond's profile Viper007Bond 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)

8026.patch (3.6 KB) - added by Viper007Bond 17 years ago.
8026.2.patch (1.5 KB) - added by Viper007Bond 17 years ago.
upload.php?detached=1

Download all attachments as: .zip

Change History (14)

@Viper007Bond
17 years ago

#1 @Viper007Bond
17 years ago

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.

#2 @westi
17 years ago

  • Keywords dev-reviewed added

Looks good
+1

#3 @ryan
17 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [9456]) Validation fixes from Viper007Bond. fixes #8026

@Viper007Bond
17 years ago

upload.php?detached=1

#4 @Viper007Bond
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.

#5 @azaozz
17 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [9470]) Fix validation errors, add CDATA on inline JS, props Viper007Bond, fixes #8026

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.

bramus commented on PR #8131:


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:

https://github.com/user-attachments/assets/4e2720cf-1a23-4343-9a57-f15932be77dc

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!

Note: See TracTickets for help on using tickets.