Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Document.title to match DOM spec (create <title> if missing). #29

Merged
merged 1 commit into from
Apr 9, 2013

Conversation

cscott
Copy link
Collaborator

@cscott cscott commented Apr 6, 2013

domino would do a no-op for "document.title = ...." if there wasn't already a <title> element in the document's . Fix the behavior to match the DOM spec (http://www.whatwg.org/specs/web-apps/current-work/#the-title-element-0).

cscott added a commit that referenced this pull request Apr 9, 2013
Fix Document.title to match DOM spec (create <title> if missing).
@cscott cscott merged commit a654d65 into fgnass:master Apr 9, 2013
wmfgerrit pushed a commit to wikimedia/mediawiki-services-parsoid that referenced this pull request Jul 16, 2014
 * Removes the hotfix file for domino. Was only being used in some
   places (a potential source of errors) and considering cscott lives
   upstream, it shouldn't be a problem getting fixes merged. Also, we
   seem to be using the "fork and put a link to a git repo in the
   package.json" pattern in other cases.

 * The bug being worked around in the PostProcessor was fixed by:
     fgnass/domino#29

 * Some other minor cleanup.

Change-Id: I0968e1eeca05e07f1ddd736a99acd7a88c5e610d
cscott added a commit that referenced this pull request Feb 14, 2018
We can't make direct array access (`node.attributes[n]`) work without
Proxy support (see gh #29) but by defining a custom iterator we can
make `Array.from` and for-of loops work on `node.attributes`.
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.

None yet

1 participant