Page MenuHomePhabricator

Remove shadow from the Toolbar on Reading themes (dark, black, sepia)
Closed, ResolvedPublic

Description

Why are we doing this?

Due to the lower contrast on secondary reading themes with shadows, we have removed shadows from the Explore feed and other parts of the app for these themes. We would like to be consistent and not show a shadow on Dark, Black and Sepia in the editing mode.

User story

As a user of secondary reading themes, I would like my experience to be consistent.

Proposed solution

Remove the shadow from the editing toolbar on Dark, Black and Sepia

Sepia

CurrentUpdated
IMG_3476.PNG (2×1 px, 513 KB)
03 Template collapsed SEPIA.png (1×750 px, 197 KB)

Dark + Black

CurrentUpdated DarkUpdated Black
IMG_3480.PNG (2×1 px, 561 KB)
03 Template collapsed Dark.png (1×750 px, 211 KB)
03 Template collapsed Black.png (1×750 px, 189 KB)

Event Timeline

cmadeo renamed this task from Audit of shadows for non-default reading themes throughout the app to Remove shadow from the Toolbar on Reading themes (dark, black, sepia) .May 2 2019, 7:05 PM
cmadeo updated the task description. (Show Details)

@cmadeo are the color changes on the toolbar and navbar intentional and a part of this ticket or should the only change be the shadow removal?

@JoeWalsh Good eye, sorry the header (nav bar) should not be included (this was my mistake), but the editing toolbar should be updated so that there is enough depth without the shadow. Thank you!

@cmadeo would you want to remove the shadows from the input views too (& update the bg color)?

Simulator Screen Shot - iPhone Xʀ - 2019-05-14 at 12.25.54.png (1×828 px, 256 KB)

Simulator Screen Shot - iPhone Xʀ - 2019-05-14 at 12.26.09.png (1×828 px, 255 KB)


For color updates, would you want to go with the ones in the screenshots or something else?

I picked new colors with Color Picker so they're probably off 🙈

current sepia f0e6d6 -> new sepia eee6d8?
current dark 27292D -> new dark 27292d?
current black 222222 -> new black 27292d?

@NHarateh_WMF Ah! Yes, if you could remove them from the input views too that would be great!
Sorry! I should have linked to Zeplin: https://app.zeplin.io/project/57a120ce9787dcf26230651f/dashboard?seid=5c367a1180fc3e29bdfba1d0

ThemeNew color
Black#27292d
Dark#27292d
Sepia#F0E6D6