#21018 closed defect (bug) (fixed)
get_the_password_form() inserting two </p> before the </form> causing W3C validation error
| Reported by: | kevinhaig | Owned by: | nacin |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.4.1 |
| Component: | Formatting | Version: | 3.4 |
| Severity: | normal | Keywords: | has-patch needs-unit-tests |
| Cc: | Focuses: |
Description
In the get_the Password_form()code at the end of the function :
esc_attr__("Submit") . '" /></p>
</form>
';
should be
esc_attr__("Submit") . '" /></p></form>';
Attachments (1)
Change History (7)
#3
@
14 years ago
Let's remove the extra tab before </form> to fix this in the 3.4 branch. We can look into wpautop() in 3.5, with unit tests.
#6
in reply to: ↑ 1
@
14 years ago
- Milestone Awaiting Review → 3.4.1
Replying to duck_:
I thought that we had fixed this in #16456.
After some quick investigation it appears that the problem is that the spacing before the closing </form> tag is being captured in paragraph tags. Removing <p> tags from around block-level elements first fixes this. However, it will need a lot more testing.
Not entirely sure the best way to turn this into a bug report. duck_, can you open a new ticket for 3.5?
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
I thought that we had fixed this in #16456.
After some quick investigation it appears that the problem is that the spacing before the closing </form> tag is being captured in paragraph tags. Removing <p> tags from around block-level elements first fixes this. However, it will need a lot more testing.