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

Bad formatting for Maze Maker exercise #717

Closed
anshulvv opened this issue Oct 15, 2024 · 7 comments · Fixed by #718
Closed

Bad formatting for Maze Maker exercise #717

anshulvv opened this issue Oct 15, 2024 · 7 comments · Fixed by #718

Comments

@anshulvv
Copy link
Contributor

Bad formatting for the instructions sections of Maze Maker exercise.
Can be seen on the exercise's online editor.
https://exercism.org/tracks/elm/exercises/maze-maker/edit

image
@mpizenberg
Copy link
Member

Hum, I’m wondering if this is an issue in the viewer. I can’t find a source that would have line ending at this location.

@jiegillet
Copy link
Contributor

Definitely an issue with the viewer. The source for the instructions is this:

Define the `mazeOfDepth n` generator so that it generates a maze of depth `n`.
Only the deepest level `0` should contain dead ends or treasure rooms (with equal probability), all other levels should be a branch containing mazes of depth `n - 1` generated recursively with `mazeOfDepth`.
Make sure to use `deadend`, `room` and `branch` in the generator.

The viewer is interpreting the minus sign in n - 1 to be the first item of a list.
I have no idea how the viewer could be fixed, we could try the workaround of removing spaces in n-1?

@anshulvv
Copy link
Contributor Author

This line here in complex numbers exercise is similar (inline code with -):

The conjugate of the number `a + b * i` is the number `a - b * i`.

But it's formatting is correct in edit view
image

So, it's weird behavior.

@jiegillet's suggestion seem potential solution for now.

@ceddlyburge
Copy link
Contributor

Maybe the n - 1 is being confused with \n - 1 which would be a bulleted list.

In which case we could use a different symbol instead of n.

We could also try and escape the -?

Markdown supports that, so it would work in theory ...

@jiegillet
Copy link
Contributor

I doubt it thinks it's a \n because the n is rendered.
Escaping the - is my new favorite potential solution. let's try that!

@ceddlyburge
Copy link
Contributor

Ok, I'm happy to try that. I should have time tomorrow.

@jiegillet
Copy link
Contributor

This issue was closed automatically when we merged the PR, but I actually never checked that it worked.
It worked :)

Screenshot 2024-10-23 at 21 38 04

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 a pull request may close this issue.

4 participants