Re: [Powertop] [Announce] v2.5-rc1

[Alexandra Yates <alexandra.yates at linux.intel.com>]

[-- Attachment #1: Type: text/plain, Size: 1756 bytes --]


> On (10/21/13 17:21), Yates, Alexandra wrote:
>> IE 8.0 up to 10 doesn't display the menu bar nor any of the links.
>
> please test the following patch.
>
> ------8<-----8<--------
>
> IE up to v10 does not support object classList. workaround
> by manual className manipulation.
>
> Reported-by: Alexandra Yates <alexandra.yates(a)intel.com>
> Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky(a)gmail.com>
>
> ---
>
>  src/powertop.css | 16 +++++++++++-----
>  1 file changed, 11 insertions(+), 5 deletions(-)
>
> diff --git a/src/powertop.css b/src/powertop.css
> index 6d43e07..c58d757 100644
> --- a/src/powertop.css
> +++ b/src/powertop.css
> @@ -17,14 +17,20 @@ var powertop = {
>   },
>   cadd: function(idx, c){
>     var el = document.getElementById(idx);
> -   if (el)
> -     el.classList.add(c);
> -
> +   if (el) {
> +      var cn = el.className;
> +      cn += ' ' + c;
> +      el.className = cn;
> +   }
>   },
>   crm: function(id, c){
>     var el = document.getElementById(id);
> -   if (el)
> -     el.classList.remove(c);
> +   if (el) {
> +      var cn = el.className
> +      while (cn.indexOf(' ' + c) != -1)
> +         cn = cn.replace(' ' + c,'');
> +      el.className = cn;
> +   }
>   },
>   newbutton: function(id, txt) {
>    var x = document.createElement('div');
>
> _______________________________________________
> PowerTop mailing list
> PowerTop(a)lists.01.org
> https://lists.01.org/mailman/listinfo/powertop
>

This change enables the menu links and mouse over, but the font and the
button background are all black, I don't think the css are working here. 
I will send you the image to your email that way you get to see it.

Thank you,
Alexandra.