Skip to content

jsdoc mouse hover definition {Array.<static class>} bug #33455

Open
Open
jsdoc mouse hover definition {Array.<static class>} bug#33455
Labels
Experience EnhancementNoncontroversial enhancementsSuggestionAn idea for TypeScript
Milestone
Backlog

Description

@jonlepage
jon lepage (jonlepage)
opened on Sep 16, 2019

TypeScript Version: TS-next-3.7.0-2019.09.11

hi, if we add a definition jsdoc with static class array like this
/**@type {Array.<_A._B>} */
Mouse hover show any. And am not seem able to get link from vscode.
So i can't use ctrl+click to get definitions and travel in my codes.

I don't know if is a bug related to vscode engine or a mistake from my side?
Try this code and hover {Array.<_A._B>}

class _A {
    constructor() {
        /**@type {Array.<_A._B>} */
        this.test = null;
        //test intelisence
        this.test[0].TESTT
    };

    static _B = (function() {
        return class _B {
            constructor() {
                this.TESTT = true;
                
             };
        };
    })();

    static _C = class _C {
            constructor() {
                this.TESTT = true;
                
             };
        };

};

bug jsdoc

from suggest, it can also maybe related here.
microsoft/vscode#17912

thanks in advance

Activity

  1. Matt Bierner (mjbvz)
    self-assigned this
    on Sep 16, 2019
  2. Matt Bierner (mjbvz)
    transferred this issue frommicrosoft/vscodeon Sep 16, 2019

  3. mjbvz commented on Sep 16, 2019

    @mjbvz
    Matt Bierner (mjbvz)
    on Sep 16, 2019

    Tested on 37-20190911. IntelliSense seems to work but hover in intellisense doesn't

  4. Ryan Cavanaugh (RyanCavanaugh)
    added
    Experience EnhancementNoncontroversial enhancements
    SuggestionAn idea for TypeScript
    on Sep 17, 2019
  5. Ryan Cavanaugh (RyanCavanaugh)
    added this to the Backlog milestone on Sep 17, 2019
  6. Ryan Cavanaugh (RyanCavanaugh)
    unassigned
    Matt Bierner (mjbvz)
    on Sep 17, 2019

  7. RyanCavanaugh commented on Sep 17, 2019

    @RyanCavanaugh
    Ryan Cavanaugh (RyanCavanaugh)
    on Sep 17, 2019
    Member

    Classes aren't a recognized form of namespace, but we could do this in JS.

    I'm not sure the IIFE initializer for B will ever be correctly detected, though.

  8. jonlepage commented on Oct 23, 2019

    @jonlepage
    jon lepage (jonlepage)
    on Oct 23, 2019
    Author

    i just want link another thread can be related.
    I think
    image
    Classes imbrication only work in constructor.
    But static classes not working and not showing correctly.
    #33715

    I think the good english name of this issue is nested-es6-classes.
    But am not sure.
    https://stackoverflow.com/questions/28784375/nested-es6-classes

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

    Experience EnhancementNoncontroversial enhancementsSuggestionAn idea for TypeScript

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions