Skip to content

Conversation

TeodorTaushanov
Copy link
Contributor

fixes #10894

@ui5-webcomponents-bot
Copy link
Collaborator

ui5-webcomponents-bot commented Oct 9, 2025

🧹 Preview deployment cleaned up: https://pr-12458--ui5-webcomponents.netlify.app

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview October 9, 2025 13:02 Inactive
Copy link
Contributor

@Copilot 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 PR adds a new noDataHeaderLevel property to the UploadCollection component, allowing developers to customize the header level of the "No data" text from the default "H2" to any valid title level.

  • Adds noDataHeaderLevel property with TypeScript typing and JSDoc documentation
  • Updates the template to use the configurable header level instead of hard-coded "H2"
  • Includes comprehensive test coverage for the new functionality

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
packages/fiori/src/UploadCollection.ts Adds the noDataHeaderLevel property with proper typing and documentation
packages/fiori/src/UploadCollectionTemplate.tsx Updates template to use the configurable header level property
packages/fiori/test/pages/UploadCollection.html Adds test usage of the new property with "H3" value
packages/fiori/cypress/specs/UploadCollection.cy.tsx Adds comprehensive test coverage for the new property functionality

@TeodorTaushanov TeodorTaushanov requested review from a team and Stoev October 9, 2025 13:28
Copy link

@Stoev Stoev left a comment

Choose a reason for hiding this comment

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

Looks good. No grammatical or syntax errors.

@TeodorTaushanov TeodorTaushanov merged commit aa61398 into main Oct 10, 2025
20 of 22 checks passed
@TeodorTaushanov TeodorTaushanov deleted the upload_collection_heading branch October 10, 2025 07:48
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview October 10, 2025 07:48 Inactive
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.

[UploadCollection] headline level of the noDataText should be customizable

4 participants