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

docs: add clarification to the task audio player #1710

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

radomskaia
Copy link
Contributor

Title of Pull Request

Add clarification to the task audio player

🤔 This is a ...

  • 🐛 Fix in a task or related content

Description

  • Brief Overview:

Add a more detailed description to the task audio player (meaning of the Pause button, moving the thumb on progress bar)
Add a rating criterion - autoplay next track

  • Implementation Approach:

help students better understand the technical task

@lyutails
Copy link
Collaborator

Now cannot find you @EternalRival to add as reviewer maybe go review too if you are in the context :)

tasks/js30#/js30-2.md Show resolved Hide resolved
tasks/js30#/js30-2.md Outdated Show resolved Hide resolved
@@ -89,7 +90,7 @@ js30 - три небольших js-проекта, которые вы выпо

## Требования к репозиторию

- задание выполняется в приватном репозитории школы. [Как работать с приватным репозиторием школы](https://docs.rs.school/#/private-repository)
- задание выполняется в приватном репозитории школы. Если прошлое задание было сделано в приватном репозитории школы - продолжить работать надо в нем же [Как работать с приватным репозиторием школы](https://docs.rs.school/#/private-repository)
Copy link
Contributor

Choose a reason for hiding this comment

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

не совсем понимаю про что это

Copy link
Contributor Author

Choose a reason for hiding this comment

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

У меня из 4 работ только одна в школьном репозитории.
При этом Шелтер у них в нем, значит школьный репозиторий имеется.
И много вопросов было про "Ну там же Шелтер, значит надо новый репозиторий создавать?"
При этом в Документации школы обозначено, что в одном репозитории будет несколько папок.
Я и подумала, можно еще указать на это дополнительно.

Copy link
Contributor

@EternalRival EternalRival Sep 24, 2024

Choose a reason for hiding this comment

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

кмк, лучше было бы, чтоб тз конкретного задания не ссылалось на другие задания (было самодостаточным, изолированным), т.к. контекст в другой раз может быть другим (порядок заданий в расписании и т.п.)

более того, на текущем потоке "прошлое задание" - мемслайдер, которое "было сделано" не "в приватном репозитории школы" :D

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Соглашусь
Как тогда решить проблему с непониманием, что в репозитории может быть несколько папок с тасками?
Или пусть делают, как делают?

Copy link
Contributor

@EternalRival EternalRival Sep 24, 2024

Choose a reason for hiding this comment

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

тут скорее стоит освежить саму страницу документации, на которую приведена ссылка (напр. дополнить faq).

помню, что когда-то пришла идея - переписать странички про приватный репо, пуллреквесты, сабмиты и т.п., но никогда не удавалось собрать все мысли в кучу :)

т.е. идея в том, чтобы расписать чуть подробнее "что? зачем? куда?", т.к. вопрос "а что нужно сабмитить?" возникает из-за непонимания принципа, по которому авторами выбран тот или иной способ :D

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image

Мне кажется, как раз написано все понятно. Просто никто не переходит по ссылке и не читает(

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Исправила на "в отдельной папке под задание". Так лучше?
Или убрать?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Мне кажется, путаница из-за того, что дальше написано про личный репо.
Я бы предложила убрать тут уточнение про папку, т.к. оно излишне, и по ссылке всё расписано + убрать пункт про личный репо, т.к. это не частая ситуация и обычно она решается в частном порядке

tasks/js30#/js30.md Outdated Show resolved Hide resolved
- внешний вид и функционал кнопки Play/Pause изменяется в зависимости от того, проигрывается ли в данный момент аудиотрек +5
3. При кликах по кнопкам "Вперёд" и "Назад" переключается проигрываемый аудиотрек. Аудиотреки пролистываются по кругу - после последнего идёт первый +10
3. При кликах по кнопкам "Вперёд" и "Назад" переключается проигрываемый аудиотрек. Аудиотреки пролистываются по кругу - после последнего идёт первый. После окончания воспроизведения текущего аудиотрека - автоматически включается следующий. +10
Copy link
Collaborator

Choose a reason for hiding this comment

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

Есть ощущение, что этот критерий вполне себе отлично делится на 2, а то и на 3.
Один - про переключение треков кнопками
Второй - про автоматическое переключение треков
Возможный третий - про пролистывание треков по кругу, независимо от способа переключения

Copy link
Contributor

Choose a reason for hiding this comment

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

согласен. делить критерии - здорово! 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants