Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dark Mode Master Issue #12320

Closed
40 of 57 tasks
nheagy opened this issue Aug 14, 2019 · 36 comments · Fixed by #12616
Closed
40 of 57 tasks

Dark Mode Master Issue #12320

nheagy opened this issue Aug 14, 2019 · 36 comments · Fixed by #12616
Assignees
Labels
Projects

Comments

@nheagy
Copy link
Contributor

nheagy commented Aug 14, 2019

Goals

dark-mode-i2

Tasks

Global

  • Add XCODE11 build flag to support parallel development between Xcode 10 & 11 (Allow building with Xcode 11 by conditionally disabling Zendesk #12120)
  • Create initial set of preview screenshots
  • First-pass at dark mode support on main tab bar screens
  • Dark mode versions for badged state of tab bar images
  • Fancy alerts have white backgrounds
  • Revisions section has white backgrounds
  • QuickStart has some white background
  • Add new site flow has some white background
  • Update fancy button border style (Dark Mode Master Issue #12320 (comment)) [Dark Mode] Update fancy button border style #12592
  • Change textPlaceholder Muriel color to use tertiaryLabel on iOS 13.
  • Register domain has white backgrounds
  • Edit comment is white text on white background.
  • Pending comments have a white background.
  • Gravatar cropping screen has a white background

Regressions

NUX

  • Dark mode for NUX
  • Login NUX button has black spinner while signing in
  • Header text on sign up bottom sheet isn't readable
  • Header and footer on signup epilogue are white and should be listForeground
  • New account epilogue has a different color behind 'new account' which shouldn't be there [Dark Mode] Account epilogue has a different color #12593
  • Modal HUD shown after hitting Continue on New Account screen should use system grey background. [Dark Mode] HUD background color #12595
  • Activity indicator on secondary NUX buttons appears white-on-white in light mode. #140

Editor

Stats

  • Some areas in Stats Insights still have white backgrounds
  • Dark countries map can be too dark to read

My Sites

  • Posts cells (expanded view) have white backgrounds for some labels
  • Post / pages tab bar style should match Stats
  • Themes support icon isn't being tinted correctly
  • Ghost view in posts / pages has a white background
  • Activity logs (in help and support) have white text on white background
  • Plans are still light colored
  • Activity screen is still light colored
  • Reverse text and textSubtle colors in Activity Log list.
  • Remove divider below 'free plan' footer in Activity Log list
  • Plans list 'Best for...' text should use textSubtle
  • Menus: Down arrows and reorder handles should use textTertiary (via textPlaceholder?) Page icon beside 'new item' should use listIcon color.
  • Menus: When editing a menu name, use tertiarySystemBackground / gray80 for the background color. Text should be text and X icon should be listIcon.
  • Sharing > Sharing Buttons / Manage > Edit sharing buttons – icons should use listIcon. "share this" and "official buttons" should use textSubtle.
  • Posts list: After trashing a post, the "undo" cell has a white background [Dark Mode] Undo cell has white background #12598
  • Settings: Time Zone & Tags screens have a white background when you pull to refresh [Dark Mode] Time Zone & Tags screens have a white background #12599
  • Settings: Pickers have white text on white background (Posts per page & Discussion settings: Close commenting, Paging, Links in comments) [Dark Mode] Pickers have white text on white background  #12600

Reader

Extensions

  • Update widget
  • Update share extension
  • Update notifications rich content extension

V2

  • Make tab bar translucent

Switching between dark and light mode #12441

  • Reader Detail
  • Fancy button
  • SearchBar strokes

Notifications

  • Comment moderation has incorrect background on the comment cell, screenshot added to the thread.

Analytics

There are no analytics events for this project.

Related issues

This will finally close #6373 🎉

@nheagy nheagy added the Dark mode iOS 13 dark mode label Aug 14, 2019
@nheagy nheagy self-assigned this Aug 14, 2019
nheagy added a commit that referenced this issue Aug 20, 2019
nheagy added a commit that referenced this issue Aug 20, 2019
Dark mode: Fix no results background color
@mattmiklic
Copy link
Member

Let's swap the gray being used for the bottom tab bar to the default system gray color used for tabs. I'm not sure if the color has a name but it's #757575 in dark mode, #999999 in light mode. That will make the tab bar match what's seen in the mockups above.

nheagy added a commit that referenced this issue Aug 21, 2019
Dark mode: fixes for stats: maps and view more
nheagy added a commit that referenced this issue Aug 22, 2019
Dark mode: loading animations and no results view text
nheagy added a commit that referenced this issue Aug 22, 2019
Dark mode: remove shade label from semantic color functions
@frosty frosty mentioned this issue Aug 22, 2019
1 task
nheagy added a commit that referenced this issue Aug 23, 2019
This was referenced Aug 23, 2019
@frosty frosty mentioned this issue Aug 26, 2019
1 task
nheagy added a commit that referenced this issue Aug 26, 2019
@frosty
Copy link
Contributor

frosty commented Aug 26, 2019

My Sites subsections outstanding:

  • Post / pages tab bar style should match Stats
  • Themes support icon isn't being tinted correctly
  • Ghost view in posts / pages has a white background

nheagy added a commit that referenced this issue Aug 26, 2019
Dark Mode: My Sites section fixes part 1
nheagy added a commit that referenced this issue Aug 27, 2019
@jkmassel
Copy link
Contributor

jkmassel commented Aug 28, 2019

IMG_64CCB4F0DE2C-1

Adding this to the pile – the black-on-black text is hard to read

@mattmiklic
Copy link
Member

On the Menus screen:

  • The chevron-down icon and the icon that's used to drag and reorder items in the menu should both use the textTertiary color. The Page icon (beside "New item") and the pencil icon (beside "Default Menu") should use the listIcon color.

Simulator Screen Shot - iPhone Xs - 2019-09-05 at 15 27 35

@mattmiklic
Copy link
Member

Also in Menus, when editing a menu name in dark mode the background color is too light. For iOS 13 we should use tertiarySystemBackground and for iOS 12 and below we can use muriel Gray 80.

The text should be text color and the (X) icon should be listIcon.

Simulator Screen Shot - iPhone Xs - 2019-09-05 at 15 27 16

@danielebogo
Copy link
Contributor

Thanks @mattmiklic for these feedback! I already fixed some (even the page in Pages @iamthomasbishop )

@mattmiklic
Copy link
Member

Under Sharing > Sharing Buttons / Manage > Edit sharing buttons, the icons in the list should use listIcon.

Simulator Screen Shot - iPhone Xs - 2019-09-05 at 15 50 14

@mattmiklic
Copy link
Member

Also in Menus, the "Share this" and "Official buttons" text should use whatever we're using elsewhere in this kind of view. I think it's textSubtle but am not sure.

Simulator Screen Shot - iPhone Xs - 2019-09-05 at 15 51 11

@mattmiklic
Copy link
Member

mattmiklic commented Sep 6, 2019

This is testing 13.2.1.0 from Testflight:

On the New Account created screen, the background color behind the status bar is different from the color behind the rest of the page. The color behind the status bar is the one that's incorrect.

IMG_1142

The modal progress indicator that appears after hitting Continue on the New Account screen looks off; this should use a regular material background, a standard spinner, and text color text; but I'm not sure if that is time consuming. Maybe just using a system gray background in the interim would help so it responds to dark mode.

IMG_1143

There's a layout bug on Notifications the first time the screen is loaded. If the user hasn't already allowed notifications it causes the tab bar to be cut off.

IMG_1146

If the user has dismissed the push notification prompt, there's just a bunch of extra space above the first notification in the list.

IMG_8DEB8F00F29C-1

This isn't specific to dark mode but it is new as of 13.2.

@frosty
Copy link
Contributor

frosty commented Sep 9, 2019

I've added all the open reports in here to the task list at the top of this issue.

@rachelmcr
Copy link
Member

@frosty I've added the additional issues I saw to the task list (under My Sites, Editor, Reader). If anything is unclear please let me know and I can clarify with screenshots, etc.

@danielebogo
Copy link
Contributor

Thanks @rachelmcr !

@elibud
Copy link
Contributor

elibud commented Sep 19, 2019

Comment moderation has incorrect background on the comment cell.

File (2)

@melindahelt
Copy link

#2364138-zen

Reported: I have a bug in Wordpress iOS night mode. In my iOS 13.1 beta in paragraph and order text is white and background is white to! I hope you will update to fix this.

B01502C2-20E9-4568-AD13-16A538B3264C_L0_001
D2BC9489-D2F7-43F2-9A8B-AAC6741EDCC3_L0_001

@elibud
Copy link
Contributor

elibud commented Sep 20, 2019

Pinging @etoledom for that last one.

@iamthomasbishop
Copy link

@melindahelt @elibud that's on Aztec so I think @frosty might also be able to assist with that issue

@designsimply
Copy link
Contributor

Profile > Help & Support > My Tickets screen is still light colored:

IMG_3107

Tested with WPiOS 13.3.0.0 (TestFlight) on iPhone 6S iOS 13.1 beta 4.

@etoledom
Copy link
Contributor

etoledom commented Oct 1, 2019

@melindahelt @elibud that's on Aztec so I think @frosty might also be able to assist with that issue

I can take it during the week unless someone is already taking care of it. @frosty, @danielebogo ?

@danielebogo danielebogo added this to Backlog in Dark Mode Oct 1, 2019
Dark Mode automation moved this from References to Closed Oct 4, 2019
@frosty
Copy link
Contributor

frosty commented Oct 4, 2019

This issue closed because tracking of these issues has now moved over to a Github Project: https://github.com/orgs/wordpress-mobile/projects/45

@kean kean mentioned this issue May 2, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Dark Mode
  
Closed
Development

Successfully merging a pull request may close this issue.