Skip to main content

Bug Tracker

Side navigation

#9634 closed bug (fixed)

Opened June 21, 2011 09:07AM UTC

Closed August 04, 2011 09:15PM UTC

Last modified March 09, 2012 05:34AM UTC

visible "t" character in upper left corner during support.js init

Reported by: aet@aet.fi Owned by: rwaldron
Priority: high Milestone: 1.6.3
Component: support Version: 1.6.1
Keywords: Cc:
Blocked by: Blocking:
Description

Hi,

Ever since jQuery 1.4.3 release, there's been the following line

in support.js:

div.innerHTML = "<table><tr><td style='padding:0;display:none'></td><td>t</td></tr></table>";

Does the visible t character have a specific purpose for the tests? I'm implementing

a mobile html5 offline app and when the app gets reloaded, it shows the

character on the screen, causing minor annoyance

Thanks,

-Antti

Attachments (0)
Change History (14)

Changed June 21, 2011 03:44PM UTC by rwaldron comment:1

owner: → jaubourg
status: newassigned

reassigning to jaubourg

Julian, can you take a look at this?

Changed June 29, 2011 07:12PM UTC by timmywil comment:2

component: unfiledsupport
priority: undecidedhigh

Changed June 29, 2011 10:14PM UTC by jaubourg comment:3

I thought we had this fixed in an earlier release? What changed?

Changed July 12, 2011 06:24PM UTC by john comment:4

owner: jaubourgaet@aet.fi
status: assignedpending

We're not seeing this on our end. Can you provide a test case or more specifics regarding the browser?

Changed July 13, 2011 08:04AM UTC by aet@aet.fi comment:5

status: pendingnew

I've seen this behaviour with webkit (Safari 5, Chrome, iOS 4.3.x) but it's very rare and I barely get to see the character before it vanishes.

No proper test case in hand atm, I've seen this on lots of pages that generate all

html/css code dynamically after $(document).ready and json loads.

I've disabled the 't' character from my own builds for now. Can you confirm that the character isn't a typo?

Changed July 25, 2011 04:24PM UTC by rwaldron comment:6

status: newpending

Is this still an issue?

Changed July 25, 2011 05:22PM UTC by aet@aet.fi comment:7

status: pendingnew

Yes, could someone please confirm that <td>t</td> isn't just a typo?

Changed July 25, 2011 05:54PM UTC by rwaldron comment:8

owner: aet@aet.firwaldron
status: newassigned

No, it's not a typo. The comments in the source explain it's existence, though I think we can use a non-breaking-space to ensure the td doesn't collapse

Changed July 25, 2011 10:57PM UTC by rwaldron comment:9

Changed August 04, 2011 09:15PM UTC by Dave Methvin comment:10

resolution: → fixed
status: assignedclosed

Merge pull request #444 from rwldrn/9634

Non-breaking-space should achieve the same effect as a visible character. Fixes #9634

Changeset: ea0303ecbf8e6b8a5c39bbad463e3e0d1dc84543

Changed August 26, 2011 02:55AM UTC by dmethvin comment:11

milestone: 1.next1.6.3

Changed August 31, 2011 02:36PM UTC by Rick Waldron comment:12

Landing pull request 478. Apply fixes as noted by Diego Perini, David Murdoch & Mathias Bynens. Supplements #9634 Fixes #9634.

More Details:

Changeset: 5d899651713c9027f3368b33a6995761a644111b

Changed August 31, 2011 02:41PM UTC by timmywil comment:13

#10180 is a duplicate of this ticket.

Changed September 01, 2011 03:17PM UTC by Rick Waldron comment:14

Landing pull request 478. Apply fixes as noted by Diego Perini, David Murdoch & Mathias Bynens. Supplements #9634 Fixes #9634.

More Details:

Changeset: a42273807d2b1b1ff1362c1b9fc4cb48f65fda11