humanity-icon-theme: black magnifier icons on dark toolbars have contrast issues

Bug #832170 reported by Paul Sladen
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
humanity-icon-theme (Ubuntu)
Fix Released
Low
Danielle Foré

Bug Description

Cimi noticed that some icons from lp:elementaryicons into the ubuntu-mono package.

The icons that need deleting/replacing are:

  The magnifier icons used for find, find & replace, and zoom levels

because:

  Several action icons look crap on the new dark toolbars introduced in the light themes.

Revision history for this message
Paul Sladen (sladen) wrote :

<danrabbit> The changes involve action icons in Humanity.

affects: ubuntu-mono (Ubuntu) → humanity-icon-theme (Ubuntu)
Revision history for this message
Vish (vish) wrote :

I can't see any icon list in the bug description.
@Cimi, what icons need merging?

Changed in humanity-icon-theme (Ubuntu):
assignee: nobody → Vish (vish)
importance: Undecided → Low
status: New → Incomplete
Revision history for this message
Andrea Cimitan (cimi) wrote :

@vish: just use the new trunk, nothing has to be merged, some instead removed.

Revision history for this message
Paul Sladen (sladen) wrote :

Vish: yes, quite, that big space was a placeholder for Cimi/Danrabbit to fill in exactly what needed merging/replacing/updating. It's still a placeholder but guess per comment #3 apparently only involves removals.

Revision history for this message
Paul Sladen (sladen) wrote :

Dan/Cimi: some more detail about what exactly is the request, would still be handy; the delta is 60 MB and 400 revisions:

  $ du --exclude .bzr* -sch humanity elementaryicons
  69M humanity
  137M elementaryicons

Revision history for this message
Vish (vish) wrote :

Could someone attach a screenshot of what problem we are trying to solve by removing icons?
I *think* we can fix this without removing icons.

Revision history for this message
Danielle Foré (danrabbit) wrote :

There was a need this cycle to make sure that Humanity icons look okay on the new dark toolbars.

There have been several changes in action icons, along with adding more sizes of these icons.

The magnifier icons have been removed since they are very difficult to see. There are plans to create new magnifier icons, this is just a "quick fix".

Revision history for this message
Paul Sladen (sladen) wrote :

Dan: Thanks for that lead about the dark toolbars; are there any other contextual emails/bug report numbers or the like that you have to hand to give more background/details/context/examples. Are we talking about:

  ./Humanity/actions/*/viewmag*.svg

?

Revision history for this message
Danielle Foré (danrabbit) wrote :

Here's a bug report: https://bugs.launchpad.net/humanity/+bug/833271

It involves both the zoom level icons and the find/find-replace icons specifically, yes.

Revision history for this message
Vish (vish) wrote :

@Dan: Yea, I'm pushing an update with magnifier icons similar style to the Unity icons.. get on IRC! ;p

Paul Sladen (sladen)
description: updated
summary: - Please merge lp:elementaryicons into ubuntu-mono
+ Please remove toolbar magnifier icons from humanity-icon-theme because
+ of contrast issues against dark background
Revision history for this message
Paul Sladen (sladen) wrote : Re: Please remove toolbar magnifier icons from humanity-icon-theme because of contrast issues against dark background

So, as I understand, there are two options available:

  a. rm humanity/Humanity/actions/24/{viewmag{1,-,+,fit},zoom-{best-fit,fit-best,in,original,out}.svg

thus allowing the default gnome-icon-theme versions to shine through; or, to development replacements and install those at:

  b. humanity/Humanity-Dark/actions/24/{viewmag{1,-,+,fit},zoom-{best-fit,fit-best,in,original,out}.svg

This is correct? Are there any icons I've missed from the list (eg. 'window-close.svg' and 'media-eject.svg' appear to have the same sort of dark palette)?

Revision history for this message
Vish (vish) wrote :

Looks like Dan is working on icons (tango style?); if its not done within the week we could remove the icons.

If we are removing the icons from Humanity now and then adding them later next week with a new style, We'd be changing the UI twice in a week. IMO, we can do the change once within next week, either removing or updating the icons

Changed in humanity-icon-theme (Ubuntu):
assignee: Vish (vish) → Daniel Fore (daniel-p-fore)
status: Incomplete → Triaged
Revision history for this message
Andrea Cimitan (cimi) wrote :

We can't. UI Freeze is tomorrow at 21UTC. We should just update the package with the new icons by dan.
WE MUST HAVE A PACKAGE UPLOADED WITH THE FIX FOR BOTH AMBIANCE AND RADIANCE :-)
Loud and clear, roger :)

Revision history for this message
Paul Sladen (sladen) wrote :

I think the point is that Dan might not have the icons (which exact ones?) done by 2100UTC today; in which case we're left with the single option of dropping the icons in question (which exact ones?).

Revision history for this message
Paul Sladen (sladen) wrote :

  <Cimi> sladen: oh my god, ok. I'll ask kenvandine when he wakes up

Ken: any ideas? You're welcome to upload this if you can figure out what's needed.

Revision history for this message
Paul Sladen (sladen) wrote :

  <Cimi> sladen: we can't ship black magnifier icons on dark toolbars, so you should have a look at which icons need to be updated and update them

Revision history for this message
Paul Sladen (sladen) wrote :

Cimi: ta.

summary: - Please remove toolbar magnifier icons from humanity-icon-theme because
- of contrast issues against dark background
+ humanity-icon-theme: black magnifier icons on dark toolbars have
+ contrast issues
Revision history for this message
Andrea Cimitan (cimi) wrote :

 take commit 844 of lp:elementaryicons, look at which icons were removed (they should be mags and so) and which changes are requested in index.theme (if there are any, I doubt), backport them into ubuntu humanity icons, then we have done.

Revision history for this message
Paul Sladen (sladen) wrote :

Cimi: Purfect; I'm intending to apply:

  $ time bzr diff lp:elementaryicons -r843..844 | grep '^=== removed' | cut -f2 -d\' | cut -d/ -f2- | xargs ls -1 | sed -e 'sX/[0-9][0-9]*/X/*/X' | sort | uniq
  Humanity/actions/*/edit-find-replace.svg
  Humanity/actions/*/edit-find.svg
  Humanity/actions/*/filefind.svg
  Humanity/actions/*/find.svg
  Humanity/actions/*/gnome-searchtool.svg
  Humanity/actions/*/gtk-find-and-replace.svg
  Humanity/actions/*/gtk-find.svg
  Humanity/actions/*/gtk-search.svg
  Humanity/actions/*/gtk-zoom-100.svg
  Humanity/actions/*/gtk-zoom-fit.svg
  Humanity/actions/*/gtk-zoom-in.svg
  Humanity/actions/*/gtk-zoom-out.svg
  Humanity/actions/*/kfind.svg
  Humanity/actions/*/search.svg
  Humanity/actions/*/stock_search-and-replace.svg
  Humanity/actions/*/stock_search.svg
  Humanity/actions/*/stock_zoom-1.svg
  Humanity/actions/*/stock_zoom-in.svg
  Humanity/actions/*/stock_zoom-out.svg
  Humanity/actions/*/stock_zoom-page.svg
  Humanity/actions/*/system-search.svg
  Humanity/actions/*/viewmag1.svg
  Humanity/actions/*/viewmagfit.svg
  Humanity/actions/*/viewmag-.svg
  Humanity/actions/*/viewmag+.svg
  Humanity/actions/*/zoom-best-fit.svg
  Humanity/actions/*/zoom-fit-best.svg
  Humanity/actions/*/zoom-in.svg
  Humanity/actions/*/zoom-original.svg
  Humanity/actions/*/zoom-out.svg

  … | xargs bzr rm

Revision history for this message
Andrea Cimitan (cimi) wrote : Re: [Bug 832170] Re: humanity-icon-theme: black magnifier icons on dark toolbars have contrast issues

Good Paul, I don't have time to test it but I trust you. Be sure to upload
the package in time for UI FREEZE
Il giorno 25/ago/2011 13:40, "Launchpad Bug Tracker" <
<email address hidden>> ha scritto:
> ** Branch linked: lp:~sladen/humanity/lp-832170-dark-on-dark-toolbar-
> removals
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/832170
>
> Title:
> humanity-icon-theme: black magnifier icons on dark toolbars have
> contrast issues
>
> Status in “humanity-icon-theme” package in Ubuntu:
> Triaged
>
> Bug description:
> Cimi noticed that some icons from lp:elementaryicons into the ubuntu-
> mono package.
>
> The icons that need deleting/replacing are:
>
> The magnifier icons used for find, find & replace, and zoom levels
>
> because:
>
> Several action icons look crap on the new dark toolbars introduced
> in the light themes.
>
> To manage notifications about this bug go to:
>
https://bugs.launchpad.net/ubuntu/+source/humanity-icon-theme/+bug/832170/+subscriptions

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package humanity-icon-theme - 0.5.3.11

---------------
humanity-icon-theme (0.5.3.11) oneiric; urgency=low

  [Andrea Cimitan]
  * Remove dark icons on dark toolbars that have low contrast (LP: #832170)
    Cherry-pick deletions from upstream revision lp:elementaryicons -r844
    Removals:
    - Humanity/actions/*/edit-find{,-replace}.svg
    - Humanity/actions/*/{file,,k}find.svg
    - Humanity/actions/*/gnome-searchtool.svg
    - Humanity/actions/*/gtk-{find{,-and-replace},search,zoom-{100,fit,in,out}}.svg
    - Humanity/actions/*/{,system-}search.svg
    - Humanity/actions/*/stock_{search{,-and-replace},zoom-{1,in,out,page}}.svg
    - Humanity/actions/*/viewmag{1,fit,-,+}.svg
    - Humanity/actions/*/zoom-{best-fit,fit-best,in,original,out}.svg
 -- Paul Sladen <email address hidden> Thu, 25 Aug 2011 20:59:59 +0000

Changed in humanity-icon-theme (Ubuntu):
status: Triaged → Fix Released
Revision history for this message
Dmitry Shachnev (mitya57) wrote :

We still need better search icons in ubuntu-mono, see bug 909603 for that.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.