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

Tree (cursor got changed when hovering the tree item) #9146

Closed
1 task done
jamesPodo opened this issue Jun 6, 2024 · 2 comments · Fixed by #9152
Closed
1 task done

Tree (cursor got changed when hovering the tree item) #9146

jamesPodo opened this issue Jun 6, 2024 · 2 comments · Fixed by #9152
Assignees
Labels
bug This issue is a bug in the code Low Prio TOPIC P
Milestone

Comments

@jamesPodo
Copy link

Describe the bug

We've noticed that the cursor is changed to pointer when hovering the tree item after updating @ui5wc 1.23.3
@ui5wc-react 1.26.2 . The cursor was not changed on Tree with mode None previously. The issue can also be reproduced from https://sap.github.io/ui5-webcomponents-react/?path=/docs/data-display-tree--docs

Isolated Example

No response

Reproduction steps

  1. Enter the above link on the browser
  2. Try hovering the tree item
  3. Notice the cursor is changed to pointer
  4. change the mode from the properties tab (e.g. SingleSelection)
  5. change back to None
  6. Notice the issue is fix ( cursor is not changed on hovering)

Expected Behaviour

The cursor should not change when hovering tree item on the Tree with mode None.

Screenshots or Videos

No response

UI5 Web Components for React Version

1.26.2

UI5 Web Components Version

1.23.3

Browser

Chrome

Operating System

Mac

Additional Context

No response

Relevant log output

No response

Organization

SAP SE

Declaration

  • I’m not disclosing any internal or sensitive information.
@jamesPodo jamesPodo added the bug This issue is a bug in the code label Jun 6, 2024
@Lukas742
Copy link
Collaborator

Lukas742 commented Jun 7, 2024

Hi @jamesPodo

Thanks for reporting! I'll forward this issue to our UI5 Web Components Colleagues as the affected component is developed in their repository.


Hi colleagues,
here you can find an example describing the issue using your playground sandbox.

@Lukas742 Lukas742 transferred this issue from SAP/ui5-webcomponents-react Jun 7, 2024
@niyap
Copy link
Contributor

niyap commented Jun 7, 2024

Hello @SAP/ui5-webcomponents-topic-p,

Could you please analyse the issue? It can be easily reproduced in the sample provided by Lukas: sample

Thank you in advance!

Kind Regards,
Niya

@niyap niyap added the TOPIC P label Jun 7, 2024
@niyap niyap assigned niyap and unassigned niyap Jun 7, 2024
@niyap niyap added the Low Prio label Jun 7, 2024
@niyap niyap added this to Issues in Maintenance - Topic P via automation Jun 7, 2024
@kgogov kgogov self-assigned this Jun 7, 2024
kgogov added a commit that referenced this issue Jun 7, 2024
Added CSS rule to set cursor to default when `_mode` attribute is not present

Fixes: #9146
@kgogov kgogov moved this from Issues to In Progress in Maintenance - Topic P Jun 7, 2024
kgogov added a commit that referenced this issue Jun 7, 2024
Added CSS rule to set cursor to default when `_mode` attribute is not present

Fixes: #9146
kgogov added a commit that referenced this issue Jun 13, 2024
Added CSS rule to set cursor to default when `_mode` attribute is not present

Fixes: #9146
kgogov added a commit that referenced this issue Jun 13, 2024
Added CSS rule to set cursor to default when `_mode` attribute is not present

Fixes: #9146
kgogov added a commit that referenced this issue Jun 13, 2024
Added CSS rule to set cursor to default when `_mode` attribute is not present

Fixes: #9146
Maintenance - Topic P automation moved this from In Progress to Completed Jun 13, 2024
kgogov added a commit that referenced this issue Jun 13, 2024
Added CSS rule to set cursor to default when `_mode` attribute is not present

Fixes: #9146
ilhan007 pushed a commit that referenced this issue Jun 14, 2024
Added CSS rule to set cursor to default when `_mode` attribute is not present

Fixes: #9146
kgogov added a commit that referenced this issue Jun 17, 2024
Added CSS rule to set cursor to default when `_mode` attribute is not present

Fixes: #9146
@ilhan007 ilhan007 added this to the 1.24.5 milestone Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug in the code Low Prio TOPIC P
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

5 participants