Details

    • Type: Task
    • Status: Closed
    • Priority: Minor
    • Resolution: Completed
    • Affects Version/s: None
    • Fix Version/s: Master
    • Component/s: Alloy UI
    • Labels:
      None

      Description

      Aim here is to mark liferay-tree-view-icons as deprecated:

      https://github.com/liferay/liferay-portal/blob/master/modules/apps/frontend-js/frontend-js-aui-web/src/main/resources/META-INF/resources/liferay/tree_view_icons.js

      Replaced by the React Treeview component:

      https://github.com/liferay/liferay-portal/blob/master/modules/apps/frontend-js/frontend-js-components-web/test/treeview/Treeview.js

      liferay-tree-view-icons is activated when aui-tree-view is used, which means that in order to effectively deprecate it we need to do three things:

      1. Verify that existing usage sites of aui-tree-view can be ported to React Treeview (this may require changes to the React Treeview, or flexibilization of the requirements where aui-tree-view is being used).
      2. Actually port those aui-tree-view usages sites to React. Sample results for "git grep aui-tree-view": https://gist.github.com/wincent/a87224f87282de5608ebdfc0bd312f3b (note that anything under "modules/apps/archived" can be ignored)
      3. Mark liferay-aui-tree-view-icons as deprecated.

      Note that step "1" is very important: at the time I write this, I haven't even looked at the usage sites, so it is speculation that React Treeview can be used in its current form to replace aui-tree-view; we may have to spin off another task (or tasks) to make adaptations or additions to that API.

        Attachments

          Activity

            People

            Assignee:
            brian.chan Brian Chan
            Reporter:
            greg.hurrell Greg Hurrell (Inactive)
            Engineering Assignee:
            Mateo Mustapić (Inactive)
            Recent user:
            Greg Hurrell (Inactive)
            Participants of an Issue:
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Packages

                Version Package
                Master