Skip to content

Commit

Permalink
Remove unnecessary clipboard clear button
Browse files Browse the repository at this point in the history
This button fills no real purpose. It's easy to mark everything and
delete with either "Ctrl + A -> Delete" or, on touch devices, "long
press -> mark everything -> Delete".
  • Loading branch information
samhed committed Sep 9, 2022
1 parent edc7520 commit 69e0f0f
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 10 deletions.
7 changes: 0 additions & 7 deletions app/ui.js
Original file line number Diff line number Diff line change
Expand Up @@ -337,8 +337,6 @@ const UI = {
.addEventListener('click', UI.toggleClipboardPanel);
document.getElementById("noVNC_clipboard_text")
.addEventListener('change', UI.clipboardSend);
document.getElementById("noVNC_clipboard_clear_button")
.addEventListener('click', UI.clipboardClear);
},

// Add a call to save settings when the element changes,
Expand Down Expand Up @@ -968,11 +966,6 @@ const UI = {
Log.Debug("<< UI.clipboardReceive");
},

clipboardClear() {
document.getElementById('noVNC_clipboard_text').value = "";
UI.rfb.clipboardPasteFrom("");
},

clipboardSend() {
const text = document.getElementById('noVNC_clipboard_text').value;
Log.Debug(">> UI.clipboardSend: " + text.substr(0, 40) + "...");
Expand Down
3 changes: 0 additions & 3 deletions vnc.html
Original file line number Diff line number Diff line change
Expand Up @@ -142,9 +142,6 @@ <h1 class="noVNC_logo" translate="no"><span>no</span><br>VNC</h1>
<img alt="" src="app/images/clipboard.svg"> Clipboard
</div>
<textarea id="noVNC_clipboard_text" rows=5></textarea>
<br>
<input id="noVNC_clipboard_clear_button" type="button"
value="Clear" class="noVNC_submit">
</div>
</div>

Expand Down

0 comments on commit 69e0f0f

Please sign in to comment.