Skip to content

Add carousel of code snippets to front page #1462

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

Merged
merged 9 commits into from
Feb 6, 2023

Conversation

bishabosha
Copy link
Member

@bishabosha bishabosha commented Jan 31, 2023

The idea here is to add a carousel that is immediately visible. Each slide comes with a title, code snippet, Scastie link, and summary. it can be operated without JS on, but when JS is on then it changes slides every 8 seconds, until the user interacts with the carousel, which cancels the timer.

Also added a dark mode theme for code on the front page.

Screenshot 2023-01-31 at 14 00 29

Screenshot 2023-01-31 at 13 59 54

Also tag @romanowski @Kordyjan for review

these are only visible with javascript enabled,
which will inject hljs dark theme.
Copy link
Member

@sjrd sjrd left a comment

Choose a reason for hiding this comment

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

Looks great. I only have minor internal comments.

@SethTisue SethTisue self-assigned this Jan 31, 2023
Copy link
Contributor

@julienrf julienrf left a comment

Choose a reason for hiding this comment

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

This is a great improvement, thank you Jamie, please find my suggestions and questions below.

@bishabosha
Copy link
Member Author

added a compact view for buttons that expands on narrow screens:
Screenshot 2023-02-01 at 17 02 28

@bishabosha bishabosha requested a review from julienrf February 1, 2023 16:07
@bishabosha
Copy link
Member Author

bishabosha commented Feb 4, 2023

trying a different header colour:
Edit (now result of 26f0013)
Screenshot 2023-02-04 at 23 32 01

@sjrd
Copy link
Member

sjrd commented Feb 6, 2023

Let's merge this. It seems like a definite improvement over the status quo. We can always further iterate later.

@sjrd sjrd merged commit b9ffe80 into scala:main Feb 6, 2023
@bishabosha bishabosha deleted the add-code-carousel branch February 6, 2023 09:00
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.

4 participants