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

fix(ui5-page): fix double footer and adjust animation timing #9885

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MapTo0
Copy link
Member

@MapTo0 MapTo0 commented Sep 17, 2024

FIXES: #9755
FIXES: #9756

@MapTo0
Copy link
Member Author

MapTo0 commented Sep 17, 2024

Hello @SAP/ui5-webcomponents-topic-b

Could you please look at this request.
We've made some changes in the ui5-bar so we can set relative (not fixed) width of the host element.

Basically, the problem is that the width gets in inherited by the internal element which breaks when having relative values.

Check: link

@nikoletavnv nikoletavnv self-requested a review September 17, 2024 12:55
@MapTo0
Copy link
Member Author

MapTo0 commented Sep 17, 2024

@tsanislavgatev do you think we should remove the double box-shadow as well?

@nikoletavnv
Copy link
Member

Dear @SAP/ui5-webcomponents-topic-b,
There is a double box-shadow style that comes from the ui5-bar-root element. Please take a look at the screenshot below:

image

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.

[ui5-page]: double footer wrapper [ui5-page]: still has animation when animationMode config is set to none
3 participants