← Back to team overview

touch-packages team mailing list archive

[Bug 1407720] [NEW] Dash uses incorrect serialisation for indicator-network menu icons

 

Public bug reported:

Currently indicator-network exports icons to show like this (gdbus
dump):

'icons': <
          [
           <('themed', <['gsm-3g-full']>)>,
           <('themed', <['nm-signal-50-secure']>)>
          ]
         >,

That is they are an array of variants (that happen to be tuples).
However this seems to be incorrect. At least if you use the indicator-
loader it chokes on these and refuses to draw them. The form that it
accepts looks like this:

'icons': <
          [
           ('themed', <['gsm-3g-full']>),
           ('themed', <['simcard-locked']>),
           ('themed', <['nm-signal-75-secure']>)
          ]
         >

That is, an array of tuple variants. One of these two gets it wrong and
since indicator-loader is the older of the two, I'm guessing that is the
correct one. Feel free to reassign as appropriate if you feel that
Unity8 is doing the right thing here.

** Affects: unity8 (Ubuntu)
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to unity8 in Ubuntu.
https://bugs.launchpad.net/bugs/1407720

Title:
  Dash uses incorrect serialisation for indicator-network menu icons

Status in unity8 package in Ubuntu:
  New

Bug description:
  Currently indicator-network exports icons to show like this (gdbus
  dump):

  'icons': <
            [
             <('themed', <['gsm-3g-full']>)>,
             <('themed', <['nm-signal-50-secure']>)>
            ]
           >,

  That is they are an array of variants (that happen to be tuples).
  However this seems to be incorrect. At least if you use the indicator-
  loader it chokes on these and refuses to draw them. The form that it
  accepts looks like this:

  'icons': <
            [
             ('themed', <['gsm-3g-full']>),
             ('themed', <['simcard-locked']>),
             ('themed', <['nm-signal-75-secure']>)
            ]
           >

  That is, an array of tuple variants. One of these two gets it wrong
  and since indicator-loader is the older of the two, I'm guessing that
  is the correct one. Feel free to reassign as appropriate if you feel
  that Unity8 is doing the right thing here.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/unity8/+bug/1407720/+subscriptions


Follow ups

References