Re: [xwiki-devs] [xwiki-notifications] r32928 - platform/skins/trunk/colibri/src/main/resources/colibri
Marius please revert this commit because ie-all means IE6, IE7 and IE8. * hacks are supposed to go for the IE6, IE7 part, while the non-affected property goes for IE8. for example: .xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; } even if you made this clean - you supposed to delete the overridden margin property, but all these properties are needed to display correctly. Thanks, Caty On Wed, Nov 17, 2010 at 09:52, mflorea <[email protected]>wrote:
Author: mflorea Date: 2010-11-17 08:52:49 +0100 (Wed, 17 Nov 2010) New Revision: 32928
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css Log: XSCOLIBRI-251: ie-all.css uses color theme variables without importing the color theme template * Also removed useless * hack since it's a IE dedicated stylesheet.
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css =================================================================== --- platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:43:03 UTC (rev 32927) +++ platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:52:49 UTC (rev 32928) @@ -1,3 +1,4 @@ +#template('colorThemeInit.vm') body { behavior: url("$xwiki.getSkinFile('/htc/csshover3.htc')"); } @@ -7,7 +8,7 @@ }
#docextrapanes { - *display: inline-block; + display: inline-block; }
.minwidth, .minwidthb, .minwidthc, .minwidthd { @@ -29,7 +30,7 @@
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
#commentscontent #AddComment .commentcontainer label { @@ -37,7 +38,7 @@ }
input.button { - *padding: 0 0 1px; + padding: 0 0 1px; }
fieldset.xwikimessage {
_______________________________________________ notifications mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/notifications
On 11/17/2010 11:53 AM, Ecaterina Moraru (Valica) wrote:
Marius please revert this commit because ie-all means IE6, IE7 and IE8.
* hacks are supposed to go for the IE6, IE7 part, while the non-affected property goes for IE8.
Caty, * is not valid before CSS property name. I know IE ignores it but I need to parse the CSS and that * prevents me. We need to find another way (using proper CSS syntax) to write CSS rules for IE6/7 only. Thanks, Marius P.S.: I asked Raluca and Sergiu about the * hack before doing this commit. They seemed to agree that the * is useless in a IE dedicated stylesheet. I guess I haven't explained myself too well.
for example:
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
even if you made this clean - you supposed to delete the overridden margin property, but all these properties are needed to display correctly.
Thanks, Caty
On Wed, Nov 17, 2010 at 09:52, mflorea<[email protected]>wrote:
Author: mflorea Date: 2010-11-17 08:52:49 +0100 (Wed, 17 Nov 2010) New Revision: 32928
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css Log: XSCOLIBRI-251: ie-all.css uses color theme variables without importing the color theme template * Also removed useless * hack since it's a IE dedicated stylesheet.
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css =================================================================== --- platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:43:03 UTC (rev 32927) +++ platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:52:49 UTC (rev 32928) @@ -1,3 +1,4 @@ +#template('colorThemeInit.vm') body { behavior: url("$xwiki.getSkinFile('/htc/csshover3.htc')"); } @@ -7,7 +8,7 @@ }
#docextrapanes { - *display: inline-block; + display: inline-block; }
.minwidth, .minwidthb, .minwidthc, .minwidthd { @@ -29,7 +30,7 @@
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
#commentscontent #AddComment .commentcontainer label { @@ -37,7 +38,7 @@ }
input.button { - *padding: 0 0 1px; + padding: 0 0 1px; }
fieldset.xwikimessage {
_______________________________________________ notifications mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/notifications
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
And what is the solution? to have ie-6.css ie-7.css ie-8.css (maybe this won't be needed if we have ie-7.css because IE8 knows more things) instead of ie-6.css + ie-all.css? Thanks, Caty On Wed, Nov 17, 2010 at 12:57, Marius Dumitru Florea < [email protected]> wrote:
On 11/17/2010 11:53 AM, Ecaterina Moraru (Valica) wrote:
Marius please revert this commit because ie-all means IE6, IE7 and IE8.
* hacks are supposed to go for the IE6, IE7 part, while the non-affected property goes for IE8.
Caty, * is not valid before CSS property name. I know IE ignores it but I need to parse the CSS and that * prevents me. We need to find another way (using proper CSS syntax) to write CSS rules for IE6/7 only.
Thanks, Marius
P.S.: I asked Raluca and Sergiu about the * hack before doing this commit. They seemed to agree that the * is useless in a IE dedicated stylesheet. I guess I haven't explained myself too well.
for example:
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
even if you made this clean - you supposed to delete the overridden
margin
property, but all these properties are needed to display correctly.
Thanks, Caty
On Wed, Nov 17, 2010 at 09:52, mflorea<[email protected] wrote:
Author: mflorea Date: 2010-11-17 08:52:49 +0100 (Wed, 17 Nov 2010) New Revision: 32928
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css Log: XSCOLIBRI-251: ie-all.css uses color theme variables without importing the color theme template * Also removed useless * hack since it's a IE dedicated stylesheet.
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css =================================================================== --- platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:43:03 UTC (rev 32927) +++ platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:52:49 UTC (rev 32928) @@ -1,3 +1,4 @@ +#template('colorThemeInit.vm') body { behavior: url("$xwiki.getSkinFile('/htc/csshover3.htc')"); } @@ -7,7 +8,7 @@ }
#docextrapanes { - *display: inline-block; + display: inline-block; }
.minwidth, .minwidthb, .minwidthc, .minwidthd { @@ -29,7 +30,7 @@
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
#commentscontent #AddComment .commentcontainer label { @@ -37,7 +38,7 @@ }
input.button { - *padding: 0 0 1px; + padding: 0 0 1px; }
fieldset.xwikimessage {
_______________________________________________ notifications mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/notifications
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
On 11/17/2010 01:04 PM, Ecaterina Moraru (Valica) wrote:
And what is the solution? to have ie-6.css ie-7.css ie-8.css (maybe this won't be needed if we have ie-7.css because IE8 knows more things) instead of ie-6.css + ie-all.css?
Yes, this is an option. Sergiu, Raluca, wdyt? Thanks, Marius
Thanks, Caty
On Wed, Nov 17, 2010 at 12:57, Marius Dumitru Florea< [email protected]> wrote:
On 11/17/2010 11:53 AM, Ecaterina Moraru (Valica) wrote:
Marius please revert this commit because ie-all means IE6, IE7 and IE8.
* hacks are supposed to go for the IE6, IE7 part, while the non-affected property goes for IE8.
Caty, * is not valid before CSS property name. I know IE ignores it but I need to parse the CSS and that * prevents me. We need to find another way (using proper CSS syntax) to write CSS rules for IE6/7 only.
Thanks, Marius
P.S.: I asked Raluca and Sergiu about the * hack before doing this commit. They seemed to agree that the * is useless in a IE dedicated stylesheet. I guess I haven't explained myself too well.
for example:
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
even if you made this clean - you supposed to delete the overridden
margin
property, but all these properties are needed to display correctly.
Thanks, Caty
On Wed, Nov 17, 2010 at 09:52, mflorea<[email protected] wrote:
Author: mflorea Date: 2010-11-17 08:52:49 +0100 (Wed, 17 Nov 2010) New Revision: 32928
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css Log: XSCOLIBRI-251: ie-all.css uses color theme variables without importing the color theme template * Also removed useless * hack since it's a IE dedicated stylesheet.
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css =================================================================== --- platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:43:03 UTC (rev 32927) +++ platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:52:49 UTC (rev 32928) @@ -1,3 +1,4 @@ +#template('colorThemeInit.vm') body { behavior: url("$xwiki.getSkinFile('/htc/csshover3.htc')"); } @@ -7,7 +8,7 @@ }
#docextrapanes { - *display: inline-block; + display: inline-block; }
.minwidth, .minwidthb, .minwidthc, .minwidthd { @@ -29,7 +30,7 @@
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
#commentscontent #AddComment .commentcontainer label { @@ -37,7 +38,7 @@ }
input.button { - *padding: 0 0 1px; + padding: 0 0 1px; }
fieldset.xwikimessage {
_______________________________________________ notifications mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/notifications
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
On 11/17/2010 12:04 PM, Ecaterina Moraru (Valica) wrote:
And what is the solution? to have ie-6.css ie-7.css ie-8.css (maybe this won't be needed if we have ie-7.css because IE8 knows more things) instead of ie-6.css + ie-all.css?
Use the star hack: * html <rest of the selector> { /* ie-6 rules here, but this should better go in ie-6.css */ } *+html <rest of the selector> { /* ie-7 rules here, but this should better go in ie-7.css */ } I didn't add ie-7.css initially because it wasn't needed, but we can add it if needed.
On Wed, Nov 17, 2010 at 12:57, Marius Dumitru Florea< [email protected]> wrote:
On 11/17/2010 11:53 AM, Ecaterina Moraru (Valica) wrote:
Marius please revert this commit because ie-all means IE6, IE7 and IE8.
* hacks are supposed to go for the IE6, IE7 part, while the non-affected property goes for IE8.
Caty, * is not valid before CSS property name. I know IE ignores it but I need to parse the CSS and that * prevents me. We need to find another way (using proper CSS syntax) to write CSS rules for IE6/7 only.
Thanks, Marius
P.S.: I asked Raluca and Sergiu about the * hack before doing this commit. They seemed to agree that the * is useless in a IE dedicated stylesheet. I guess I haven't explained myself too well.
for example:
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
even if you made this clean - you supposed to delete the overridden
margin
property, but all these properties are needed to display correctly.
Thanks, Caty
On Wed, Nov 17, 2010 at 09:52, mflorea<[email protected] wrote:
Author: mflorea Date: 2010-11-17 08:52:49 +0100 (Wed, 17 Nov 2010) New Revision: 32928
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css Log: XSCOLIBRI-251: ie-all.css uses color theme variables without importing the color theme template * Also removed useless * hack since it's a IE dedicated stylesheet.
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css =================================================================== --- platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:43:03 UTC (rev 32927) +++ platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:52:49 UTC (rev 32928) @@ -1,3 +1,4 @@ +#template('colorThemeInit.vm') body { behavior: url("$xwiki.getSkinFile('/htc/csshover3.htc')"); } @@ -7,7 +8,7 @@ }
#docextrapanes { - *display: inline-block; + display: inline-block; }
.minwidth, .minwidthb, .minwidthc, .minwidthd { @@ -29,7 +30,7 @@
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
#commentscontent #AddComment .commentcontainer label { @@ -37,7 +38,7 @@ }
input.button { - *padding: 0 0 1px; + padding: 0 0 1px; }
fieldset.xwikimessage {
-- Sergiu Dumitriu http://purl.org/net/sergiu/
Hello, On Wed, Nov 17, 2010 at 1:47 PM, Sergiu Dumitriu <[email protected]> wrote:
On 11/17/2010 12:04 PM, Ecaterina Moraru (Valica) wrote:
And what is the solution? to have ie-6.css ie-7.css ie-8.css (maybe this won't be needed if we have ie-7.css because IE8 knows more things) instead of ie-6.css + ie-all.css?
Use the star hack:
* html <rest of the selector> { /* ie-6 rules here, but this should better go in ie-6.css */ } *+html <rest of the selector> { /* ie-7 rules here, but this should better go in ie-7.css */ }
I didn't add ie-7.css initially because it wasn't needed, but we can add it if needed.
We should add ie-7.css . I think that we should have this: * ie-all.css * ie-6.css * ie-7.css and as Sergiu says, the * html and *+html hacks inside the ie-all.css stylesheet. Raluca.
On Wed, Nov 17, 2010 at 12:57, Marius Dumitru Florea< [email protected]> wrote:
On 11/17/2010 11:53 AM, Ecaterina Moraru (Valica) wrote:
Marius please revert this commit because ie-all means IE6, IE7 and IE8.
* hacks are supposed to go for the IE6, IE7 part, while the non-affected property goes for IE8.
Caty, * is not valid before CSS property name. I know IE ignores it but I need to parse the CSS and that * prevents me. We need to find another way (using proper CSS syntax) to write CSS rules for IE6/7 only.
Thanks, Marius
P.S.: I asked Raluca and Sergiu about the * hack before doing this commit. They seemed to agree that the * is useless in a IE dedicated stylesheet. I guess I haven't explained myself too well.
for example:
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
even if you made this clean - you supposed to delete the overridden
margin
property, but all these properties are needed to display correctly.
Thanks, Caty
On Wed, Nov 17, 2010 at 09:52, mflorea<[email protected] wrote:
Author: mflorea Date: 2010-11-17 08:52:49 +0100 (Wed, 17 Nov 2010) New Revision: 32928
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css Log: XSCOLIBRI-251: ie-all.css uses color theme variables without importing the color theme template * Also removed useless * hack since it's a IE dedicated stylesheet.
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css =================================================================== --- platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:43:03 UTC (rev 32927) +++ platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:52:49 UTC (rev 32928) @@ -1,3 +1,4 @@ +#template('colorThemeInit.vm') body { behavior: url("$xwiki.getSkinFile('/htc/csshover3.htc')"); } @@ -7,7 +8,7 @@ }
#docextrapanes { - *display: inline-block; + display: inline-block; }
.minwidth, .minwidthb, .minwidthc, .minwidthd { @@ -29,7 +30,7 @@
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
#commentscontent #AddComment .commentcontainer label { @@ -37,7 +38,7 @@ }
input.button { - *padding: 0 0 1px; + padding: 0 0 1px; }
fieldset.xwikimessage {
-- Sergiu Dumitriu http://purl.org/net/sergiu/ _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
And btw, I see there is a ie-6.css . Can't we organise the IE rules in a better way? Can we move the IE6/7 specific rules from ie-all.css to ie-6.css ? If not, can we have ie-all.css (common rules) and then ie-X.css where X in [6,7,8] if needed. Thanks, Marius On 11/17/2010 12:57 PM, Marius Dumitru Florea wrote:
On 11/17/2010 11:53 AM, Ecaterina Moraru (Valica) wrote:
Marius please revert this commit because ie-all means IE6, IE7 and IE8.
* hacks are supposed to go for the IE6, IE7 part, while the non-affected property goes for IE8.
Caty, * is not valid before CSS property name. I know IE ignores it but I need to parse the CSS and that * prevents me. We need to find another way (using proper CSS syntax) to write CSS rules for IE6/7 only.
Thanks, Marius
P.S.: I asked Raluca and Sergiu about the * hack before doing this commit. They seemed to agree that the * is useless in a IE dedicated stylesheet. I guess I haven't explained myself too well.
for example:
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
even if you made this clean - you supposed to delete the overridden margin property, but all these properties are needed to display correctly.
Thanks, Caty
On Wed, Nov 17, 2010 at 09:52, mflorea<[email protected]>wrote:
Author: mflorea Date: 2010-11-17 08:52:49 +0100 (Wed, 17 Nov 2010) New Revision: 32928
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css Log: XSCOLIBRI-251: ie-all.css uses color theme variables without importing the color theme template * Also removed useless * hack since it's a IE dedicated stylesheet.
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css =================================================================== --- platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:43:03 UTC (rev 32927) +++ platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:52:49 UTC (rev 32928) @@ -1,3 +1,4 @@ +#template('colorThemeInit.vm') body { behavior: url("$xwiki.getSkinFile('/htc/csshover3.htc')"); } @@ -7,7 +8,7 @@ }
#docextrapanes { - *display: inline-block; + display: inline-block; }
.minwidth, .minwidthb, .minwidthc, .minwidthd { @@ -29,7 +30,7 @@
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
#commentscontent #AddComment .commentcontainer label { @@ -37,7 +38,7 @@ }
input.button { - *padding: 0 0 1px; + padding: 0 0 1px; }
fieldset.xwikimessage {
_______________________________________________ notifications mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/notifications
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
the rules in ie-all.css marked with * are used for IE7. On Wed, Nov 17, 2010 at 13:07, Marius Dumitru Florea < [email protected]> wrote:
And btw, I see there is a ie-6.css . Can't we organise the IE rules in a better way? Can we move the IE6/7 specific rules from ie-all.css to ie-6.css ? If not, can we have ie-all.css (common rules) and then ie-X.css where X in [6,7,8] if needed.
Thanks, Marius
On 11/17/2010 12:57 PM, Marius Dumitru Florea wrote:
On 11/17/2010 11:53 AM, Ecaterina Moraru (Valica) wrote:
Marius please revert this commit because ie-all means IE6, IE7 and IE8.
* hacks are supposed to go for the IE6, IE7 part, while the non-affected property goes for IE8.
Caty, * is not valid before CSS property name. I know IE ignores it but I need to parse the CSS and that * prevents me. We need to find another way (using proper CSS syntax) to write CSS rules for IE6/7 only.
Thanks, Marius
P.S.: I asked Raluca and Sergiu about the * hack before doing this commit. They seemed to agree that the * is useless in a IE dedicated stylesheet. I guess I haven't explained myself too well.
for example:
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
even if you made this clean - you supposed to delete the overridden
margin
property, but all these properties are needed to display correctly.
Thanks, Caty
On Wed, Nov 17, 2010 at 09:52, mflorea<[email protected] wrote:
Author: mflorea Date: 2010-11-17 08:52:49 +0100 (Wed, 17 Nov 2010) New Revision: 32928
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css Log: XSCOLIBRI-251: ie-all.css uses color theme variables without importing the color theme template * Also removed useless * hack since it's a IE dedicated stylesheet.
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css =================================================================== --- platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:43:03 UTC (rev 32927) +++ platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:52:49 UTC (rev 32928) @@ -1,3 +1,4 @@ +#template('colorThemeInit.vm') body { behavior: url("$xwiki.getSkinFile('/htc/csshover3.htc')"); } @@ -7,7 +8,7 @@ }
#docextrapanes { - *display: inline-block; + display: inline-block; }
.minwidth, .minwidthb, .minwidthc, .minwidthd { @@ -29,7 +30,7 @@
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
#commentscontent #AddComment .commentcontainer label { @@ -37,7 +38,7 @@ }
input.button { - *padding: 0 0 1px; + padding: 0 0 1px; }
fieldset.xwikimessage {
_______________________________________________ notifications mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/notifications
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
On Wed, Nov 17, 2010 at 13:07, Marius Dumitru Florea < [email protected]> wrote:
And btw, I see there is a ie-6.css . Can't we organise the IE rules in a better way? Can we move the IE6/7 specific rules from ie-all.css to ie-6.css ?
we cannot move the IE7 rules to ie-6.css because ie-6 is used only for IE6.
If not, can we have ie-all.css (common rules) and then ie-X.css where X in [6,7,8] if needed.
if we have ie-X.css where [6,7,8] we won't need a ie-all.css.
Thanks, Marius
On 11/17/2010 12:57 PM, Marius Dumitru Florea wrote:
On 11/17/2010 11:53 AM, Ecaterina Moraru (Valica) wrote:
Marius please revert this commit because ie-all means IE6, IE7 and IE8.
* hacks are supposed to go for the IE6, IE7 part, while the non-affected property goes for IE8.
Caty, * is not valid before CSS property name. I know IE ignores it but I need to parse the CSS and that * prevents me. We need to find another way (using proper CSS syntax) to write CSS rules for IE6/7 only.
Thanks, Marius
P.S.: I asked Raluca and Sergiu about the * hack before doing this commit. They seemed to agree that the * is useless in a IE dedicated stylesheet. I guess I haven't explained myself too well.
for example:
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
even if you made this clean - you supposed to delete the overridden
margin
property, but all these properties are needed to display correctly.
Thanks, Caty
On Wed, Nov 17, 2010 at 09:52, mflorea<[email protected] wrote:
Author: mflorea Date: 2010-11-17 08:52:49 +0100 (Wed, 17 Nov 2010) New Revision: 32928
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css Log: XSCOLIBRI-251: ie-all.css uses color theme variables without importing the color theme template * Also removed useless * hack since it's a IE dedicated stylesheet.
Modified: platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css =================================================================== --- platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:43:03 UTC (rev 32927) +++ platform/skins/trunk/colibri/src/main/resources/colibri/ie-all.css 2010-11-17 07:52:49 UTC (rev 32928) @@ -1,3 +1,4 @@ +#template('colorThemeInit.vm') body { behavior: url("$xwiki.getSkinFile('/htc/csshover3.htc')"); } @@ -7,7 +8,7 @@ }
#docextrapanes { - *display: inline-block; + display: inline-block; }
.minwidth, .minwidthb, .minwidthc, .minwidthd { @@ -29,7 +30,7 @@
.xwikiintracontent legend { margin: -5px 0 5px 0; - *margin: 5px 0 10px -7px; + margin: 5px 0 10px -7px; }
#commentscontent #AddComment .commentcontainer label { @@ -37,7 +38,7 @@ }
input.button { - *padding: 0 0 1px; + padding: 0 0 1px; }
fieldset.xwikimessage {
_______________________________________________ notifications mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/notifications
_______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
participants (4)
-
Ecaterina Moraru (Valica) -
Marius Dumitru Florea -
Raluca Stavro -
Sergiu Dumitriu