-
Notifications
You must be signed in to change notification settings - Fork 479
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
add buttons directly to new CSD and CSP course overview pages #15799
Conversation
@@ -5,7 +5,7 @@ video_player: true | |||
theme: responsive | |||
--- | |||
|
|||
# Our newest course: CS Discoveries # | |||
# Our newest course: CS Discoveries [<button style="margin-top:0;">View course</button>](https://studio.code.org/courses/csd) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want relative links here?
@@ -5,7 +5,7 @@ video_player: true | |||
theme: responsive | |||
--- | |||
|
|||
# Our newest course: CS Discoveries # | |||
# Our newest course: CS Discoveries [<button style="margin-top:0;">View course</button>](https://studio.code.org/courses/csd) | |||
|
|||
## <a name="what"></a>What is CS Discoveries? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This name isn't descriptive. Maybe name="CSDlink"
or something related?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh I thought you wanted me to add a name to my link. hahah I see this is existing stuff. It's not the best, but I don't want to update this because Josh made these anchor links. I could update site wide but he may already be using these links in his resources. Considering how small a difference the name makes, I don't think it's worth cleaning up now but I can include a tip in our editor guides about choosing more descriptive names for future times.
CSD
CSP Desktop
CSP Tablet
CSP Mobile