Ticket #20238 (closed PLIP: fixed)

Opened 14 months ago

Last modified 13 months ago

Remove IE8 support on Plone

Reported by: ramon Owned by: ramon
Priority: minor Milestone: 5.0
Component: Unknown Version:
Keywords: Cc:

Description (last modified by ramon) (diff)

Proposer: Ramon Navarro Seconder:

Abstract

Plone bundle has tons of resources that are loaded throw css, all this resources means requests and makes really difficult to compile them (as they are relative resources and copying all the resources as mockup did is a huge task to maintain). The out of the box plone-compiled.css will not be touched on most of the cases so it would be great that is as much as optimized as possible. The idea of this plip is to propose to use url to data-url pluggin on less to include on the css files all the file resources that are needed. The problem of this technology is that is not supported on IE8.

Motivation

  • Reducing the amount of requests for each view
  • Avoiding the need to copy/take care of relative paths on css

Assumptions

Proposal & Implementation

Use less url to data-uri pluggin on grunt compilation of the plone bundle

Deliverables

plone-compiled.css

Risks

IE8 will not work properly as it only allows 32Kb of max resource and there is some fonts that are bigger than 32Kb.

In case IE8 needs to be supported we can deliver two css with a conditional, that means we need to add an option to bundles to check if they can be data-uri converted.

In case IE8 needs to be supported we should take care about barceloneta theme as its bootstrap media querys does not work so we need to include  https://github.com/scottjehl/Respond to plone bundle.

More information about bootstrap and IE8 :  http://getbootstrap.com/getting-started/#support-ie8-ie9

Participants

Ramon Navarro Bosch

Progress

Its implemented and pushed to master, it can be reverted easy as its only a matter to change the grunt file and recompile the bundled css version for plone.

 https://github.com/plone/Products.CMFPlone/blob/master/Products/CMFPlone/static/plone-compiled.css

Change History

comment:1 Changed 14 months ago by ramon

  • Description modified (diff)

comment:2 Changed 13 months ago by ramon

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.