SmartSprites

Add support for shorthand background property

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Priority: Critical Critical
  • Resolution: Unresolved
  • Affects Version/s: 0.2.0
  • Fix Version/s: 0.4.0
  • Component/s: None
  • Description:
    Hide

    Renamed the original improvement request to what it became after the discussions shown in comments. Original description follows below.

    Currently only the "background-image" css property is allowed for SmartSprites annotations/directives.
    But the designer should not be forced to use separate background-* properties to define the background style. Moreover it's more or less impossible to modify/correct every css file which come with some libraries and don't follow the current rule of SmartSprites.
    And it's more efficient to use the single "background" property and compose the different options in there like:

    background: #6891C9 url(../images/background.jpg) repeat-y; /** sprite-ref: mysprite; */

    instead of 3 separate definitions:

    background-color: #6891C9;
    background-repeat: repeat-y;
    background-image: url(../images/background.jpg); /** sprite-ref: mysprite; */

    Currently when trying the first example the ant-tasks spits out a warning: "No 'background-image' CSS rule next to sprite reference comment..."

    And here is another property which should be supported:

    list-style-type: none;
    list-style-position: outside;
    list-style-image: url(../images/pdf-icon16x16.gif);

    again: the shortened property should also be supported:

    list-style: none url(../images/pdf-icon16x16.gif) outside;

    Show
    Renamed the original improvement request to what it became after the discussions shown in comments. Original description follows below. — Currently only the "background-image" css property is allowed for SmartSprites annotations/directives. But the designer should not be forced to use separate background-* properties to define the background style. Moreover it's more or less impossible to modify/correct every css file which come with some libraries and don't follow the current rule of SmartSprites. And it's more efficient to use the single "background" property and compose the different options in there like: background: #6891C9 url(../images/background.jpg) repeat-y; /** sprite-ref: mysprite; */ instead of 3 separate definitions: background-color: #6891C9; background-repeat: repeat-y; background-image: url(../images/background.jpg); /** sprite-ref: mysprite; */ Currently when trying the first example the ant-tasks spits out a warning: "No 'background-image' CSS rule next to sprite reference comment..." And here is another property which should be supported: list-style-type: none; list-style-position: outside; list-style-image: url(../images/pdf-icon16x16.gif); again: the shortened property should also be supported: list-style: none url(../images/pdf-icon16x16.gif) outside;

Issue Links

Activity

No work has yet been logged on this issue.

People

Dates

  • Created:
    02/Oct/08 02:26 PM
    Updated:
    26/Jul/09 10:05 PM