Edits to New Capitalization Page

MeridelW commented 6 years ago

Copy edits and design suggestions for the new capitalization page. New Page: https://firefoxux.github.io/photon-staging/copy/capitalization.html#platform-specific-styles

Requested Edits: https://docs.google.com/document/d/1iHYyO1kCN78uLFcgaU-Xts1ywqNvO4ngWs-pX8eUeFc/edit?usp=sharing

Happy to discuss if needed!

aminalhazwani commented 6 years ago

Hey @MeridelW, we pushed a first update to staging https://firefoxux.github.io/photon-staging/copy/capitalization.html

A couple of considerations and questions:

Images don't share the same height, yet!
On a two-col layout we place pictures on the left and texts on the right. But when we have more than one picture we first display text full-width and then pictures on a two-col layout. I think that some content parity is missing. Isn't it odd that sometimes the text comes before the pictures and sometimes after? What do you think? (Screenshot below)
Most of the other things outlined in the doc should be fixed. Let us know if we missed anything!

aminalhazwani commented 6 years ago

Hey @MeridelW we pushed new changes to the page https://firefoxux.github.io/photon-staging/copy/capitalization.html. Now the images share all the same height. For the magnifying glass we are going to put it on hold until we figure out a way to apply it everywhere.

The images were the last missing piece for this page. Let us know if anything is missing, we merge to master next Tuesday, February 27th.

MeridelW commented 6 years ago

Hi Amin! Thank you for doing this work. The page is looking good, especially with the images all being the same height. =)

Two notes:

I am still struggling with the lay-out of the Capitalization Styles section. The amount of copy we have there, in particular for explaining Title Case, doesn't work well with the image hanging to the side of it. Would it be possible to have single images stretch across beneath the text, as we do in the attached?
Should we toggle the Capitalization styles section? It may make it easier for the user to digest the content there if she can see each style in isolation, as we do in the Platform section.

Please keep in mind these are nice-to-have changes that can be saved for another time (and also keep in mind that I don't have a good sense of how long these design changes take!).

On Fri, Feb 23, 2018 at 10:14 AM, Amin Al Hazwani notifications@github.com wrote:

Hey @MeridelW https://github.com/meridelw we pushed new changes to the page https://firefoxux.github.io/photon-staging/copy/capitalization.html. Now the images share all the same height. For the magnifying glass we are going to put it on hold until we figure out a way to apply it everywhere.

The images were the last missing piece for this page. Let us know if anything is missing, we merge to master next Tuesday, February 27th.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/FirefoxUX/photon/issues/274#issuecomment-368055910, or mute the thread https://github.com/notifications/unsubscribe-auth/AgR7ccBdkt8vx8Kht_L_hooC5XVErUO9ks5tXuPWgaJpZM4SGAZm .

aminalhazwani commented 6 years ago

@aminalhazwani reduce the list indentation ☺️

FirefoxUX / photon

Edits to New Capitalization Page #274