Skip to content

feat: lovely lesson layout #16

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 2 commits into from
Jul 26, 2018
Merged

feat: lovely lesson layout #16

merged 2 commits into from
Jul 26, 2018

Conversation

olizilla
Copy link
Collaborator

WIP on #15

Initial
screenshot 2018-07-26 15 09 07

oh noes
screenshot 2018-07-26 15 09 56

YASSSSSS QUEEEEN
screenshot 2018-07-26 15 10 26

License: MIT
Signed-off-by: Oli Evans [email protected]

WIP on #15

License: MIT
Signed-off-by: Oli Evans <[email protected]>
@olizilla olizilla requested a review from mikeal July 26, 2018 14:13
License: MIT
Signed-off-by: Oli Evans <[email protected]>
@olizilla
Copy link
Collaborator Author

Added styles for inline code and code blocks in lesson text

screenshot 2018-07-26 16 25 16

@mikeal mikeal merged commit ffc6a28 into master Jul 26, 2018
@mikeal
Copy link
Member

mikeal commented Jul 26, 2018

Love it!

@mikeal
Copy link
Member

mikeal commented Jul 26, 2018

One thing we lost was the "next lesson" button on success. Was that an oversight or is there a design reason you think it should be removed?

@olizilla
Copy link
Collaborator Author

Submit button should be replaced with the Next button. Perhaps that's too subtle.

@mikeal
Copy link
Member

mikeal commented Jul 26, 2018

If we change the button color, maybe animate it, it'll be pretty obvious :)

@olizilla
Copy link
Collaborator Author

just to check, is the next button showing for you? It should already be changing colour and "submit" should be replaced with "next" when you complete an exercise.

@mikeal
Copy link
Member

mikeal commented Jul 26, 2018

It is, I guess I just didn't notice it :(

Maybe we should add a bouncing CSS animation?

@daviddias daviddias deleted the nice-lessons branch August 4, 2018 03:57
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.

2 participants