[go: up one dir, main page]

Page MenuHomePhabricator

Improve 'Train image algorithm' onboarding illustrations for different themes
Closed, ResolvedPublic

Assigned To
Authored By
scblr
Mar 26 2021, 10:41 AM
Referenced Files
F34388637: Screenshot_20210414-124650.png
Apr 14 2021, 10:50 AM
F34388635: Screenshot_20210414-124057.png
Apr 14 2021, 10:50 AM
F34276291: Screen Shot 2021-04-07 at 9.40.25 AM.png
Apr 7 2021, 4:40 PM
F34212282: add-images-04.png
Apr 2 2021, 10:27 AM
F34212288: add-images-04-dark.png
Apr 2 2021, 10:27 AM
F34212280: add-images-03.png
Apr 2 2021, 10:27 AM
F34212286: add-images-02-dark.png
Apr 2 2021, 10:27 AM
F34191660: Screenshot_20210326-113803.png
Mar 26 2021, 10:41 AM

Description

Problem

Currently, the onboarding illustrations are not created to be used for various themes.

Screenshot_20210326-113557.png (2×1 px, 205 KB)
Screenshot_20210326-113803.png (2×1 px, 79 KB)
  • The logo and and article title on the first screenshot have an unnecessary white background.
  • Colors on the second illustrations are not optimized for dark and black theme (too bright)
Solution

Apply the following, revised, illustrations:

Event Timeline

JTannerWMF triaged this task as High priority.
JTannerWMF moved this task from Backlog to Doing on the Android Design board.

Hi @schoenbaechler... as we discussed on the call, the images are not rendering properly.. I see the same problem with dark mode image too... this is how it is getting rendered:[ignore the light background]

Screen Shot 2021-04-07 at 9.40.25 AM.png (242×476 px, 19 KB)

@Sharvaniharan — I updated the illustrations (Light+Sepia, Dark+Black) on Zeplin. Let me know if it works!

One more thing @Sharvaniharan — realized that we should align text and image vertically at the bottom in all onboarding dialogs in order to be consistent with other places of the app. Updated the designs accordingly.

@schoenbaechler Please review from here: https://github.com/wikimedia/apps-android-wikipedia/pull/2133/checks

Other than the text align oter things should work. I will keep this ticket in design review till I align the text..

@schoenbaechler I did not understand the text alignment fully... we might need to talk tomorrow .. heads-up.

Thanks @Sharvaniharan — this is what I was referring to:

Screenshot_20210414-124057.png (2×1 px, 252 KB)
Screenshot_20210414-124650.png (2×1 px, 193 KB)

Please vertically align the onboarding dialogs to the bottom not top.

The alignment is now done @schoenbaechler