Skip to content
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

Adds experimental PointerIcon.fromKeyword function to change the browser cursor in Web source set #1704

Merged
merged 2 commits into from
Dec 4, 2024

Conversation

ApoloApps
Copy link

@ApoloApps ApoloApps commented Dec 3, 2024

Adds remaining cursors pointer icons in Web source set which are available in https://developer.mozilla.org/en-US/docs/Web/CSS/cursor
Note that not all of the cursors are available in all browsers and OS's but they default to default one if not available
If you were planning a different approach, i'm open to feedback

Fixes https://youtrack.jetbrains.com/issue/CMP-6951/Consider-making-BrowserCursor-dataclass-public-or-add-more-PointerIcons-to-WasmJs-target

Release Notes

Features - Web

  • Adds experimental PointerIcon.fromKeyword function to change the browser cursor

@ApoloApps ApoloApps changed the title Adds remaining Web cursors based on https://developer.mozilla.org/en-US/docs/Web/CSS/cursor Adds remaining cursors to web source set Dec 3, 2024
@MatkovIvan MatkovIvan requested review from eymar and Schahen December 3, 2024 10:31
…omKeyword as an ExperimentalComposeUiApi

Signed-off-by: Apolo Studio <dev@composevisualeditor.com>
@ApoloApps ApoloApps changed the title Adds remaining cursors to web source set Adds experimental PointerIcon.fromKeyword function to change the browser cursor in Web source set Dec 4, 2024
@eymar eymar merged commit 20ed662 into JetBrains:jb-main Dec 4, 2024
1 check passed
@ApoloApps ApoloApps deleted the add_remaining_web_cursors branch December 4, 2024 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants