Click
  1. Click
  2. CLK-363

Allow Page htmlImports to override Control imports

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.5 M2
    • Fix Version/s: 1.5 M3
    • Component/s: None
    • Labels:
      None

      Description

      Starting with 1.5-M1, Page allows us to import JS and CSS.

      It would be nice if Page could also expose a boolean property to inform PageImports not to include the Controls JS and CSS.

      Reason: Rule #1 of Yahoo Performance Rules states that the making connections to the server is expensive. If a Page imports 10 CSS and 10 JS, the browser has to make 20 connections to the server. For a sites HomePage it would be great to optimize this to only 1 CSS and 1 JS. Libraries such as YUICompressor allows one to merge and compress all the JS and CSS into a single file that can be served from the Page. By allowing the Page to override the Control imports, one could have the Page import the single monolithic CSS and JS file, and skip the Control imports. Thus making for a much faster Page experience.

        Activity

        Hide
        Malcolm Edgar added a comment -

        Improvement checked in to enable optimized page imports.

        This class still needs further work to improve the parsing of import lines, it is currently very simple and not tollerant of simple formatting mistakes.

        Show
        Malcolm Edgar added a comment - Improvement checked in to enable optimized page imports. This class still needs further work to improve the parsing of import lines, it is currently very simple and not tollerant of simple formatting mistakes.
        Hide
        Bob Schellink added a comment -

        Page could expose a property "overrideControlImports" which indicates that Page will include all Control imports in an optimized way.

        Moving this issue to 1.5-M3

        Show
        Bob Schellink added a comment - Page could expose a property "overrideControlImports" which indicates that Page will include all Control imports in an optimized way. Moving this issue to 1.5-M3

          People

          • Assignee:
            Malcolm Edgar
            Reporter:
            Bob Schellink
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development