Merge branch 'master' into mkirova/fix-border-calc

Author: dkamburov

.github/AGENTS-README.md

@@ -88,38 +88,15 @@ Accessibility is an integral part of any UI component. We as a team are committe
 2. `status: localized` this status is for issues that were with a pending translation status and have already been localized. Place this status label once these translation changes have been included in the current pull request, or the changes are already pulled with a different pull request.
 
 ## Localization (i18n) - applicable to components' string resources
-There are several ways to localize components' string resources:
+There are several ways to localize components' string resources. For more information on how it works, refer to our [Documentation - Localization (i18n)](https://www.infragistics.com/products/ignite-ui-angular/angular/components/general/localization#localization-i18n-1).
 
-1. Using custom resource strings:
-    1.1. Localize a given instance of component - each component which supports localization has input property `resourceStrings`. Setting a newly instantiated object to this property will localize only that given component's instance.
-    1.2. Localize all resources for a component type - each component which supports localization has input property `resourceStrings`. To localize all instances of a given component in the application the following steps should be performed - get the value of the input property `resourceStrings` of the component to be localized; do not create a new instance but replace the existing strings within the object. By default all components of a given type in an application share one instance of the resource strings. Replacing a value in that instance affects all components of that type in the application.
-    1.3. Localize all resources for all components - use global method `getCurrentResourceStrings` to get an object containing current resource strings for all components. To provide localized resources just pass an object of type `IResourceStrings` to the global method `changei18n`.
-    1.4 As of 21.1.x the localization has new implementation and you can use the new API `registerI18n` to register resource string for a component or all components for the whole app, as well as which locale it corresponds to. To localize a single component you will need to get is corresponding resource string keys using one of the available resources and provide only those keys.
+**NOTE** As of 21.1.x, the localization resource strings have been moved to the [`igniteui-i18n`](https://github.com/IgniteUI/igniteui-i18n) repository under `projects/igniteui-i18n-resources`. The package `igniteui-angular-i18n` under `./projects/igniteui-angular-i18n` remains as a source of the localization resources for Ignite UI for Angular, that derives the resources from [`igniteui-i18n`](https://github.com/IgniteUI/igniteui-i18n).
 
-2. Using npm package:
-We've created new repository which will hold the resource strings for languages different than English:
-https://github.com/IgniteUI/igniteui-angular-i18n 
-
-**NOTE** The localization repo has been moved to live inside the `igniteui-angular` repository under `./projects/igniteui-angular-i18n`
-**NOTE** As of 21.1.x the localization resource strings have been moved to the [`igniteui-i18n`](https://github.com/IgniteUI/igniteui-i18n) repository under `projects/igniteui-i18n-resources`.
+**NOTE (21.0.x and lower)** The localization resource strings live inside the `igniteui-angular` repository under `./projects/igniteui-angular-i18n`.
 
 A npm package should be published each time we release new version of Ignite UI for Angular. Its version should correspond to the version of the igniteui-angular npm package.
 One could localize an application by importing the corresponding localized resource strings from the localization package (`igniteui-angular-i18n`) and use the methods described in the previous bullet to localize the whole application or part of it.
 
-**Example:**
-
-Inside app.module you can perform:
-```ts
-import { IgxResouceStringsJA } from ‘igniteui-angular-i18n’;
-changei18n(IgxResouceStringsJA);
-```
-
-**Example new API:**
-```ts
-import { ResouceStringsJA } from ‘igniteui-i18n-resources’;
-registerI18n(IgxResouceStringsJA, 'ja');
-```
-
 ### Resource strings keys naming convention
 Each key in the `IResourceStrings` (and `IGridResourceStrings`, `ITimePickerResourceStrings`, etc.) is prefixed with components' selector and followed by the resource string key. Having components' selectors as prefixes allows us to have same resource strings keys for more than one component.
 Example: _igx_grid_groupByArea_message_.

.github/CONTRIBUTING.md

@@ -0,0 +1,173 @@
+---
+name: bug-fixing-implementer-agent
+description: Implements the minimum fix (GREEN phase) for bugs in igniteui-angular. Preserves the public API, accessibility, and localization. Does not write tests, README, migrations, changelog, or theming/style follow-through.
+tools:
+  - search/codebase
+  - edit/editFiles
+  - edit/createFile
+  - read/problems
+  - execute/runTests
+  - read/terminalLastCommand
+  - web
+---
+
+# Bug Fix Implementer — GREEN Phase
+
+You write **production code** for Ignite UI for Angular to fix bugs and make failing reproduction tests pass.
+
+You operate in one of two modes depending on context.
+
+Treat failing tests as guidance, not as the full specification.
+
+---
+
+## How You Work
+
+You receive either a **Bug Knowledge** block (from the orchestrator or user) or a raw bug report.
+
+### Bug Knowledge Block
+
+When provided, a Bug Knowledge block contains pre-investigated findings:
+
+- **Bug report**: summary of expected vs. actual behavior
+- **Root cause**: identified root cause
+- **Affected files**: source file paths relevant to the fix
+- **Failing test**: path and description of the reproduction test
+- **Impact notes**: flags (breaking change, i18n, accessibility, etc.)
+
+### Mode 1 — Orchestrated (Bug Knowledge provided)
+
+Skip investigation. The orchestrator has already done it.
+
+1. **Read the Bug Knowledge block** — understand the root cause, affected files, and scope.
+2. **Read the affected source files** — confirm the root cause and understand the code you will change.
+3. **Read the failing test** — understand what behavior it reproduces.
+4. **Implement the fix** — write the minimum code to make the failing test pass without breaking existing behavior.
+5. **Run all tests** — the reproduction test and all existing tests must pass.
+6. **Run lint** — `npm run lint:lib` must pass.
+
+### Mode 2 — Standalone (no Bug Knowledge provided)
+
+Do your own investigation.
+
+1. **Read the original bug report** — understand expected vs. actual behavior.
+2. **Read the existing component source** — understand the current implementation, patterns, and conventions.
+3. **Read the failing test** (if one exists) — understand what behavior it is trying to reproduce.
+4. **Identify the root cause** — trace the code path that triggers the bug.
+5. **Implement the fix** — write the minimum code to make the failing test pass without breaking existing behavior.
+6. **Run all tests** — the reproduction test and all existing tests must pass.
+7. **Run lint** — `npm run lint:lib` must pass.
+
+---
+
+## Component-Specific Patterns
+
+Check the relevant skill file for component APIs and patterns:
+- Non-grid components → `skills/igniteui-angular-components/SKILL.md`
+- Grid components → `skills/igniteui-angular-grids/SKILL.md`
+- Theming & styling → `skills/igniteui-angular-theming/SKILL.md`
+
+---
+
+## GREEN Phase — Fix the Bug
+
+1. Write the **minimum code** to make the failing reproduction test pass.
+2. Follow the conventions from `.github/copilot-instructions.md`.
+3. Follow existing coding patterns: signals, standalone components, `ChangeDetectionStrategy.OnPush`.
+4. Do not change the public API unless the fix requires it.
+5. Place source changes in `projects/igniteui-angular/<component>/src/`.
+6. If public exports must change, update `projects/igniteui-angular/<component>/index.ts`.
+7. Run tests — **all new and existing tests must pass**.
+8. Run `npm run lint:lib` — must pass.
+
+---
+
+## Accessibility
+
+Every fix must preserve or improve accessibility compliance:
+- **Section 508**, **WCAG** (AA minimum, AAA where achievable), and **WAI-ARIA** standards apply.
+- All interactive elements must be fully keyboard navigable.
+- Do not remove existing `role`, `aria-*`, or `tabindex` attributes unless the fix explicitly corrects an accessibility issue.
+- If the fix changes DOM structure or ARIA attributes, verify with browser accessibility tools.
+
+---
+
+## Localization
+
+If the fix adds or modifies user-facing strings:
+- Add strings to the relevant resource strings interface (e.g., `IGridResourceStrings`).
+- Follow the naming convention: `igx_<component>_<key>`.
+- Flag the PR with `status: pending-localization`.
+
+---
+
+## Theming and Styles Follow-Through
+
+If the bug requires SCSS, theme wiring, or style-test changes, do not implement that work here. Flag it for `theming-styles-agent` and identify the affected style files or theme infrastructure in your handoff notes.
+
+---
+
+## What You Do NOT Do
+
+- Do not write tests — the `tdd-test-writer-agent` handles that.
+- Do not modify component SCSS or theme infrastructure — the `theming-styles-agent` handles that.
+- Do not update `README.md` — the `component-readme-agent` handles that.
+- Do not create migration schematics — the `migration-agent` handles that.
+- Do not update `CHANGELOG.md` — the `changelog-agent` handles that.
+- Do not modify dependency manifests or lock files (`package.json`, `package-lock.json`, etc.). Ask for approval first if a dependency change is truly required.
+
+---
+
+## Breaking Changes
+
+If the fix unavoidably introduces a breaking change:
+- State clearly that a migration is required so the orchestrator can route to `migration-agent`.
+- Add a `BREAKING CHANGE:` section to the commit message body.
+- Keep the old API functional with delegation if possible.
+- Add `@deprecated` JSDoc when deprecating: `@deprecated in version X.Y.0. Use \`newName\` instead.`
+
+---
+
+## Final Self-Validation
+
+Before finishing:
+
+1. Run the smallest relevant test suite.
+2. Confirm the reproduction test and all affected existing tests pass.
+3. Run `npm run lint:lib` — must pass.
+4. Confirm the fix is minimal and does not expand scope unnecessarily.
+5. If the change is user-visible, state clearly whether a demo/sample update is recommended.
+6. If the public API or documented behavior changed, state clearly that a component README update is required.
+7. If the change is breaking, state clearly that a migration is required.
+8. If the change affects i18n strings, state clearly that localization follow-through is needed.
+
+---
+
+## Running Tests
+
+Run the smallest relevant suite:
+
+| Components changed  | Command                   |
+| ------------------- | ------------------------- |
+| Non-grid components | `npm run test:lib:others` |
+| Grid                | `npm run test:lib:grid`   |
+| Tree-grid           | `npm run test:lib:tgrid`  |
+| Hierarchical-grid   | `npm run test:lib:hgrid`  |
+| Pivot-grid          | `npm run test:lib:pgrid`  |
+
+---
+
+## Commit
+
+```
+fix(<component>): <short description> (#<issue>)
+```
+
+For breaking changes, add a `BREAKING CHANGE:` footer:
+```
+fix(<component>): <short description> (#<issue>)
+
+BREAKING CHANGE: <description of what changed>
+```
+
+Use the component name as scope. Keep the subject concise and in imperative mood.