- Notifications
You must be signed in to change notification settings - Fork12.5k
Open
Description
Update Object.assign() example to include modern spread operator alternative
Summary
The current example for "Set default objects with Object.assign" should be updated to include the modern spread operator syntax as the preferred approach, while keeping Object.assign() for reference.
Current State
The documentation currently shows Object.assign() as the "good" example:
functioncreateMenu(config){letfinalConfig=Object.assign({title:"Foo",body:"Bar",buttonText:"Baz",cancellable:true},config);returnfinalConfig;}
Proposed Change
Update the example to show the spread operator as the primary modern approach:
functioncreateMenu(config){constfinalConfig={title:"Foo",body:"Bar",buttonText:"Baz",cancellable:true, ...config// Override defaults with user config};returnfinalConfig;}
Suggested Implementation
Show both approaches with clear preference:
// Modern approach (recommended)functioncreateMenu(config){constfinalConfig={title:"Foo",body:"Bar",buttonText:"Baz",cancellable:true, ...config};returnfinalConfig;}// Alternative using Object.assign()functioncreateMenuAlternative(config){constfinalConfig=Object.assign({title:"Foo",body:"Bar",buttonText:"Baz",cancellable:true},config);returnfinalConfig;}
Metadata
Metadata
Assignees
Labels
No labels