TorstenHettstedt/PHP-App-Timekeeping
1
0
Fork 0
Code Issues Pull Requests Projects Releases 3 Wiki Activity

js-rest-aktionen #6

Merged
TorstenHettstedt merged 9 commits from js-rest-aktionen into master 2021-04-12 14:11:32 +02:00
Conversation 2 Commits 9 Files Changed 11
+4 -4
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ae7491f002 - Show all commits
ui/src/routes/WorkingHours.svelte
+4 -4
View File
@@ -127,7 +127,7 @@
<!--suppress HtmlUnknownTarget -->
<form action="/working-hours">
<fieldset>
<button on:click={() => {activeRecord = {}; newRecord = true}}>Neuer Eintrag
<button on:click={() => {activeRecord = {workingTime: '00:00:00'}; newRecord = true}}>Neuer Eintrag
TorstenHettstedt marked this conversation as resolved Outdated
TorstenHettstedt commented 2021-04-12 13:22:49 +02:00
Outdated
Review
Copy Link
Copy Source

Es ist eine Überlegung wert, die einzelnen anonymen Funktionen zu einem Objekt zusammenzufassen.

Es ist eine Überlegung wert, die einzelnen anonymen Funktionen zu einem Objekt zusammenzufassen.
<IconifyIcon icon={documentIcon}/>
</button>
</fieldset>
@@ -168,13 +168,13 @@
<fieldset name="record-data">
<div>
<label for="workingDay">Arbeitstag</label>
<input bind:value={activeRecord.workingDay} placeholder="Arbeitstag" type="date"
id="workingDay">
<input bind:value={activeRecord.workingDay} type="date"
TorstenHettstedt marked this conversation as resolved Outdated
TorstenHettstedt commented 2021-04-12 11:33:13 +02:00
Outdated
Review
Copy Link
Copy Source

Es sollte das aktuelle Datum als Defaultwert genommen werden.

Es sollte das aktuelle Datum als Defaultwert genommen werden.
TorstenHettstedt commented 2021-04-12 12:16:09 +02:00
Outdated
Review
Copy Link
Copy Source

Wird leider nicht unterstützt.

Wird leider nicht unterstützt.
id="workingDay" required pattern="\d{4}-\d{2}-\d{2}">
</div>
<div>
<label for="workingTime">Arbeitszeit</label>
<input bind:value={activeRecord.workingTime} placeholder="Arbeitszeit" type="time"
TorstenHettstedt marked this conversation as resolved
TorstenHettstedt commented 2021-04-12 11:34:09 +02:00
Review
Copy Link
Copy Source

Es sollten auch die Sekunden eintragbar sein und der Defaultwert sollte '00:00:00' sein.

Es sollten auch die Sekunden eintragbar sein und der Defaultwert sollte '00:00:00' sein.
id="workingTime">
id="workingTime" required pattern="[0-5]\d:[0-5]\d:[0-5]\d">
</div>
</fieldset>
<fieldset name="buttons">
1