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

Přesunutí řešení LeviExpress #791

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jakubtopic
Copy link

Řešení u LeviExpressu se vykresluje jako klasická lekce, ne jako cvičení. Takže to má nějaké divné okraje a nefunguje skrývání. Pokud by byly další projekty se samostatnou stránkou pro řešení, tak by to asi chtělo upravit na kodim :))

Copy link
Contributor

@FilipJirsak FilipJirsak left a comment

Choose a reason for hiding this comment

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

V materiálech jsem to dal schválně jako samostatnou kapitolu, aby to nebylo utopené u posledního úkolu. Myslím, že v případě celodenních projektů není nutné řešení skrývat – stejně by se pravděpodobně odemykalo na konci lekce.

@jakubtopic
Copy link
Author

jakubtopic commented Jun 24, 2024

Rozumím. Takhle to stále bylo jako samostatná kapitola, jen to vyžadovalo jedno kliknutí navíc – podobně jako ostatní stránky s cvičeními, kde je pouze jedno cvičení. Například hned začátek druhé části leviexpresu.

Jelikož není nutné řešení skrývat, vrátil jsem to zpět a pouze odstranil hlavičku cvičení a blok :::solution, který způsoboval chybějící okraje :)

edit: rebased to master

@jakubtopic jakubtopic force-pushed the fix-leviexpress-reseni branch from 48139ec to dd2f5f5 Compare June 24, 2024 13:30
@jakubtopic jakubtopic requested a review from FilipJirsak June 24, 2024 13:30
Řešení je v samostatné kapitole, kterou Kódím vykresluje jako klasickou
lekci, a proto blok :::solution způsoboval chybějící okraje.
@jakubtopic jakubtopic force-pushed the fix-leviexpress-reseni branch from 5053489 to ba65ba1 Compare June 24, 2024 20:10
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