Skip to content

Add ShowToolbar property to ContentLayout#24582

Merged
yagmurcelk merged 2 commits intorel-10.0from
Home-Breadcrumb
Jan 13, 2026
Merged

Add ShowToolbar property to ContentLayout#24582
yagmurcelk merged 2 commits intorel-10.0from
Home-Breadcrumb

Conversation

@maliming
Copy link
Member

@maliming maliming commented Jan 9, 2026

No description provided.

Copilot AI review requested due to automatic review settings January 9, 2026 01:00
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds a ShowToolbar property to the ContentLayout class and refactors the ShouldShowBreadCrumb() method logic.

  • Adds a new ShowToolbar boolean property with a default value of true
  • Simplifies the ShouldShowBreadCrumb() method by removing the Title validation logic
  • Changes breadcrumb display conditions to check ShowCurrent or ShowHome flags

@maliming maliming requested a review from yagmurcelk January 12, 2026 05:52
@maliming maliming added this to the 10.0-patch-final milestone Jan 12, 2026
@yagmurcelk yagmurcelk merged commit e33deda into rel-10.0 Jan 13, 2026
2 of 3 checks passed
@yagmurcelk yagmurcelk deleted the Home-Breadcrumb branch January 13, 2026 10:33
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.

3 participants