Modify ↓
Ticket #12957 (closed feature: fixed)
Improve wrapMap
| Reported by: | markel | Owned by: | markel |
|---|---|---|---|
| Priority: | low | Milestone: | 1.9 |
| Component: | manipulation | Version: | 1.8.3 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
In most cases end tags are unnecessary and their absence make jQuery.buildFragment even more consistent across browsers, when jQuery.clean takes incorrect HTML argument as a string.
Change History
comment:2 Changed 6 months ago by markel
@rwaldron If i add tests it would mean that jQuery can take "incorrect HTML argument as a string", which is not true.
comment:3 Changed 6 months ago by timmywil
- Status changed from new to assigned
- Component changed from unfiled to manipulation
- Priority changed from undecided to low
- Version changed from git to 1.8.3
- Milestone changed from None to 1.9
- Owner set to markel
comment:4 Changed 6 months ago by Oleg
- Status changed from assigned to closed
- Resolution set to fixed
Fix #12957. Simplify wrapMap, it doesn't need end tags. Close gh-1044.
Changeset: 9ae6b1a019553e95a6205d5c42ff7fa25e7a482e
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.
In the PR provided, I don't see any proof of "more consistent across browsers". Some tests would be ideal, as what I _am_ seeing is the removal of a test, but no additions.