Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-92876

Dom: model JS expressions/statements in QmlDom (to complement QQmlJSScope's information)

    XMLWordPrintable

Details

    • Task
    • Resolution: Unresolved
    • P2: Important
    • 6.8, 6.x
    • None
    • QML: Tooling
    • 89b096041 (dev), be92908ac (dev), 1c566c512 (dev), fba9a80d4 (dev), a9e821d6a (dev), 4a1559862 (dev), 447fbe918 (dev), 091191853 (dev), af6234be9 (dev), 087a988ef (dev), ed131e936 (dev), 81f3dca10 (dev), 83f3213ba (dev), c98719b8c (dev), 70977b0d1 (dev)

    Description

      QmlJSScope supports js scopes and usages, it should be cleanly integrated in the Dom, to give access to declarations and usages in QML code. As QQmlJSScope itself is not enough to implement all the LSP-features, model JS statements and expressions in the DOM to complement its information.

      For example, implementing findUsages in qmlls cannot be done only with QQmlJSScope's, as the semantic scopes contain the definitions of JS identifiers but not their usages in the code. Also, the current JS-reformatting works directly on the AST-classes used by the Parser and the DOM for JS statements/expressions might leave a door open to move the reformatting implementation inside the DOM JS classes for later...

      Also, model the JS-stuff in the DOM with their own classes etc.

       

      For FF of 6.6, try first to implement the Dom elements such that the Dom can be constructed (leaving out some information that are not needed for qmlls's find usages feature).

       

      Mostly, all of the JS support for classes can be left out for now, assuming that most of the users are using QML objects instead.

      List of statements/expressions to model in the DOM:

      done

      to be done

      do not do

      in progress

      Attachments

        Issue Links

          For Gerrit Dashboard: QTBUG-92876
          # Subject Branch Project Status CR V

          Activity

            People

              sami.shalayel Sami Shalayel
              fawzi Fawzi Mohamed
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:

                Gerrit Reviews

                  There is 1 open Gerrit change