carbon-design-system / carbon-website-archive

The old website for the Carbon Design System. This repo is archived.
https://www.carbondesignsystem.com
238 stars 116 forks source link

[Dev]: Random cleanup issues #515

Closed alisonjoseph closed 5 years ago

alisonjoseph commented 5 years ago

List of dev issues on new site that need to be fixed/cleaned up

Staging link: https://deploy-preview-516--carbon-website.netlify.com/

claycrenshaw commented 5 years ago

~Anchor links need to be hooked up across the board.~ duplicate tracked in #513

claycrenshaw commented 5 years ago

On Chrome (on my machine, but not Tyler's, using the same version of Chrome) The side nav scroll bar is rendering white instead of the intended dark grey color.

image

should be tracked as UI Shell issue in components repo

claycrenshaw commented 5 years ago

~https://deploy-preview-516--carbon-website.netlify.com/guidelines/grid/design This page should be changed to "Layout" on the page itself, as well as in the nav. "Grid" will be removed as a label in the nav.~

this is being tracked here https://github.com/carbon-design-system/carbon-website/issues/484

claycrenshaw commented 5 years ago
claycrenshaw commented 5 years ago

~Bar graph component getting cut off on left side of page https://deploy-preview-516--carbon-website.netlify.com/data-visualization/bar-graph/code *This seems to be an issue with all of the data viz components.~

duplicate

claycrenshaw commented 5 years ago

https://deploy-preview-516--carbon-website.netlify.com/resources

image

designertyler commented 5 years ago

App switcher icon should be vertically centered. Right now there's too much space below.

screen shot 2018-12-06 at 9 30 34 am

@designertyler this should probably be created as a bug in carbon-components for the UI Shell.

designertyler commented 5 years ago

~"Skip to main content" appears in top left after clicking search link. Steps to reproduce: Search for a term in search bar. Click a link in the results. You're linked from staging site to prod. (Could be part of the issue)~

screen shot 2018-12-06 at 10 31 35 am screen shot 2018-12-06 at 9 16 15 am

being tracked here https://github.com/carbon-design-system/carbon-website/issues/574

jeanservaas commented 5 years ago

Just noticed that the H3s (sorry, I referred to them as secondary) are the wrong size... they're a little big.

They need to be 20px or 1.25rem... right now they're set at 1.5rem and they look very close to the Headlines.

screen shot 2018-12-05 at 8 59 43 am

jeanservaas commented 5 years ago

They should look like this to align with IDL:

Primary index style

enabled image

hover: image

Secondary anchor link index style

This only happens on the Layout page, but we will use it on other pages too:

Type size:

16px (1rem) 24 px (1.5rem) image

instead of this:

image

alisonjoseph commented 5 years ago

Homepage todo:

jeanservaas commented 5 years ago

Type tokens/classes and alignment


image



image


image

image

Also, this one should follow the same pattern

image

image


image


Also found on: Getting started Developers, Accessibility (a lot), Support and FAQ

image


Captions are not done in italic now (see layout page for correct treatment) — they pull $body-short-01

image

image


Image sizing

image


Inline Code Snippet Component

image


Type Spec Component

image

image


Code Snippet Spec Component

image


Spacing

Type page examples

screen shot 2018-12-14 at 9 54 59 am

image


Table (Structured list) Component

it does actually looks like you're using two sizes now... I see the 12 col component on the accessibility page and a shorter one on layout

Not ideal (we want to avoid these long line lengths and make sure the first column has enough space so it doesn't have to break) Examples of these are on the Spacing, Accessibility and Disabled State and Overflow Content pages:

image

image

Ideal:

image

image

image

alisonjoseph commented 5 years ago

@jeanservaas moving this to its own issue since thats a lot of stuff https://github.com/carbon-design-system/carbon-website/issues/608

also closing out this one