Skip to content

Updated to be compatible with XHTML and HTML5 polyglot documents#6

Open
kabili207 wants to merge 2 commits intoscottycc:masterfrom
kabili207:master
Open

Updated to be compatible with XHTML and HTML5 polyglot documents#6
kabili207 wants to merge 2 commits intoscottycc:masterfrom
kabili207:master

Conversation

@kabili207
Copy link
Copy Markdown

I have modifed coin.js to fix issues that prevented CoinWidget from working with documents served as application/xhtml+xml. This will partially fix #2. I did not implement my changes as a jQuery plugin as suggested by the bug reporter due to the time that would be required to do so.

I also added ALT attributes to the IMG tags to make the widget a little more accessible to people with screen readers.

sebaslogen added a commit to sebaslogen/bitcoin-price-ticker that referenced this pull request Feb 9, 2014
@chezinut
Copy link
Copy Markdown

merged to https://github.com/chezinut/coinwidget

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.

Remove document.write

2 participants