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

fix: a few clarifications for the "Hangman" task #1467

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

Conversation

Svetlaya87
Copy link

🤔 This is a ...

  • 🌟 New task
  • 🌐 New module
  • ⚙️ Update to an existing task
  • 🔧 Update to an existing module
  • 🔗 Update or addition of external resources or links
  • 🐛 Fix in a task or related content
  • 🛠 Fix in a module or related content
  • ✏️ Fixed a typo or grammatical error
  • 🔗 Fixed a broken link
  • ❓ Other (specify: ****____****)

Description

  • Brief Overview:
  • Add link to repo. Without the link it wasn't clear how to correctly create a new task in private repository.
  • Add new condition for check. For example pressing alt-tab must not count as an incorrect try.
  • Add new penalty to avoid the entering a large chunk or whole html code via innerHTML, insertAdjacentHTML.
  • Add new links in "Useful links". In some works after the window with wrong msg opened, the letters can be entered and then, if the all letters are correct the "loss" msg changes to "win" msg in opened window. I have added the links to clarify the info about disabled elements to avoid such situations.

@valerydluski valerydluski changed the title A few clarifications for the "Hangman" task fix: a few clarifications for the "Hangman" task Jan 20, 2024
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.

3 participants