Navigation Menu

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

feat: Make the prelude available in description page #393

Merged
merged 4 commits into from Sep 29, 2021

Conversation

YoanwM
Copy link
Contributor

@YoanwM YoanwM commented May 12, 2021

This PR aims to add a Prelude pane in the /description view of an exercise (if its prelude.ml is nonempty).

Just to recall, this is intended to be used with learn-ocaml-mode, where the browser will only display questions (no code editor).

@YoanwM YoanwM force-pushed the text_tab_prelude branch 2 times, most recently from c700ead to 02f3f08 Compare May 14, 2021 09:11
@erikmd erikmd added the kind: enhancement Enhancement to an existing user-facing feature. label May 14, 2021
@yurug
Copy link
Collaborator

yurug commented Jun 12, 2021

It would be nice to have a description for this PR.

@erikmd erikmd self-assigned this Sep 20, 2021
@erikmd
Copy link
Member

erikmd commented Sep 20, 2021

Rebasing this PR…

Copy link
Member

@erikmd erikmd left a comment

Choose a reason for hiding this comment

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

@yurug

It would be nice to have a description for this PR.

Done.

I think this PR can now be merged.

FWIW, below are some screenshots in four different situations.

with a prelude / full mode

2021-09-21_Screenshot_description_prelude_full

with a prelude / responsive mode

2021-09-21_Screenshot_description_prelude_responsive

with no prelude / full mode

2021-09-21_Screenshot_description_no-prelude_full

with no prelude / responsive mode

2021-09-21_Screenshot_description_no-prelude_responsive

@erikmd erikmd requested a review from yurug September 21, 2021 01:24
@erikmd erikmd changed the title Make the prelude available in description page feat: Make the prelude available in description page Sep 26, 2021
@yurug yurug merged commit 51ed717 into ocaml-sf:master Sep 29, 2021
erikmd added a commit to pfitaxel/learn-ocaml that referenced this pull request Oct 3, 2021
* Make the prelude available in description page

* fix(description.html): Load learnocaml_tryocaml.css & Comment-out unneeded file

* fix(description): Initially show the Prelude

* fix(description): Simplify/Fix HTML+CSS+Js_of_ocaml for the prelude

Co-authored-by: Erik Martin-Dorel <erik.martin-dorel@irit.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: enhancement Enhancement to an existing user-facing feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants