Skip to content

RSS Virtual Keyboard#1

Open
DKNatallia wants to merge 2 commits intomainfrom
gh-pages
Open

RSS Virtual Keyboard#1
DKNatallia wants to merge 2 commits intomainfrom
gh-pages

Conversation

@DKNatallia
Copy link
Owner

  1. Task: https://github.com/rolling-scopes-school/tasks/blob/master/tasks/virtual-keyboard/virtual-keyboard-en.md

Снимок экрана 2023-05-02 в 00 27 30

  1. https://dknatallia.github.io/Virtual-Keyboard/index.html
  2. Done 02.05.23 (deadline 02.05.23)
  3. Score: 110 / 100
    [+] the generation of DOM elements is implemented. body in the index.html is empty (can contain only script tag): +20
    [+ - ] pressing a key on a physical keyboard highlights the key on the virtual keyboard (you should check keystrokes of numbers, letters, punctuation marks, backspace, del (if it's present), enter, shift, alt, ctrl, tab, caps lock, space, arrow keys: +10
    [+] switching keyboard layouts between English and another language is implemented. Selected language should be saved and used on page reload. A keyboard shortcut for switching a language should be specified on the page: +15
    [+] mouse clicks on buttons of the virtual keyboard or pressing buttons on a physical keyboard inputs characters to the input field (text area): +15
    [+] animation of pressing a key is implemented: +15
    [+] usage of ES6+ features (classes, property destructuring, etc): +15
    [+] usage of ESLint: +10
    [+] requirements to the repository, commits and pull request are met: +10
    [-] there're errors related to the executable code (errors like favicon.ico: Failed to load resource: the server responded with a status of 404 are not taken into account) or there're eslint-config-airbnb-base warnings: -15
    [-] the repository (virtual-keyboard) or development branch (development) name does not comply with the requirements: -110
    [-] less than 5 commits are made or commits are made on the same day: -110 or expel from the course
    [-] a significant portion of the application was completed after the deadline: -110 or expel from the course

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