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

[ui5-resposive-popover]: the responsive popover is not centered in rtl direction and existing scroll bar #9966

Open
1 task done
iva-sap opened this issue Oct 1, 2024 · 2 comments · May be fixed by #10008
Open
1 task done
Assignees
Labels
bug This issue is a bug in the code Low Prio TOPIC RD

Comments

@iva-sap
Copy link

iva-sap commented Oct 1, 2024

Bug Description

ui5-resposive-popover is not centered in cases there is a scroll bar that changes its position on direction - RTL. The issue is reproducible on Mac if the appearance of the scroll bar is set to be always shown.

Affected Component

ui5-resposive-popover

Expected Behaviour

ui5-resposive-popover should be centered.

Isolated Example

Playground example

Steps to Reproduce

  1. Open the isolated example
  2. Change the dir to rtl of the body element of the "Project preview" iframe in dev tools
  3. Click on btn1

Log Output, Stack Trace or Screenshots

Screenshot 2024-10-01 at 17 11 49

If the overflow is set to hidden in the body element then the responsive popover is positioned properly.

Screenshot 2024-10-01 at 17 12 14

Priority

Low

UI5 Web Components Version

2.2.0

Browser

Chrome

Operating System

Windows 11, macOS sonoma 14.6.1

Additional Context

The issue is relevant for ui5-popover.

Organization

No response

Declaration

  • I’m not disclosing any internal or sensitive information.
@iva-sap iva-sap added the bug This issue is a bug in the code label Oct 1, 2024
@LidiyaGeorgieva
Copy link
Contributor

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

The issue is reproducible also for Windows.

Best Regards,
Lidiya

@PlutaKatarzyna
Copy link

Hi UI5 Team, just a quick notice that it's issue related with UDEx components. We are using ui5 components version 1.24.8 so we appreciate if fix could be down-ported.
Many Thanks!
Kate

@LidiyaGeorgieva LidiyaGeorgieva self-assigned this Oct 7, 2024
LidiyaGeorgieva added a commit that referenced this issue Oct 10, 2024
When the popover is inside iFrame in RTL direction and
 there is a scrollbar (in left) - the position of the popover
(and its arrow) should be adjusted (moved to left
 with the size of the scrollbar).

fixes: #9966
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 RD
Projects
Status: New Issues
Development

Successfully merging a pull request may close this issue.

3 participants