refactor(radio-group): revert event rename

Author: damyanpetev

CHANGELOG.md

@@ -37,7 +37,6 @@ ng update igniteui-angular --migrate-only --from=20.1.0 --to=21.0.0 --migrate-im
 
 **Type Renames (to avoid conflicts):**
 - `Direction` → `IgxCarouselDirection` (in carousel)
-- `IChangeCheckboxEventArgs` → `IChangeRadioEventArgs` (in radio)
 
 **Benefits:**
 - Better tree-shaking - unused components won't be bundled

projects/igniteui-angular/checkbox/src/checkbox/public_api.ts

@@ -1,2 +1,2 @@
-export { LabelPosition, IChangeCheckboxEventArgs } from "igniteui-angular/directives";
+export { LabelPosition, type IChangeCheckboxEventArgs } from "igniteui-angular/directives";
 export * from "./checkbox.component";

projects/igniteui-angular/directives/src/public_api.ts

@@ -25,7 +25,7 @@ export * from './directives/layout/layout.directive';
 export * from './directives/mask/mask.directive';
 // Note: Radio-group directive moved to radio entry point in v21.0.0
 // Users should now import from 'igniteui-angular/radio' instead
-// export { IgxRadioGroupDirective, IChangeRadioEventArgs } from 'igniteui-angular/radio';
+// export { IgxRadioGroupDirective } from 'igniteui-angular/radio';
 export * from './directives/ripple/ripple.directive';
 export * from './directives/scroll-inertia/scroll_inertia.directive';
 export * from './directives/size/ig-size.directive';

projects/igniteui-angular/migrations/update-21_0_0/README.md

@@ -32,7 +32,6 @@ The following directives have been moved to new entry points:
 The following types have been renamed to avoid conflicts:
 
 - `Direction` → `IgxCarouselDirection` (carousel)
-- `IChangeCheckboxEventArgs` → `IChangeRadioEventArgs` (radio)
 
 ### Example
 

projects/igniteui-angular/migrations/update-21_0_0/changes/classes.json

@@ -4,10 +4,6 @@
         {
             "name": "Direction",
             "replaceWith": "IgxCarouselDirection"
-        },
-        {
-            "name": "IChangeCheckboxEventArgs",
-            "replaceWith": "IChangeRadioEventArgs"
         }
     ]
 }

projects/igniteui-angular/migrations/update-21_0_0/index.ts

@@ -349,7 +349,6 @@ const ENTRY_POINT_MAP = new Map<string, string>([
     ['IgxRadioComponent', 'radio'],
     ['IgxRadioModule', 'radio'],
     ['IgxRadioGroupDirective', 'radio'], // Breaking change - moved from directives
-    ['IChangeRadioEventArgs', 'radio'], // Renamed from IChangeCheckboxEventArgs
 
     // Select
     ['IgxSelectComponent', 'select'],
@@ -446,7 +445,6 @@ const ENTRY_POINT_MAP = new Map<string, string>([
 // Type renames (old name -> new name and entry point)
 const TYPE_RENAMES = new Map<string, { newName: string, entryPoint: string }>([
     ['Direction', { newName: 'IgxCarouselDirection', entryPoint: 'carousel' }],
-    ['IChangeCheckboxEventArgs', { newName: 'IChangeRadioEventArgs', entryPoint: 'radio' }],
 ]);
 
 function migrateImportDeclaration(node: ts.ImportDeclaration, sourceFile: ts.SourceFile): { start: number, end: number, replacement: string } | null {
@@ -645,7 +643,6 @@ export default (options: MigrationOptions = {}): Rule => async (host: Tree, cont
         context.logger.info('  - IgxRadioGroupDirective moved to igniteui-angular/radio');
         context.logger.info('Type renames:');
         context.logger.info('  - Direction → IgxCarouselDirection');
-        context.logger.info('  - IChangeCheckboxEventArgs → IChangeRadioEventArgs');
     } else {
         context.logger.info('Skipping import migration. You can continue using the main entry point.');
         context.logger.info('Note: The library now supports granular entry points for better tree-shaking.');

projects/igniteui-angular/radio/README.md

@@ -88,7 +88,7 @@ export class RadioSampleComponent {
 | `@Input()` disableRipple | boolean | Specifies the whether the ripple effect should be disabled for the radio button. |
 | `@Input()` labelPosition | string `|` enum RadioLabelPosition | Specifies the position of the text label relative to the radio button element. Possible values are "before" and "after". |
 | `@Input("aria-labelledby")` ariaLabelledBy | string | Specify an external element by id to be used as label for the radio button. |
-| `@Output()` change | EventEmitter<IChangeRadioEventArgs> | Emitted when the radio button checked value changes. |
+| `@Output()` change | EventEmitter<IChangeCheckboxEventArgs> | Emitted when the radio button checked value changes. |
 
 ### Methods
 

projects/igniteui-angular/radio/src/public_api.ts

@@ -1 +1,2 @@
+export type { IChangeCheckboxEventArgs } from "igniteui-angular/directives";
 export * from './radio/public_api';

projects/igniteui-angular/radio/src/radio/radio-group/radio-group.directive.ts

@@ -19,13 +19,7 @@ import { ControlValueAccessor, NgControl, Validators } from '@angular/forms';
 import { fromEvent, noop, Subject, takeUntil } from 'rxjs';
 import { IgxRadioComponent } from '../radio.component';
 import { ɵIgxDirectionality } from 'igniteui-angular/core';
-import { IBaseEventArgs } from 'igniteui-angular/core';
-
-export interface IChangeRadioEventArgs extends IBaseEventArgs {
-    checked: boolean;
-    value?: any;
-}
-
+import { IChangeCheckboxEventArgs } from 'igniteui-angular/directives';
 /**
  * Determines the Radio Group alignment
  */
@@ -196,7 +190,7 @@ export class IgxRadioGroupDirective implements ControlValueAccessor, OnDestroy,
      * ```
      */
     // eslint-disable-next-line @angular-eslint/no-output-native
-    @Output() public readonly change: EventEmitter<IChangeRadioEventArgs> = new EventEmitter<IChangeRadioEventArgs>();
+    @Output() public readonly change: EventEmitter<IChangeCheckboxEventArgs> = new EventEmitter<IChangeCheckboxEventArgs>();
 
     /**
      * The css class applied to the component.
@@ -559,7 +553,7 @@ export class IgxRadioGroupDirective implements ControlValueAccessor, OnDestroy,
             takeUntil(button.destroy$),
             takeUntil(this.destroy$),
             takeUntil(this.queryChange$)
-        ).subscribe((ev: IChangeRadioEventArgs) => this._selectedRadioButtonChanged(ev));
+        ).subscribe((ev: IChangeCheckboxEventArgs) => this._selectedRadioButtonChanged(ev));
 
         button.blurRadio
             .pipe(takeUntil(this.destroy$))
@@ -574,7 +568,7 @@ export class IgxRadioGroupDirective implements ControlValueAccessor, OnDestroy,
      * @hidden
      * @internal
      */
-    private _selectedRadioButtonChanged(args: IChangeRadioEventArgs) {
+    private _selectedRadioButtonChanged(args: IChangeCheckboxEventArgs) {
         this._radioButtons().forEach((button) => {
             button.checked = button.id === args.owner.id;
             if (button.checked && button.ngControl) {