Apache OpenOffice (AOO) Bugzilla – Issue 67061
Remove unrequired HC icons in '/dbaccess'
Last modified: 2017-05-20 11:31:06 UTC
Following the consolidation of icons (i35193), these High Contrast icons were added to CVS: ui/default_images/dbaccess/res/sxh03187.png ui/default_images/dbaccess/res/sxh03188.png ui/default_images/dbaccess/res/sxh16670.png Since they lack 'Default' theme counter-parts, I imagine they are redundant. Regards, Andrew
Committed to CWS icons01.
Checked out CVS branch cws_src680_icons01. Everything looks good - Verified. Kendy, I assume that you were satisfied these were actually redundant. Cheers.
>kendy: I've noticed that sxh16670.png was recently updated by CWS qiq, for issue 51143... could it be that these icons are actually in use?
Some more info... It seems the 'active' Default icons live here (how helpful!) : ui/default_images/res/sx03187.png ui/default_images/res/sx03188.png ui/default_images/res/sx16670.png (updated with CWS qiq) And these two are older HC variants (just engineering placeholders): ui/default_images/res/sxh03187.png ui/default_images/res/sxh03188.png So we have HC icons in /dbaccess and the Default icons in /res, with some redundancy somewhere... seems a bit of a mess. Not sure whether to reopen this issue, or best to raise new issue and get into icons02. Kendy, what is going on in the code?
The HC versions in /res are bogus, and should be removed. Given at least Writer has different icons for the same thing (see Edit / Exchange Database, or Insert / Fields / Other / Database), we on the mid term probably should a) move the icons to /res alltogether so Writer can use them, too and b) given them a meaningful name.
->Sun's ArtTeam (icon issue): Re-opening issue and reassigning to 'sts' for feedback. [icons]
.
->sts, Ok for CWS 'iconupdate03'?...
set target OOo 3.0
set target 3.x
STS -> VA: Please add this issue to CWS "iconupdate310"
Reset assigne to the default "issues@openoffice.apache.org".