Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Medium
    • Resolution: Fixed
    • Affects Version/s: 2.6.0
    • Fix Version/s: 2.8
    • Component/s: None
    • Last commented by user?:
      true

      Description

      First, the main-action CSS has this:

      .wiki-content ol {
          list-style-type: decimal;
          padding: 0;
          padding-left: 22px;
          margin: 10px 0;
      }
      
      .wiki-content ol ol {
          list-style-type: lower-alpha;
          margin: 0;
      }
      
      .wiki-content ol ol ol {
          list-style-type: lower-roman;
      }
      
      .wiki-content ol ol ol ol {
          list-style-type: decimal;
      }
      
      .wiki-content ol ol ol ol ol {
          list-style-type: lower-alpha;
      }
      
      .wiki-content ol ol ol ol ol ol {
          list-style-type: lower-roman;
      }
      
      .wiki-content ol ol ol ol ol ol ol {
          list-style-type: decimal;
      }
      
      .wiki-content ol ol ol ol ol ol ol ol {
          list-style-type: lower-alpha;
      }
      
      .wiki-content ol ol ol ol ol ol ol ol ol {
          list-style-type: lower-roman;
      }
      
      .wiki-content ol ol ol ol ol ol ol ol ol ol {
          list-style-type: decimal;
      }
      
      .wiki-content ol ol ol ol ol ol ol ol ol ol ol {
          list-style-type: lower-alpha;
      }
      
      .wiki-content ol ol ol ol ol ol ol ol ol ol ol ol {
          list-style-type: lower-roman;
      }
      

      Then a bit further down, it has this:

      .mceContentBody ol {
          list-style-type: decimal;
          padding: 0;
          padding-left: 22px;
          margin: 10px 0;
      }
      
      .mceContentBody ol ol {
          list-style-type: lower-alpha;
          margin: 0;
      }
      
      .mceContentBody ol ol ol {
          list-style-type: lower-roman;
      }
      
      .mceContentBody ol ol ol ol {
          list-style-type: decimal;
      }
      
      .mceContentBody ol ol ol ol ol {
          list-style-type: lower-alpha;
      }
      
      .mceContentBody ol ol ol ol ol ol {
          list-style-type: lower-roman;
      }
      
      .mceContentBody ol ol ol ol ol ol ol {
          list-style-type: decimal;
      }
      
      .mceContentBody ol ol ol ol ol ol ol ol {
          list-style-type: lower-alpha;
      }
      
      .mceContentBody ol ol ol ol ol ol ol ol ol {
          list-style-type: lower-roman;
      }
      
      .mceContentBody ol ol ol ol ol ol ol ol ol ol {
          list-style-type: decimal;
      }
      
      .mceContentBody ol ol ol ol ol ol ol ol ol ol ol {
          list-style-type: lower-alpha;
      }
      
      .mceContentBody ol ol ol ol ol ol ol ol ol ol ol ol {
          list-style-type: lower-roman;
      }
      

      Then a bit further down it has this:

      div#preview ol {
          list-style-type: decimal;
          padding: 0;
          padding-left: 22px;
          margin: 10px 0;
      }
      
      div#preview ol ol {
          list-style-type: lower-alpha;
          margin: 0;
      }
      
      div#preview ol ol ol {
          list-style-type: lower-roman;
      }
      
      div#preview ol ol ol ol {
          list-style-type: decimal;
      }
      
      div#preview ol ol ol ol ol {
          list-style-type: lower-alpha;
      }
      
      div#preview ol ol ol ol ol ol {
          list-style-type: lower-roman;
      }
      
      div#preview ol ol ol ol ol ol ol {
          list-style-type: decimal;
      }
      
      div#preview ol ol ol ol ol ol ol ol {
          list-style-type: lower-alpha;
      }
      
      div#preview ol ol ol ol ol ol ol ol ol {
          list-style-type: lower-roman;
      }
      
      div#preview ol ol ol ol ol ol ol ol ol ol {
          list-style-type: decimal;
      }
      
      div#preview ol ol ol ol ol ol ol ol ol ol ol {
          list-style-type: lower-alpha;
      }
      
      div#preview ol ol ol ol ol ol ol ol ol ol ol ol {
          list-style-type: lower-roman;
      }
      

      There are lots of other chunks of CSS relating to list that are repeated once for normal view of pages, then again for RTE then again for preview. Crazy!

      Surely it would be a lot cleaner to just merge the three blocks of CSS as follows:

      .wiki-content ol, .mceContentBody ol, div#preview ol {
          list-style-type: decimal;
          padding: 0;
          padding-left: 22px;
          margin: 10px 0;
      }
      
      etc...
      

      Instantly reduces the size of that CSS from having everything repeated 3 times to just having it in there once. Also makes things significantly easier to maintain as the CSS will appear in one place rather than several.

      This same scenario exists throughout the main-action.css - identical CSS is included 3 or more times for different views of the same thing. You could probably halve the size of main-action.css by merging such duplications.

      While making main-action.css smaller might not seem like a worthwhile task, we've got clients delivering content via Confluence to rural India (e-Consult who provide non-profit telemedicine services) and throughout Africa (Widernet and eGranary) - bandwidth is a very major issue in such circumstances. We also have clients delivering content throughout a medical ship where there are some bandwidth issues with microwave links, etc.

        Attachments

          Activity

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Last commented:
                10 years, 17 weeks, 2 days ago