Skip to content

[FlexibleColumnLayout | ui5/webcomponents-react | v2.10.0]: Incorrect class and attribute for first column #11545

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

Closed
1 task done
KonradPietocha opened this issue May 19, 2025 · 8 comments · Fixed by #11565
Assignees
Labels
bug This issue is a bug in the code Medium Prio TOPIC P

Comments

@KonradPietocha
Copy link

Bug Description

Describe the bug
When we quickly change (during tests) the layout from "TwoColumnsMidExpanded" to "MidColumnFullScreen" and so on, at some point the first column stays forever with the class "ui5-fcl-column--hidden" and remains invisible even in the case of "TwoColumnsMidExpanded" and "OneColumn" (empty).

Affected Component

FlexibleComponent

Expected Behaviour

Fist column should be visible in case "TwoColumnsMidExpanded" and "OneColumn".

Isolated Example

No response

Steps to Reproduce

  1. Quickly change the layout from "TwoColumnsMidExpanded" to "MidColumnFullScreen" and so on.

Log Output, Stack Trace or Screenshots

Image Image

Priority

None

UI5 Web Components Version

2.10.0

Browser

Chrome

Operating System

No response

Additional Context

No response

Organization

SAP

Declaration

  • I’m not disclosing any internal or sensitive information.
@KonradPietocha KonradPietocha added the bug This issue is a bug in the code label May 19, 2025
@LidiyaGeorgieva
Copy link
Contributor

LidiyaGeorgieva commented May 19, 2025

Hello @KonradPietocha please provide a link for testing

@LidiyaGeorgieva
Copy link
Contributor

Hello @Lukas742 could you take over?

@LidiyaGeorgieva LidiyaGeorgieva self-assigned this May 19, 2025
@LidiyaGeorgieva LidiyaGeorgieva removed their assignment May 19, 2025
@KonradPietocha
Copy link
Author

Hello @KonradPietocha please provide a link for testing

We wanted to update kyma/busola to ui5 v2.10, but my PR has this exact bug, so for now we can't upgrade to this version and the PR is blocked.
Link to PR: kyma-project/busola#3909

@LidiyaGeorgieva
Copy link
Contributor

@KonradPietocha we need an isolated sample of the issue. You can use this stackblitz sample, created by us. You can choose the ui5-webcomponents version in the "dependency panel",
to import the required components in the "index.js", use them in the" index.html" and observe the result.

@Lukas742
Copy link
Collaborator

Lukas742 commented May 19, 2025

Hi @LidiyaGeorgieva

the behavior is reproducible in your docs example, so this is not an issue with our wrapper but the component.

Steps to reproduce the issue:

  • Go to the example I extended
  • Toggle the layout quickly by pressing the "Toggle Layout" button
  • See the described issue
2025-05-19_15-23-42.mp4

@KonradPietocha
Copy link
Author

KonradPietocha commented May 19, 2025

Hi @LidiyaGeorgieva

the behavior is reproducible in your docs example, so this is not an issue with our wrapper but the component.

Steps to reproduce the issue:

  • Go to the example I extended
  • Toggle the layout quickly by pressing the "Toggle Layout" button
  • See the described issue

2025-05-19_15-23-42.mp4

Yes. @LidiyaGeorgieva Here you can also reproduce my case in the middle column by quickly clicking the "Enter full screen" button

@LidiyaGeorgieva
Copy link
Contributor

Hello @SAP/ui5-webcomponents-topic-p please have a look at this issue.

Steps to reproduce:

  1. Go to the extended example
  2. Toggle the layout quickly by pressing the "Toggle Layout" button
  3. See that at some point the right column receives the class "ui5-fcl-column--hidden" even it should be visible

Image

Best Regards,
Lidiya

Copy link

This issue has been closed. To reopen, just leave a comment!

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 Medium Prio TOPIC P
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

4 participants