Last modified: 2012-03-16 01:47:01 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T36990, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 34990 - Android menu has no icon for 'Share'
Android menu has no icon for 'Share'
Status: RESOLVED FIXED
Product: Wikipedia App
Classification: Unclassified
Generic (Other open bugs)
3.1.0 (iOS) / 1.1 (Android)
All All
: Low normal
: ---
Assigned To: Nobody - You can work on this!
: design, mobile
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-05 18:54 UTC by Brion Vibber
Modified: 2012-03-16 01:47 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: Android
Assignee Huggle Beta Tester: ---


Attachments
Image of git head, no icon for share (279.65 KB, application/octet-stream)
2012-03-11 03:15 UTC, Sam Reed (reedy)
Details

Description Brion Vibber 2012-03-05 18:54:29 UTC
'Share' got moved up to the first set of icons, but it has no icon so doesn't look very nice.
Comment 1 Yuvi Panda 2012-03-10 12:02:10 UTC
Are you sure? I see an icon for it...
Comment 2 Sam Reed (reedy) 2012-03-11 03:11:24 UTC
Ditto Brion. Share has no icon
Comment 3 Sam Reed (reedy) 2012-03-11 03:15:00 UTC
Created attachment 10215 [details]
Image of git head, no icon for share

Uploaded directly from my phone!
Comment 4 Sam Reed (reedy) 2012-03-11 03:16:38 UTC
(In reply to comment #1)
> Are you sure? I see an icon for it...

Got some uncommited/unpushed changes Yuvi? ;)
Comment 5 Yuvi Panda 2012-03-11 07:23:15 UTC
Nope, been committed for a while https://github.com/wikimedia/WikipediaMobile/commit/481e00cd0e33663501946f43f0a95c79c58a1527

This is very interesting/weird...
Comment 6 Sam Reed (reedy) 2012-03-11 15:04:31 UTC
This is on ICS 4.0.2 on my Galaxy Nexus. I guess Brion is aswell...

It is ok on my Motorola Xoom on 4.0.3 though...

I can try on my Desire HD on 2.3 when my brother gets back later
Comment 7 Brion Vibber 2012-03-12 18:10:31 UTC
There's no xhdpi image, which explains why it's missing on Galaxy Nexus but not on lower-density devices.
Comment 8 Sam Reed (reedy) 2012-03-12 22:40:14 UTC
(In reply to comment #7)
> There's no xhdpi image, which explains why it's missing on Galaxy Nexus but not
> on lower-density devices.

This would be easily fixed if we had the one that is supposed to be in orig.

If it was there, ./share.sh, add, commit...
Comment 9 Yuvi Panda 2012-03-12 22:57:27 UTC
I took the share icon from android default icons - am so far unable to find the xhdpi version though. Ugh
Comment 10 Brion Vibber 2012-03-12 22:58:58 UTC
xhdpi seems to be new in android 4 so there may not be a default image for that menu style, depending on how cleverly they made their default-icon assets. :P

Worst case just scale the hdpi one up and it'll be a smidge blurry.
Comment 11 Yuvi Panda 2012-03-12 23:03:14 UTC
I'll poke heather when she's back and see if she can make us one. Will also try finding this exact icon again

There *is* a share icon available at all dpis, but it looks quite different than the one we're using.
Comment 13 Tomasz Finc 2012-03-16 01:47:01 UTC
Actually yuvi already checked in a scaled up copy https://github.com/wikimedia/WikipediaMobile/blob/master/assets/www/img/xhdpi/optionSharePage.png. Resolving

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links