Skip to content

feat(dropdown): added KbqDropdownFooter directive (#DS-3699)#1780

Open
artembelik wants to merge 5 commits into
mainfrom
feat/DS-3699
Open

feat(dropdown): added KbqDropdownFooter directive (#DS-3699)#1780
artembelik wants to merge 5 commits into
mainfrom
feat/DS-3699

Conversation

@artembelik

Copy link
Copy Markdown
Contributor

No description provided.

@artembelik artembelik self-assigned this Jul 24, 2026
@artembelik
artembelik requested a review from lskramarov as a code owner July 24, 2026 11:51
Copilot AI review requested due to automatic review settings July 24, 2026 11:51
@github-actions

Copy link
Copy Markdown

🚨 E2E tests failed

Review the report for details.


💡 Comment /approve-snapshots to approve snapshot changes.

@github-actions

github-actions Bot commented Jul 24, 2026

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit e199274):

https://koobiq-next--prs-1780-557k62z8.web.app

(expires Mon, 27 Jul 2026 13:08:50 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c9e37e518febda70d0317d07e8ceb35ac43c534c

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new KbqDropdownFooter directive to the dropdown component so consumers can project a dedicated footer region under the options, aligning dropdown capabilities with other panel-like components (e.g. autocomplete) and documenting the new API via examples and docs updates.

Changes:

  • Introduces KbqDropdownFooter directive and wires up footer content projection in KbqDropdown.
  • Updates dropdown styling to separate content padding from the panel and styles the footer using the shared %kbq-select-footer pattern.
  • Adds docs/dev/examples and e2e demo markup to showcase the new footer.

Reviewed changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tools/public_api_guard/components/dropdown.api.md Public API snapshot updated to include KbqDropdownFooter and updated content selectors.
packages/docs-examples/components/dropdown/index.ts Registers and exports the new “with footer” example.
packages/docs-examples/components/dropdown/dropdown-with-footer/dropdown-with-footer-example.ts New example demonstrating footer usage (including links and multiline footer content).
packages/components/dropdown/e2e.ts Updates e2e demo templates to include dropdown footers.
packages/components/dropdown/dropdown.scss Moves padding to .kbq-dropdown__content and adds .kbq-dropdown-footer styling.
packages/components/dropdown/dropdown.ru.md Documents the new footer section (RU).
packages/components/dropdown/dropdown.module.ts Exports KbqDropdownFooter from the module.
packages/components/dropdown/dropdown.html Adds a dedicated <ng-content> slot for the footer.
packages/components/dropdown/dropdown.en.md Documents the new footer section (EN).
packages/components/dropdown/dropdown.component.ts Adds the KbqDropdownFooter directive definition.
packages/components-dev/dropdown/module.ts Includes the new footer example in the dev app module/template.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +61 to +66
/** Footer that is rendered below the dropdown options panel. */
@Directive({
selector: '[kbqDropdownFooter], kbq-dropdown-footer',
host: { class: 'kbq-dropdown-footer' }
})
export class KbqDropdownFooter {}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@rmnturov нам такое нужно?

Comment thread packages/components/dropdown/dropdown.component.ts
@artembelik

Copy link
Copy Markdown
Contributor Author

/approve-snapshots

@github-actions

Copy link
Copy Markdown

🔄 Updating snapshots.

@artembelik
artembelik requested a review from rmnturov July 24, 2026 12:02
@github-actions

Copy link
Copy Markdown

✅ Snapshots updated!

@github-actions

Copy link
Copy Markdown

🚨 E2E tests failed

Review the report for details.


💡 Comment /approve-snapshots to approve snapshot changes.

@github-actions

Copy link
Copy Markdown

🚨 E2E tests failed

Review the report for details.


💡 Comment /approve-snapshots to approve snapshot changes.

@artembelik

Copy link
Copy Markdown
Contributor Author

/approve-snapshots

@github-actions

Copy link
Copy Markdown

🔄 Updating snapshots.

@github-actions

Copy link
Copy Markdown

✅ Snapshots updated!

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