Skip to content
This repository has been archived by the owner on Apr 26, 2019. It is now read-only.

docs: updated content to reference IBM Design Language #355

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

shinytoyrobots
Copy link

I updated the content to remove references to Duo and for clarity.

NB: We should change the url, too...but will need to liaise with web team who built https://w3.ibm.com/design/essentials, as that links to /carbon-and-duo

Copy link
Member

@alisonjoseph alisonjoseph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

screen shot 2018-09-14 at 12 01 00 pm

Looks good to me, need to update the navigation also though, and this is currently an internal only page, so not sure if it should stay that way or switch to public? I don't see why it can't be public if we are removing "duo".

@shinytoyrobots
Copy link
Author

I updated the nav link.

I also then updated the subtitle so that it didn't read the same.

...I couldn't think of a good title that would fit neatly into our nav formatting. Best I could come up with was "IBM Design Language".

I would keep it internal until we can synchronize with https://w3.ibm.com/design/essentials so that the url doesn't read "carbon-and-duo", and then make it public.

@@ -2,7 +2,7 @@
"getting-started": {
"title": "Getting Started",
"subnav": {
"carbon-and-duo": "Carbon and Duo",
"carbon-and-duo": "IBM Design Language",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we change the subnav title, as well? The "carbon-and-duo" bit.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's the actual url, we could probably add a redirect though until https://w3.ibm.com/design/essentials can update their link.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we want to do a redirect on our side then yes, we can just resolved everything and update our subnav to "design-language" or something similar.

Copy link
Member

@alisonjoseph alisonjoseph Sep 18, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The title of the page is generated from the nav (I think) so we need to update everything all at once. Not sure on how to setup redirects with this version of the website so we should coordinate with the w3.ibm.com site on updating their url.

So still need to update, the nav, the folder name, the file names inside of pages and content.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shinytoyrobots can go ahead and update the title and nav link. Then we can submit a PR here to update the link on the design website https://github.ibm.com/Design/w3-design-site/blob/master/pages/design/essentials.js#L182

Copy link
Contributor

@claycrenshaw claycrenshaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving so I can do another pull and make edits.

@alisonjoseph
Copy link
Member

Hey @claycrenshaw we can't merge this in yet until the navigation issues are worked out. If you have edits go ahead and comment them here and @shinytoyrobots will update the PR.

@joshblack
Copy link
Contributor

If it's helpful, can use the GitHub Review UI to make comments on certain lines 👍 https://help.github.com/articles/about-pull-request-reviews/ talks about it a bit

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants