Webpack Roadmap

What

We got webpack in and that is great. But we can't take full advantage until we remove global code.

After a chat with @jschatz1, we decided to make a roadmap to allow for things to get scheduled. This issue is done in collaboration by @filipa @mikegreiling and @iamphill.

For this to happen we need to divide and conquer. This is not an hard task to do, just a boring one.

We also need to teach. We all use it, but webpack can be hard to understand and there is no shame in that :) We can't force everyone to read full documentation, we could do a small talk explaining the need of webpack and how to use it.

How

1. main.js - Divide and conquer!

2. Shortcuts

   • shortcuts @iamphill
   • ./shortcuts_blob @iamphill
   • ./shortcuts_dashboard_navigation @iamphill
   • ./shortcuts_navigation @iamphill
   • ./shortcuts_find_file @iamphill
   • ./shortcuts_issuable @iamphill
   • ./shortcuts_network @iamphill

3. Templates

   • ./templates/issuable_template_selector @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15524
   • ./templates/issuable_template_selectors @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15524

4. Commit

   • ./commit/image_file

5. Utils

   • ./lib/utils/bootstrap_linked_tabs @filipa !14734 (merged)
   • ./lib/utils/datetime_utility @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15782
   • ./lib/utils/pretty_time @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15459
   • ./lib/utils/text_utility @filipa !15320 (merged)
   • ./lib/utils/url_utility @iamphill !15783 (merged)

6. u2f

   • ./u2f/authenticate @filipa !14776 (merged)
   • ./u2f/error @filipa !14776 (merged)
   • ./u2f/register @filipa !14776 (merged)
   • ./u2f/util @filipa !14776 (merged)

7. Everything else

   • ./abuse_reports; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14709
   • ./activities; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15933
   • ./admin; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15933
   • ./ajax_loading_spinner; @filipa !14738 (merged)
   • ./api; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14774
   • ./aside; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15933
   • ./autosave; @mikegreiling gitlab-org/gitlab-ce!14856
   • ./broadcast_message; @mikegreiling gitlab-org/gitlab-ce!14853
   • ./build; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14744
   • ./build_artifacts; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14744
   • ./build_variables; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14744
   • ./ci_lint_editor; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14744
   • ./commits; @filipa !14735 (merged)
   • ./compare; @mikegreiling gitlab-org/gitlab-ce!14857
   • ./compare_autocomplete; @mikegreiling gitlab-org/gitlab-ce!14857
   • ./confirm_danger_modal; @mikegreiling gitlab-org/gitlab-ce!17874
   • ./copy_as_gfm; @mikegreiling gitlab-org/gitlab-ce!14859
   • ./copy_to_clipboard; @mikegreiling
   • ./create_label; @filipa !14738 (merged)
   • ./diff; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14965
   • ./dropzone_input; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14894
   • ./due_date_select; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14881
   • ./files_comment_button; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14965
   • ./flash;
   • ./gl_dropdown; @jschatz1 https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15992
   • ./gl_field_error; @iamphill
   • ./gl_field_errors; @iamphill
   • ./gl_form; @iamphill
   • ./group_avatar; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15006
   • ./group_label_subscription; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15006
   • ./groups_select; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15008
   • ./header; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15090
   • ./importer_status; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15058
   • ./issuable_index; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15058
   • ./issuable_context; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15058
   • ./issuable_form; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15058
   • ./issue; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15090
   • ./issue_status_select; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15090
   • ./label_manager; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14794
   • ./labels; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14794
   • ./labels_select; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15091?w=1
   • ./layout_nav; @iamphill https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/3829
   • ./line_highlighter; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15960
   • ./logo; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15091?w=1
   • ./member_expiration_date; @filipa !14808 (merged)
   • ./members; @filipa !14808 (merged)
   • ./merge_request; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15960
   • ./merge_request_tabs; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15952
   • ./milestone; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15503/
   • ./milestone_select; @okoghenun !16076 (merged)
   • ./mini_pipeline_graph_dropdown; @filipa !14734 (merged)
   • ./namespace_select;
   • ./new_branch_form; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15503/
   • ./new_commit_form; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15503/
   • ./notes; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15953
   • ./notifications_dropdown; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15952
   • ./notifications_form; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15952
   • ./pager; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15952
   • ./pipelines; @filipa !14734 (merged)
   • ./preview_markdown; @jschatz1 https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/16025
   • ./project; @filipa !15355 (merged)
   • ./project_avatar; @filipa !15355 (merged)
   • ./project_find_file; @filipa !15787 (merged)
   • ./project_import; @filipa !15355 (merged)
   • ./project_label_subscription; @filipa !15356 (merged)
   • ./project_new; @filipa !15356 (merged)
   • ./project_select; @filipa !15356 (merged)
   • ./project_show; @filipa !15356 (merged)
   • ./project_variables; @filipa !15356 (merged)
   • ./projects_dropdown; @jschatz1
   • ./projects_list; @filipa !15787 (merged)
   • ./syntax_highlight; @filipa !15787 (merged)
   • ./render_math;
   • ./render_gfm; @jschatz1 https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15993
   • ./right_sidebar; @iamphill https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15960
   • ./search; @filipa !15787 (merged)
   • ./search_autocomplete; @filipa !15787 (merged)
   • ./smart_interval; @psimyn
   • ./star; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14795
   • ./subscription;
   • ./subscription_select; @filipa https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/15503/

8. Export gl utility functions as modules - https://gitlab.com/gitlab-org/gitlab-ce/issues/37220 ✅

9. Spies in gl functions - https://gitlab.com/gitlab-org/gitlab-ce/issues/30998

10. Dispacther - https://gitlab.com/gitlab-org/gitlab-ce/issues/37792

11. Docs! Let's document everything so everyone can use it! - https://gitlab.com/gitlab-org/gitlab-ce/issues/38173

12. Memory - https://gitlab.com/gitlab-org/gitlab-ce/issues/32893

13. Logs - https://gitlab.com/gitlab-org/gitlab-ce/issues/34348

14. Speed up webpack dev server - https://gitlab.com/gitlab-org/gitlab-ce/issues/35305

Followup: https://gitlab.com/gitlab-org/gitlab-ce/issues/41290