-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unable to test which cursor is used #7750
Comments
Right. Currently all cursor tests are manual, and that's lousy. |
FWIW, years ago I added a mechanism in chromium to read back the cursor state (including custom bitmaps). I think that's probably more practical than doing screenshot diffing - at least for the OS defined cursors. |
@RByers There are tests for things like colour space and size of image cursors, which means you need more than just state. |
It seems like in order to automate this, you'd need to take a screenshot of the whole desktop environment. A handful of manual tests for this might be an OK tradeoff :) |
Indeed. Image cursors would be quite a pain to test, but even worse are the various predefined cursors, that don't even have a set shape, and need human judgement to decide if the cursor is indeed a cursor that "Indicates something is to be copied. Often rendered as an arrow with a small plus sign next to it." If someone has a bright idea, that'd be great, but otherwise, we can certainly focus our energy better somewhere else. |
At the moment, we have no way to test the CSS
cursor
property. We probably need some ability to take a screenshot of the cursor at a specific place.cc/ @frivoal
The text was updated successfully, but these errors were encountered: