Material-ui: [Docs] Style > Color: Palette Color Name, Font Display

Created on 2 Oct 2017  路  5Comments  路  Source: mui-org/material-ui


In the Docs for Style > Color the display of the actual Color Palette has three (potential) updates:

  1. The font is the browser default (should be according to the theme)
  2. The weight should be 500 (to match material.io)
  3. Color names should have capitalization and spaces in their display (while maintaining the need to lowerCase and remove spaces for theme purposes).

  • [x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior


image

Current Behavior


image

Steps to Reproduce (for bugs)

N/A

Context

Your Environment

| Tech | Version |
|--------------|---------|
| Material-UI | 1.0.0-beta.13 |
| React | 15.6.2 |
| browser | Chrome |
| etc | |

docs

All 5 comments

Adding the subsequent PR now.

  1. The font is the browser default (should be according to the theme)

I agree

  1. The weight should be 500 (to match material.io)

I agree

  1. Color names should have capitalization and spaces in their display (while maintaining the need to lowerCase and remove spaces for theme purposes).

I don't agree, it's on purpose. There is a direct mapping between the import path and the name here.

Sweet.

RE: 3:
I added the original color as an additional prop, though I will take that back out to keep display in the browser like:

deepPurple for example (instead of showing Deep Purple in the browser).

I will update the PR.

PR is now updated, this is how it will look:
image

@JeromeFitz Awesome :)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

ericraffin picture ericraffin  路  3Comments

ghost picture ghost  路  3Comments

revskill10 picture revskill10  路  3Comments

iamzhouyi picture iamzhouyi  路  3Comments

finaiized picture finaiized  路  3Comments