Skip to content
This repository has been archived by the owner on Sep 1, 2023. It is now read-only.
/ hterm Public archive

Commit

Permalink
hterm: tweak custom mouse support
Browse files Browse the repository at this point in the history
After playing around with this behavior a bit more, it turns out that
it is not easy to handle the fallback keyword for the user.  The CSS
syntax allows for a single fallback keyword, but only when url() is
used.  That means if the user went with a keyword instead (e.g.
'crosshair'), or added their own fallback, we'd set the cursor to a
value like 'crosshair, auto' which is invalid and Chrome would ignore
it completely.

Update the FAQ to note the fallback requirement accordingly.

Change-Id: Ibd84eb0456f1c4c06e652808e2f445b83a01225c
Reviewed-on: https://chromium-review.googlesource.com/1112565
Reviewed-by: Vitaliy Shipitsyn <[email protected]>
Tested-by: Mike Frysinger <[email protected]>
  • Loading branch information
vapier committed Jun 22, 2018
1 parent 4a264a5 commit 9ebc18a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion js/hterm_scrollport.js
Original file line number Diff line number Diff line change
Expand Up @@ -307,7 +307,7 @@ hterm.ScrollPort.prototype.decorate = function(div) {
'height: 100%;' +
'width: 100%;' +
'overflow: hidden;' +
'cursor: var(--hterm-mouse-cursor-style), auto;' +
'cursor: var(--hterm-mouse-cursor-style);' +
'-webkit-user-select: none;' +
'-moz-user-select: none;');

Expand Down

0 comments on commit 9ebc18a

Please sign in to comment.