Skip to content
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 additional buttons to Course Explorer #43687

Merged
merged 2 commits into from Nov 19, 2021
Merged

Add additional buttons to Course Explorer #43687

merged 2 commits into from Nov 19, 2021

Conversation

Erin007
Copy link
Contributor

@Erin007 Erin007 commented Nov 18, 2021

LP-2108

BEFORE

The CSF entry on the Course Explorer didn't have a button linking to the course, because CSF is a collection of 6 courses.

Screen Shot 2021-11-18 at 3 43 19 PM

And there wasn't a "Learn More" link for the Pre-Reader Express Course.

Screen Shot 2021-11-18 at 4 59 13 PM

AFTER

Now, to be more consistent with the other course offerings, we have a "View Course" button that links to the overview page of all of the CSF courses.

Screen.Recording.2021-11-18.at.3.41.25.PM.mov

And there is a "Learn More" button for the Pre-Reader course that links to the Express overview page.

Screen.Recording.2021-11-18.at.4.58.42.PM.mov

@Erin007 Erin007 requested a review from a team November 18, 2021 20:47
@Erin007 Erin007 changed the title Add view course button to CSF entry in Course Explorer Add additional buttons to Course Explorer Nov 18, 2021
Copy link
Contributor

@jamescodeorg jamescodeorg left a comment

Choose a reason for hiding this comment

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

Nice!

@Erin007 Erin007 merged commit ee739bf into staging Nov 19, 2021
@Erin007 Erin007 deleted the csf-course-button branch November 19, 2021 17:59
snickell pushed a commit that referenced this pull request Feb 3, 2024
Add additional buttons to Course Explorer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants