Fossil SCM
/chat: 1) force the cursor on the input element to always be 'text' (I-beam), even when it doesn't have focus (browser inconsistency) and 2) when the chat window regains visibility after having lost it, give focus to the input element to try to work around a problem where Pale Moon is giving focus to one of the pseudo-buttons. Change (2) comes with a couple of minor but annoying quirks and might have to be undone, but it's worth a try.
Commit
503395796e5f9772fc519b3b0399c848a5fd6d5a85128d0e04fbf0e7fd1b3a0d
Parent
3f736de9867d85e…
2 files changed
+9
+3
| --- src/fossil.page.chat.js | ||
| +++ src/fossil.page.chat.js | ||
| @@ -780,10 +780,19 @@ | ||
| 780 | 780 | }; |
| 781 | 781 | document.addEventListener('visibilitychange', function(ev){ |
| 782 | 782 | cs.pageIsActive = ('visible' === document.visibilityState); |
| 783 | 783 | if(cs.pageIsActive){ |
| 784 | 784 | cs.e.pageTitle.innerText = cs.pageTitleOrig; |
| 785 | + console.debug("Taking back input focus."); | |
| 786 | + cs.inputFocus( | |
| 787 | + /* An attempt to resolve usability problem reported by Joe | |
| 788 | + M. where the Pale Moon browser is giving input focus to | |
| 789 | + the Preview button. The down-side of this is that it will | |
| 790 | + deselect any text which was previously selected on this | |
| 791 | + page. This also, unfortunately, places the focus at the | |
| 792 | + start of the element, rather than the last cursor position | |
| 793 | + (like a textarea would). */); | |
| 785 | 794 | } |
| 786 | 795 | }, true); |
| 787 | 796 | cs.setCurrentView(cs.e.viewMessages); |
| 788 | 797 | |
| 789 | 798 | cs.e.activeUserList.addEventListener('click', function f(ev){ |
| 790 | 799 |
| --- src/fossil.page.chat.js | |
| +++ src/fossil.page.chat.js | |
| @@ -780,10 +780,19 @@ | |
| 780 | }; |
| 781 | document.addEventListener('visibilitychange', function(ev){ |
| 782 | cs.pageIsActive = ('visible' === document.visibilityState); |
| 783 | if(cs.pageIsActive){ |
| 784 | cs.e.pageTitle.innerText = cs.pageTitleOrig; |
| 785 | } |
| 786 | }, true); |
| 787 | cs.setCurrentView(cs.e.viewMessages); |
| 788 | |
| 789 | cs.e.activeUserList.addEventListener('click', function f(ev){ |
| 790 |
| --- src/fossil.page.chat.js | |
| +++ src/fossil.page.chat.js | |
| @@ -780,10 +780,19 @@ | |
| 780 | }; |
| 781 | document.addEventListener('visibilitychange', function(ev){ |
| 782 | cs.pageIsActive = ('visible' === document.visibilityState); |
| 783 | if(cs.pageIsActive){ |
| 784 | cs.e.pageTitle.innerText = cs.pageTitleOrig; |
| 785 | console.debug("Taking back input focus."); |
| 786 | cs.inputFocus( |
| 787 | /* An attempt to resolve usability problem reported by Joe |
| 788 | M. where the Pale Moon browser is giving input focus to |
| 789 | the Preview button. The down-side of this is that it will |
| 790 | deselect any text which was previously selected on this |
| 791 | page. This also, unfortunately, places the focus at the |
| 792 | start of the element, rather than the last cursor position |
| 793 | (like a textarea would). */); |
| 794 | } |
| 795 | }, true); |
| 796 | cs.setCurrentView(cs.e.viewMessages); |
| 797 | |
| 798 | cs.e.activeUserList.addEventListener('click', function f(ev){ |
| 799 |
+3
| --- src/style.chat.css | ||
| +++ src/style.chat.css | ||
| @@ -191,10 +191,13 @@ | ||
| 191 | 191 | overflow: auto; |
| 192 | 192 | resize: vertical; |
| 193 | 193 | white-space: pre-wrap; |
| 194 | 194 | /* ^^^ Firefox, when pasting plain text into a contenteditable field, |
| 195 | 195 | loses all newlines unless we explicitly set this. Chrome does not. */ |
| 196 | + cursor: text; | |
| 197 | + /* ^^^ In some browsers the cursor may not change for a contenteditable | |
| 198 | + element until it has focus, causing potential confusion. */ | |
| 196 | 199 | } |
| 197 | 200 | #chat-input-field:empty::before { |
| 198 | 201 | content: attr(data-placeholder); |
| 199 | 202 | opacity: 0.6; |
| 200 | 203 | } |
| 201 | 204 |
| --- src/style.chat.css | |
| +++ src/style.chat.css | |
| @@ -191,10 +191,13 @@ | |
| 191 | overflow: auto; |
| 192 | resize: vertical; |
| 193 | white-space: pre-wrap; |
| 194 | /* ^^^ Firefox, when pasting plain text into a contenteditable field, |
| 195 | loses all newlines unless we explicitly set this. Chrome does not. */ |
| 196 | } |
| 197 | #chat-input-field:empty::before { |
| 198 | content: attr(data-placeholder); |
| 199 | opacity: 0.6; |
| 200 | } |
| 201 |
| --- src/style.chat.css | |
| +++ src/style.chat.css | |
| @@ -191,10 +191,13 @@ | |
| 191 | overflow: auto; |
| 192 | resize: vertical; |
| 193 | white-space: pre-wrap; |
| 194 | /* ^^^ Firefox, when pasting plain text into a contenteditable field, |
| 195 | loses all newlines unless we explicitly set this. Chrome does not. */ |
| 196 | cursor: text; |
| 197 | /* ^^^ In some browsers the cursor may not change for a contenteditable |
| 198 | element until it has focus, causing potential confusion. */ |
| 199 | } |
| 200 | #chat-input-field:empty::before { |
| 201 | content: attr(data-placeholder); |
| 202 | opacity: 0.6; |
| 203 | } |
| 204 |