Page MenuHomePhabricator

Create visalization for shadows
Closed, ResolvedPublic

Description

Create some sort of box with the box-shadow values applied.
Maybe something with a light color background which pops on the white background thanks to the shadow.

Event Timeline

Improvement suggestions:

  1. The 2 global tokens (box-shadow-inset-thin, box-shadow-inset-thick) should go on top
  2. The font size of the text in the duration value column (<pre>) should be 14px.
  3. The value text should be outside the specimen
  4. The speciment should be smaller (suggestion: 32px in height)
  5. The bg color of the specimens (#ffe599) does not help visualizing the inset. I'd suggest using a gray (e.g. #c8ccd1).

Improvement suggestions:

  1. The 2 global tokens (box-shadow-inset-thin, box-shadow-inset-thick) should go on top
  2. The font size of the text in the duration value column (<pre>) should be 14px.
  3. The value text should be outside the specimen
  4. The speciment should be smaller (suggestion: 32px in height)
  5. The bg color of the specimens (#ffe599) does not help visualizing the inset. I'd suggest using a gray (e.g. #c8ccd1).

https://github.com/wmde/wikit/pull/58
FYI: The first point is not addressed, because it spiraled a bigger discussion. The outcome of which is https://phabricator.wikimedia.org/T257065

those are indeed some lovely rectangles. looking good! ready to move to done imo (not sure if you need to wait for code review)