Skip to content

feat(ui5): Add Smart & MDC Controls best practices skill#91

Merged
flovogt merged 5 commits into
UI5:mainfrom
miloshevb:smart-mdc-controls
Jul 3, 2026
Merged

feat(ui5): Add Smart & MDC Controls best practices skill#91
flovogt merged 5 commits into
UI5:mainfrom
miloshevb:smart-mdc-controls

Conversation

@miloshevb

@miloshevb miloshevb commented Jul 2, 2026

Copy link
Copy Markdown
Member
  • Add ui5-best-practices-smart-controls skill covering sap.ui.comp annotation-driven controls for OData V2
    (SmartField, SmartForm, SmartFilterBar, SmartChart, SmartLink, SmartMultiInput, FilterBar, ValueHelpDialog)
  • Add ui5-best-practices-mdc skill covering sap.ui.mdc delegate-based controls for OData V4 and JSON models
    (FilterBar, Chart, Field, FilterField, ValueHelp, Link, MultiValueField)
  • Update README.md with descriptions for both new skills

Both skills follow the established pattern from ui5-best-practices-tables:

  • SKILL.md with trigger keywords, reference-loading table, core rules, selection matrix, common errors, and
    performance/accessibility guidance
  • Individual references/*.md files per control (~60-120 lines each) with API links, key properties/methods, XML/JS
    examples, and troubleshooting

JIRA: BGSOFUISAKAR-1630

@cla-assistant

cla-assistant Bot commented Jul 2, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@flovogt flovogt self-requested a review July 2, 2026 12:56
Comment thread plugins/ui5/skills/ui5-best-practices-mdc/references/mdc-chart.md Outdated
Comment thread plugins/ui5/README.md Outdated
Comment thread plugins/ui5/README.md Outdated
@flovogt flovogt changed the title feat(UI5): Add Smart & MDC Controls best practices skill feat(ui5): Add Smart & MDC Controls best practices skill Jul 3, 2026
@flovogt flovogt merged commit c61e0b7 into UI5:main Jul 3, 2026
4 checks passed
@openui5bot openui5bot mentioned this pull request Jul 3, 2026
@KaloyanTsv1

Copy link
Copy Markdown

LGTM

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