GitLab

Merge misc changes

See merge request !21

Image tag contains useless style and "what's this?" is updated to
reflect that the application already provides external resources.

These optimization decisions were made by using profiling data. I used
responsive design mode and rapidly played keys with touch simulation.
This helped me understand the bottlenecks that caused lag while the user
is playing the piano.

- Check spec only in development. Massive performance boost because 40%
of the time was used in validating specification

- Use `[chord-info-component]` instead of `(chord-info-component)` which
is faster since `[]` form is lazy.

- Modify touch object handling. Instead of converting `Touch` object to
a map, it's kept as is and the keys are taken directly in
`process-touches`. Gives a considerable performance boost because touch
handling is done often when the user plays the keyboard.

They are now bigger (to make it easier to press) and have 2 octaves
instead of 1. This change also includes update to `kbd->semitone` by
including as many characters as possible in a single keyboard row.

Merge miscellaneous changes

See merge request !20

Includes the main idea of the application and a link to the source code.
This is done, so that visitors have a better understanding of the
application and can take a look at the source code.

Furthermore, this could be a place to add extra information, such as
email, in case people would like to ask about this.

- Use grid instead of "X: Y" format
- Bold the left column

Since piano hover style looks bad on touch devices, it had to be
removed. Since there should be an indication of playability,
cursor-pointer is used to replace the old hover style.

Make expandable-container-component responsive

See merge request !19

chord-info-component now handles only y-overflow. This was done so that
info piano does not trigger horizontal scrolling on the container as a
whole.

Since the only element that is guaranteed to be too wide is piano
component, we set "overflow-x-scroll" there, so that only that part is
horizontally scrollable. Since unnecessary horizontal scrolling tends to
be annoying we restrict it as much as possible.

When the width is too small (as defined in re-frame-util ns) the
main-component and side-component will just stack.

Note: Currently the "side-component" in small width is rough because the
overflow is not handled nicely.

Merge `misc` into `master`

See merge request !18

This fixes a bug where touch device users could play the piano by
pressing the info component piano which is obviously undesirable.

Since it cannot be played, it should not have hover state that implies
that the info component piano can be played.

Implement last played chord

See merge request !17

Implement external resources feature

See merge request !16

Update README

See merge request !15

Add localization

See merge request !14

Refactor re-frame database

See merge request !13

Fix failed schema on initialization

See merge request !12