Skip to content

HTMLElement.setAttribute's parameters are marked optional; they are not #4407

Closed
microsoft/TypeScript-DOM-lib-generator
#29

Description

lib.d.ts defines HTML.setAttribute with a signature of (name?: string, value?: string). Neither of the parameters are actually optional and will cause runtime errors on at least Chrome and Firefox.

Activity

  1. changed the title [-]HTMLElement.setAttribute's parameters are marked optional; it is not[/-] [+]HTMLElement.setAttribute's parameters are marked optional; they are not[/+] on Aug 21, 2015
  2. mhegazy commented on Dec 8, 2015

    @mhegazy
    Contributor

    PRs are welcomed. here is some documentation for adding new types to lib.d.ts: https://github.com/Microsoft/TypeScript/blob/master/CONTRIBUTING.md#contributing-libdts-fixes

  3. modified the milestones: , TypeScript 1.8 on Dec 8, 2015
  4. LPGhatguy commented on Dec 9, 2015

    @LPGhatguy
    ContributorAuthor

    I've submitted a PR (as noted just above this) to the TSJS-lib-generator project.

  5. modified the milestones: TypeScript 1.8, on Dec 9, 2015
  6. locked and limited conversation to collaborators on Jun 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    BugA bug in TypeScriptDomain: lib.d.tsThe issue relates to the different libraries shipped with TypeScriptFixedA PR has been merged for this issueHelp WantedYou can do this

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions