System Settings Keyboard icon indistinct with Ambiance

Paul, at this point I think it's easiest to just revert the change. It was made way after UIF anyway. Do you see another quick and safe solution?

Jeremy: where are you seeing this indistinct example? Is it with:

  1. <Super>
  2. typing "keyboard" in the dash?

Really, Dash or Launcher shouldn't be using the "status" keyboard icons; but if needed I can restore/replace the previous keyboard at sizes != 22 pixels. Note that the "preferences-desktop-keyboard" icon was not changed by the upload.

  dpkg -L ubuntu-mono | grep keyboard | grep -v [gf]pm

  diff -Nru debian/changelog debian/changelog
  diff -Nru debian/control debian/control
  diff -Nru debian/rules debian/rules
  diff -Nru ubuntu-mono-dark/status/16/ibus-keyboard.svg ubuntu-mono-dark/status/16/ibus-keyboard.svg
  diff -Nru ubuntu-mono-dark/status/22/ibus-keyboard.svg ubuntu-mono-dark/status/22/ibus-keyboard.svg
  diff -Nru ubuntu-mono-dark/status/22/input-keyboard.svg ubuntu-mono-dark/status/22/input-keyboard.svg
  diff -Nru ubuntu-mono-dark/status/22/keyboard.svg ubuntu-mono-dark/status/22/keyboard.svg
  diff -Nru ubuntu-mono-dark/status/24/ibus-keyboard.svg ubuntu-mono-dark/status/24/ibus-keyboard.svg
  diff -Nru ubuntu-mono-light/status/16/ibus-keyboard.svg ubuntu-mono-light/status/16/ibus-keyboard.svg
  diff -Nru ubuntu-mono-light/status/22/ibus-keyboard.svg ubuntu-mono-light/status/22/ibus-keyboard.svg
  diff -Nru ubuntu-mono-light/status/22/input-keyboard.svg ubuntu-mono-light/status/22/input-keyboard.svg
  diff -Nru ubuntu-mono-light/status/22/keyboard.svg ubuntu-mono-light/status/22/keyboard.svg
  diff -Nru ubuntu-mono-light/status/24/ibus-keyboard.svg ubuntu-mono-light/status/24/ibus-keyboard.svg

  http://bazaar.launchpad.net/~ubuntu-branches/ubuntu/oneiric/ubuntu-mono/oneiric/revision/30?remember=31

(Diff above from:

  debdiff ubuntu-mono_0.0.34.dsc ubuntu-mono_0.0.36.dsc | grep diff | sed -e 's/ubuntu-mono-0.0.3..//g' )

Paul, I think he means the control-center shell. Session indicator -> system settings (or "gnome-control-center" in shell)

It's light gray on white, and thus has a poor contrast, but still visible, so not a release blocker. Can be fixed in an SRU as well, if really necessary.

Gnome Control Center; showing the symbolic icon; The icon has an intentional black underlay to ensure that it remains visible even when misused like this, but it's not ideal.

Otto/Xi: ideas. While sub-optimal, it's not that much worse that the mouse. I can restore the previous icon for resizes != 22 to avoid patching gnome-control-centre or gnome-settings-daemon (both of which are using plain Icon('keyboard') it appears.

The previous keyboard icon in panel really stood out as requiring improvement and I think this new one is a big improvement, so I would like to keep it.

Looking at Paul's sshot (settings-keyboard.png), the icon is not ideal - the contrast isn't great but I'm inclined to agree that it's passable and that it shouldn't even be being used in that context. Ideally we will move to using full symbolic icons that can be treated with different effects based on where they are used - 'one shape fits all' as it were.

It seems that g-c-c uses an icon name different from
"preferences-desktop-keyboard", is it easy to fix it?

Screenshot of GNOME Control Centre and previous ubuntu-mono=0.0.34 installed.

So it's just a naming issue then? Ideally we'd revert to (Paul's last sshot as example for g-c-c) while maintaing our latest keyboard panel icon.

Ok, I used solution #3 as suggested in comment #10 and pushed the patch to our gnome-control-center desktop branch. Is this what we're going to do for Oneiric?

Jeremy: At a cursory glance of:

   http://bazaar.launchpad.net/~ubuntu-desktop/gnome-control-center/ubuntu/revision/295

it looks reasonable, but I haven't built it yet.

Builds and works for me(tm).

Accepting because it fixes a regression introduced in a rather late UIFE.

I'm going to re-open this as I don't believe this was fixed correctly. For the Ubuntu GNOME Remix, we want to get as pure of a GNOME experience as possible. The fix that was applied in Oneiric changes the icon used for the Keyboard panel.

From the original bug report, I believe we basically went with #3 but I think #2 is what we want for raring. Specifically, upstream uses "input-keyboard" for the icon but we patched it to use "preferences-desktop-keyboard" instead.

Because we're using the wrong icon name, the Keyboard panel doesn't have an icon when using the High Contrast theme. See the attached screenshots.

The icon renaming also goes against the Icon Naming Specification: http://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html

This bug was fixed in the package gnome-control-center - 1:3.6.3-0ubuntu33

---------------
gnome-control-center (1:3.6.3-0ubuntu33) saucy; urgency=low

  [ Jeremy Bicha ]
  * Restore 57_use_nonsymbolic_keyboard_icon.patch, the ubuntu-themes
    merge proposal wasn't accepted yet

  [ Tim Lunn ]
  * debian/patches/git_shell_use_view_style_class.patch: git patch to
    fix mismatched backgrounds (LP: #1218102)
 -- Tim Lunn <email address hidden> Thu, 29 Aug 2013 19:23:56 +1000

This bug was fixed in the package ubuntu-themes - 13.04+13.10.20130919.2-0ubuntu1

---------------
ubuntu-themes (13.04+13.10.20130919.2-0ubuntu1) saucy; urgency=low

  [ William Hua ]
  * Add spacing between a GtkModelMenuItem's icon and its label.

  [ Jeremy Bicha ]
  * Drop input-keyboard icons, now that we have a proper
    indicator-keyboard. (LP: #862027)

  [ James Schriver ]
  * Fix light toolbar separators in Ambiance (LP: #1181661)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 315
 -- Ubuntu daily release <email address hidden> Thu, 19 Sep 2013 14:05:04 +0000