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

Post visibility unification #17686

Closed
karmatosed opened this issue Oct 1, 2019 · 5 comments · Fixed by #17731
Closed

Post visibility unification #17686

karmatosed opened this issue Oct 1, 2019 · 5 comments · Fixed by #17731
Assignees
Labels
[Status] In Progress Tracking issues with work in progress

Comments

@karmatosed
Copy link
Member

Currently, the post visibility has 2 styling options for the same content. A baseline would be unify radios, default states and copy formatting.

image

image

@karmatosed karmatosed added the Needs Design Feedback Needs general design feedback. label Oct 1, 2019
@enriquesanchez
Copy link
Contributor

The alignment and spacing issues on that second screenshot have been fixed on #17613. This is how they both look on master right now:

Screen Shot 2019-10-01 at 5 22 54 PM

I notice a slight variation of color on the text, but upon quickly inspecting the CSS they both seem to have the same color assigned (color: #82878c;). This is odd 🤔.

@richtabor
Copy link
Member

It looks like the .editor-post-publish-panel style of color: #555d66; is being applied to the whole panel, styling post visibility properly - whereas .edit-post-post-visibility__dialog-info seems to be inheriting the body color of #444.

I've changed it to #d52411 to show it:
Screen Shot 2019-10-02 at 5 36 38 PM

@enriquesanchez
Copy link
Contributor

Oh interesting. Is there any reason why we wouldn't want to use the body color (#444) in this case?

@github-actions github-actions bot added the [Status] In Progress Tracking issues with work in progress label Oct 2, 2019
@enriquesanchez
Copy link
Contributor

@karmatosed @richtabor I created a PR (#17731) to fix the text color variation. Please let me know if you have any feedback

@karmatosed karmatosed removed the Needs Design Feedback Needs general design feedback. label Oct 7, 2019
@karmatosed
Copy link
Member Author

Oh interesting. Is there any reason why we wouldn't want to use the body color (#444) in this case?

Not sure there is and consistency would be great to start from.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Status] In Progress Tracking issues with work in progress
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants