Menu

#15 incorrect background image

open
nobody
None
5
2008-08-11
2008-08-10
No

Using the css files for Ext (found at ExtJS.com), when merging two files (their way of changing themes), the background images get messed up. See the example below:

.x-tab-strip-top .x-tab-right{
background:transparent url(../images/default/tabs/tabs-sprite.gif) no-repeat 0 -51px;
padding-left:10px;
}

with

.x-tab-strip-top .x-tab-right
{
background-image:url(../images/gray/tabs/tabs-sprite.gif);
}

results in

.x-tab-strip-top .x-tab-right{
background:transparent url(../images/default/tabs/tabs-sprite.gif) no-repeat 0 -51px;
padding-left:10px;
background-image:url(../images/gray/tabs/tabs-sprite.gif);
}

Which is obviously inefficient -- it has two conflicting background images, instead of one. It should be

.x-tab-strip-top .x-tab-right{
background:transparent url(../images/gray/tabs/tabs-sprite.gif) no-repeat 0 -51px;
padding-left:10px;
}

Discussion

  • Steven Roussey

    Steven Roussey - 2008-08-10
    • status: open --> open-invalid
     
  • Steven Roussey

    Steven Roussey - 2008-08-10

    Logged In: YES
    user_id=122078
    Originator: YES

    Use set_cfg('optimise_shorthands',2)

     
  • Steven Roussey

    Steven Roussey - 2008-08-11

    Logged In: YES
    user_id=122078
    Originator: YES

    Using set_cfg('optimise_shorthands',2) is dangerous (unfinished) thus why it is not enabled by default. Unfortunately, using this setting causes many display errors.

    With the ExtJS.com example of combining these two files:
    http://extjs.com/deploy/dev/resources/css/ext-all.css
    http://extjs.com/deploy/dev/resources/css/xtheme-gray.css

    will result in many errors in the examples.

     
  • Steven Roussey

    Steven Roussey - 2008-08-11
    • status: open-invalid --> open
     

Log in to post a comment.