Skip to content

Conversation

@bengisusagir
Copy link
Contributor

@bengisusagir bengisusagir commented Nov 27, 2025

💡 What is the current behavior?

  • The ApplicationHeader component is missing the following properties: NameSuffix, CompanyLogo, CompanyLogoAlt, AppIcon, AppIconAlt, and HideBottomBorder.
  • The ApplicationHeader component also does not support a secondary slot.

🆕 What is the new behavior?

  • Added the following properties to the ApplicationHeader component: NameSuffix, CompanyLogo, CompanyLogoAlt, AppIcon, AppIconAlt, and HideBottomBorder.

  • Added support for a secondary slot in the ApplicationHeader component.

🏁 Checklist

A pull request can only be merged if all of these conditions are met (where applicable):

  • 🦮 Accessibility (a11y) features were implemented
  • 🗺️ Internationalization (i18n) - no hard coded strings
  • 📲 Responsiveness - components handle viewport changes and content overflow gracefully
  • 📄 Documentation was reviewed/updated
  • [x ] 🧪 Unit tests were added/updated and pass (dotnet test)
  • [x ] 🏗️ Successful compilation (dotnet build, changes pushed)

👨‍💻 Help & support

@emncnozge emncnozge merged commit da7c0e1 into siemens:master Nov 27, 2025
1 check passed
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.

2 participants