Re: [Powertop] [Announce] v2.5-rc1
Alexandra Yates <alexandra.yates at linux.intel.com>
| Newsgroups | dev.linux.lists.powertop |
|---|---|
| Message-ID | <[email protected]> |
> 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.