ddp/photoassistant
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix BUG-104, dont process keybindings when DBox with text inputs is shown

Browse Source
This commit is contained in:
Damien De Paoli
2022-09-20 23:08:45 +10:00
parent 71ec54807a
commit 25919f5a5c
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
BUGs
View File

@@ -1,6 +1,5 @@
### Next: 107 ### Next: 107
BUG-100: I managed to get 2 photos matching mich in the NOT_WORKING photo (probably dif refimgs but same p.tag?) BUG-100: I managed to get 2 photos matching mich in the NOT_WORKING photo (probably dif refimgs but same p.tag?)
BUG-104: in popup to search/add, etc. the underlying key bindings still fire (e.g. l -> log)
BUG-105: show unmatched needs to handle overrides (ignore face, etc.) BUG-105: show unmatched needs to handle overrides (ignore face, etc.)
BUG-106: cant add trudy /pat? as refimgs via FaceDBox BUG-106: cant add trudy /pat? as refimgs via FaceDBox
- seems the cropped trudy face is not sufficient to find a face, how odd... - seems the cropped trudy face is not sufficient to find a face, how odd...

4
templates/viewer.html
View File

@@ -280,6 +280,10 @@
{% block script_content %} {% block script_content %}
<script> <script>
$( document ).keydown(function(event) { $( document ).keydown(function(event) {
// if dbox is visible, dont process this hot-key, we are inputting text
// into inputs instead
if( $("#dbox").is(':visible') )
return
switch (event.key) switch (event.key)
{ {
case "Left": // IE/Edge specific value case "Left": // IE/Edge specific value