Oops...too tired when I wrote that. Make it
  if ((new RegExp('[^ ,]'+selector+'[$ ,]').test(this.selectorText)))
but I haven't tested that either


Danny Wachsstock wrote:
> 
> Brilliant minds think alike :) The RegExp allows for stylesheets that use
> a comma, like
> 
> .classname , .otherclass {...}
> 
> which my simple == wouldn't catch. On the other hand, yours would catch 
> 
> .classname childelement {...}
> 
> which wouldn't be right. The quick and dirty answer is to make sure your
> stylesheets don't do that. The right answer would be to change the
> condition to 
> 
> if ('/[^ ,]'.selector.'[$ ,]/'.test(this.selectorText))
> 
> but I haven't tested that. Adding the checks for the appropriate media
> types is left as an exercise for the reader.
>  
> 
> thumblewend wrote:
>> 
>> On 16/03/2007, at 10:01 AM, Danny Wachsstock wrote:
>>> function styleString(selector){
>>>   var style = "";
>>>   $.each (document.styleSheets, function(){
>>>     $.each (this.cssRules || this.rules, function() {
>>>       if (this.selectorText == selector) style +=  
>>> this.style.cssText + ';';
>>>     });
>>>   });
>>>   return style;
>>> }
>> 
>> Haha, that's funny. I was just about to post my rewrite and the code  
>> is very similar to yours except yours is better due to the use of ||  
>> instead of my ternary, plus I used a regexp as per the original which  
>> from your code looks like it's not needed. Mine's untested too. Here  
>> it is for giggles:
>> 
>> $.getClassContent = function(classname) {
>>      var cssText = '';
>>      $.each(document.styleSheets,function() {
>>              var cssRules = (typeof this.cssRules == 'array') ? 
>> this.cssRules :  
>> this.rules;
>>              $.each(cssRules,function() {
>>                      if ('/\.'+classname+'/'.test(this.selectorText)){
>>                              cssText = this.style.cssText;
>>                      }
>>              });
>>      });
>>      return cssText;
>> };
>> 
>> Joel.
>> 
>> _______________________________________________
>> jQuery mailing list
>> discuss@jquery.com
>> http://jquery.com/discuss/
>> 
>> 
> 
> 

-- 
View this message in context: 
http://www.nabble.com/How-to-convert-%22class%22-to-%22style%3D%27attrs-...%27%22-tf3409582.html#a9513147
Sent from the JQuery mailing list archive at Nabble.com.


_______________________________________________
jQuery mailing list
discuss@jquery.com
http://jquery.com/discuss/

Reply via email to