Change hours input from time to text + Make overlays close on outer click

Description

• Overlays now close when clicking outside of the box
• The hours input has been changed from a time type to a text type. This helps to prevent browser errors or needing to add AM/PM.

Related Issues

• Resolves #68 (closed)

Checklist

• I have added a changelog entry to reflect the significant changes I made.

changed milestone to %Working v2.0

added typebug + 1 deleted label

assigned to @toberhuber

requested review from @rbackx

changed title from Change hours input from time to text + Make overlay close on outer click to Change hours input from time to text + Make overlays close on outer click

changed the description

added 1 commit

• f99f5cde - Make closing overlays global

Compare with previous version

Most overlays should not close when clicking outside, only the ones that are not some kind of form. Ideally we check if an overlay has some data-closable attribute and if it does, it can be closed by clicking outside.

Submitting a declaration gives an error

Putting :00 in the box changes the value to NaN:00

added 1 commit

• 04613a71 - Fix declaration bugs

Compare with previous version

resolved all threads

added 5 commits

• 04613a71...05960151 - 4 commits from branch dev
• 18c3c199 - Merge branch 'dev' into '68-new-declaration-errors'

Compare with previous version

One more small thing: When entering a declaration time, if someone just enters 5, the time should be 5:00 and not give an error

added 1 commit

• 8f7f1aa0 - Change to JQuery + Fix declaration bug

Compare with previous version

resolved all threads

added 1 commit

• 465bb3b4 - Fix bug with buttons not clicking

Compare with previous version

approved this merge request

added this merge request to the merge train at position 2

removed this merge request from the merge train because 9:Failed to create merge commit for source_sha 465bb3b4 and target_sha c4288e77 at refs/merge-requests/189/train.

added 28 commits

• 465bb3b4...7870996a - 27 commits from branch dev
• ad4b20fc - Merge branch 'dev' into '68-new-declaration-errors'

Compare with previous version

approved this merge request

started a merge train

merged

mentioned in commit e4c28f41

unassigned @toberhuber