Skip to content

Conversation

@jkphl
Copy link
Contributor

@jkphl jkphl commented Dec 30, 2013

Hi James,

as discussed earlier (see issue #4) I added a new implementation using an additional <span> element to avoid absolute positioning on the page. Unfortunately, I don't have the time right now to really test it with a lot of different clients, but it works in IE9 (which is my primary target platform) and should work in most others as well, as the differences to your original span implementation are only marginal.

Let me know if it works for you as well!

Cheers,
Joschi

@kbjr
Copy link
Member

kbjr commented Dec 30, 2013

Great, I'll take a look.

@kbjr
Copy link
Member

kbjr commented Jan 8, 2014

Sorry for taking so long to back on this. Can you update the readme.md file with a new section like those for the other implementations?

@jkphl
Copy link
Contributor Author

jkphl commented Jan 8, 2014

Sure, I will take care of that tomorrow! (~12 p.m. over here right now)

@jkphl
Copy link
Contributor Author

jkphl commented Jan 9, 2014

Hi James, I added a readme section and demo for span-wrapper.js. The download and demo links in the readme should work but of course I could not test it — please take care of this yourself. Also, please correct my English if there's something wrong. Thanks! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants