Skip to content

[typescript] dom definitions not up to date #19040

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

Open
mjbvz opened this issue Oct 9, 2017 · 8 comments
Open

[typescript] dom definitions not up to date #19040

mjbvz opened this issue Oct 9, 2017 · 8 comments
Labels
Bug A bug in TypeScript Domain: lib.d.ts The issue relates to the different libraries shipped with TypeScript Effort: Casual Good issue if you're already used to contributing to the codebase. Harder than "good first issue". Help Wanted You can do this VS Code Tracked There is a VS Code equivalent to this issue
Milestone

Comments

@mjbvz
Copy link
Contributor

mjbvz commented Oct 9, 2017

From @yukulele on October 5, 2017 12:7

  • VSCode Version: 1.16.1
  • OS Version: macOS Sierra 10.12.6

Steps to Reproduce:

  • Create this code in typescript file
const elm = document.createElement('main')
  • elm is typed as HTMLElement instead of HTMLMainElement

lib.dom.d.ts is not up to date

how to update it without wait for vscode update ?

Reproduces without extensions: Yes

Copied from original issue: microsoft/vscode#35620

@mjbvz mjbvz self-assigned this Oct 9, 2017
@mjbvz mjbvz removed their assignment Oct 9, 2017
@mjbvz mjbvz added Domain: lib.d.ts The issue relates to the different libraries shipped with TypeScript VS Code Tracked There is a VS Code equivalent to this issue and removed typescript labels Oct 9, 2017
@mhegazy mhegazy added this to the Community milestone Oct 9, 2017
@mhegazy mhegazy added the Help Wanted You can do this label Oct 9, 2017
@mhegazy
Copy link
Contributor

mhegazy commented Oct 9, 2017

PRs welcomed. You can find more information about contributing lib.d.ts fixes at https://github.com/Microsoft/TypeScript/blob/master/CONTRIBUTING.md#contributing-libdts-fixes.

@mhegazy mhegazy added the Bug A bug in TypeScript label Oct 9, 2017
@jamOne-
Copy link

jamOne- commented Oct 10, 2017

Can I fix this one as my first contribution to TypeScript?

@SimonSchick
Copy link

@jamOne-
Copy link

jamOne- commented Oct 11, 2017

I already know it, but I don't think this change is that simple.
As far as I understand this generator code, to generate interface HTMLElementTagNameMap in dom.generated.d.ts only browser.webidl.xml interfaces are used, but HTMLMainElement is in addedTypes.json file so I have to change the generator code, or am I wrong?

@SimonSchick
Copy link

Pretty sure you can just add overload to addedTypes.json.

@jamOne-
Copy link

jamOne- commented Oct 11, 2017

I am not sure I understand you. Does generating interface HTMLElementTagNameMap take into account addedTypes.json?

@inad9300
Copy link

Please, have a look at the on-going progress before proceeding: #17828

@RyanCavanaugh RyanCavanaugh modified the milestones: Community, Backlog Mar 7, 2019
@mjbvz mjbvz added the Effort: Casual Good issue if you're already used to contributing to the codebase. Harder than "good first issue". label Oct 13, 2021
@yukulele
Copy link

The following elements are affected: abbr address article aside b bdi bdo cite code dd dfn dt em figcaption figure footer header hgroup html i kbd main mark nav noscript rp rt ruby s samp section small strong sub summary sup u var wbr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug in TypeScript Domain: lib.d.ts The issue relates to the different libraries shipped with TypeScript Effort: Casual Good issue if you're already used to contributing to the codebase. Harder than "good first issue". Help Wanted You can do this VS Code Tracked There is a VS Code equivalent to this issue
Projects
None yet
Development

No branches or pull requests

7 participants