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-dynamic-page): move subheading slot outside the title wrapper #10163

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

NakataCode
Copy link
Contributor

@NakataCode NakataCode commented Nov 8, 2024

Issue:

  • The subheading width was not 100% and was constrained by the title wrapper’s flex layout.

Solution:

  • Moved the subheading slot outside the title wrapper to allow it to take full width.

Additional change:
Aligned the avatar with the title heading and subheading.

Fixes: #10161

@kgogov kgogov self-requested a review November 8, 2024 14:24
@NakataCode NakataCode force-pushed the dynamic-page-subheading-position branch from 536f06e to aaabbe8 Compare November 8, 2024 15:05
@NakataCode NakataCode force-pushed the dynamic-page-subheading-position branch from aaabbe8 to 1d2eff2 Compare November 8, 2024 15:18
@NakataCode NakataCode marked this pull request as ready for review November 8, 2024 20:30
@NakataCode NakataCode changed the title fix(ui5-dynamic-page): move subheading slot outside title wrapper for… fix(ui5-dynamic-page): move subheading slot outside the title wrapper Nov 10, 2024
packages/fiori/src/DynamicPageTitle.hbs Outdated Show resolved Hide resolved
@NakataCode NakataCode force-pushed the dynamic-page-subheading-position branch from 1d2eff2 to cee3b29 Compare November 12, 2024 14:22
@NakataCode NakataCode merged commit 6466b8a into main Nov 13, 2024
10 checks passed
@NakataCode NakataCode deleted the dynamic-page-subheading-position branch November 13, 2024 07:31
@ui5-webcomponents-bot
Copy link
Collaborator

🎉 This PR is included in version v2.5.0-rc.0 🎉

The release is available on v2.5.0-rc.0

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DynamicPageTitle]: subheading now only spreads to half of the titleArea width
3 participants