Uploaded image for project: 'SmartSprites'
  1. SmartSprites
  2. SMARTSPRITES-22

Add support for shorthand background property

    Details

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

      Description

      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;

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                stachoo Stanisław Osiński
                Reporter:
                redclover Philipp Rosenhagen
              • Votes:
                4 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated: