From 248ca04e464b513d9faafb8dc6277d547a909f92 Mon Sep 17 00:00:00 2001 From: Daniel Stoian <63335068+daniel-stoian-lgp@users.noreply.github.com> Date: Tue, 17 Dec 2024 01:57:41 +0200 Subject: [PATCH] WRR-10797: Fixed typos and grammar errors (#1758) * fixed unsafe urls, deprecated functions and incorrect JSDocs * fixed grammar errors * Fixed typos in sandstone code * Update samples/qa-a11y/src/components/Section.js Co-authored-by: Stanca <63341832+stanca-pop-lgp@users.noreply.github.com> * reverted removal of subtraction * reverted change of URLs * revert unsafe url * reverted jsdoc issues * reverted jsdoc issues * reverted deprecated functions --------- Co-authored-by: Stanca <63341832+stanca-pop-lgp@users.noreply.github.com> --- CHANGELOG.md | 2 +- Checkbox/Checkbox.js | 2 +- Checkbox/Checkbox.module.less | 2 +- .../ContextualMenuDecorator.js | 2 +- .../ContextualPopupDecorator.js | 4 +- DayPicker/DayPicker.js | 2 +- DayPicker/DaySelectorDecorator.js | 2 +- DayPicker/tests/DayPicker-specs.js | 2 +- .../FlexiblePopupPanels.module.less | 2 +- IconItem/IconItem.js | 4 +- Input/InputField.js | 2 +- Input/Keypad.js | 2 +- MediaPlayer/MediaControls.js | 6 +-- MediaPlayer/Times.js | 2 +- PageViews/tests/PageViews-specs.js | 32 +++++++------- Picker/Picker.js | 2 +- Popup/Popup.js | 6 +-- PopupTabLayout/PopupTabLayout.js | 4 +- QuickGuidePanels/QuickGuidePanels.js | 2 +- .../tests/QuickGuidePanels-specs.js | 40 ++++++++--------- RangePicker/RangePicker.js | 2 +- Scroller/EditableWrapper.js | 8 ++-- Scroller/useThemeScroller.js | 2 +- Spinner/Spinner.js | 2 +- Sprite/Sprite.js | 8 ++-- TabLayout/TabLayout.js | 4 +- TabLayout/tests/TabLayout-specs.js | 4 +- ThemeDecorator/fontGenerator.js | 2 +- TooltipDecorator/Tooltip.js | 4 +- TooltipDecorator/TooltipDecorator.js | 2 +- TooltipDecorator/useTooltip.js | 2 +- TooltipDecorator/util.js | 12 ++--- VideoPlayer/FeedbackTooltip.js | 2 +- VideoPlayer/MediaTitle.js | 2 +- VideoPlayer/Video.js | 4 +- VideoPlayer/VideoPlayer.js | 20 ++++----- VirtualList/VirtualList.js | 4 +- VirtualList/useSpotlight.js | 2 +- VirtualList/useThemeVirtualList.js | 4 +- WizardPanels/tests/WizardPanels-specs.js | 44 +++++++++---------- docs/virtual-list-scroller.md | 2 +- internal/$L/$L.js | 4 +- internal/Panels/Arrangers.js | 2 +- internal/Panels/PanelsRouter.js | 2 +- internal/Panels/PopupDecorator.js | 4 +- internal/Panels/util.js | 4 +- internal/Picker/tests/Picker-specs.js | 6 +-- internal/WindowEventable/WindowEventable.js | 4 +- .../components/InputBoard/Filter/Filter.js | 4 +- .../src/components/InputBoard/InputBoard.js | 6 +-- .../views/VerticalDifferentHeightItemList.js | 6 +-- samples/qa-a11y/src/App/App.module.less | 6 +-- samples/qa-a11y/src/components/Section.js | 2 +- .../src/views/VirtualGridList.module.less | 2 +- samples/qa-i18n/src/App/attachErrorHandler.js | 2 +- .../README.md | 2 +- .../ThemeEnvironment.module.less | 4 +- samples/sampler/stories/default/Button.js | 2 +- samples/sampler/stories/default/Image.js | 2 +- samples/sampler/stories/default/Panel.js | 2 +- samples/sampler/stories/qa/Button.js | 2 +- .../stories/qa/common/Header_Common.js | 2 +- .../KitchenSinkSection/KitchenSinkSection.js | 2 +- tests/screenshot/apps/Sandstone-View.js | 4 +- tests/screenshot/apps/components/Button.js | 6 +-- tests/screenshot/apps/components/Icon.js | 4 +- .../screenshot/apps/components/InputField.js | 2 +- tests/screenshot/apps/components/Picker.js | 8 ++-- .../screenshot/apps/components/RangePicker.js | 4 +- tests/ui/apps/Slider/Slider-View.js | 2 +- .../ContextualMenuDecorator-specs.js | 2 +- .../ContextualPopupDecorator-specs.js | 4 +- tests/ui/specs/Item/Item-specs.js | 2 +- tests/ui/specs/Popup/Popup-specs.js | 2 +- .../RangePicker/RangePicker-RTL-specs.js | 2 +- .../ListOfThings/Scroller-QWTC-2086-specs.js | 2 +- .../ListOfThings/Scroller-QWTC-2134-specs.js | 2 +- .../TabLayoutControlledIndex-specs.js | 4 +- .../ui/specs/VideoPlayer/VideoPlayer-specs.js | 2 +- .../VirtualGridList-QWTC-2301-specs.js | 4 +- .../Samples/VirtualList-QWTC-2512-specs.js | 2 +- .../VirtualList/VirtualList-specs.js | 2 +- .../onScroll/VirtualList-QWTC-2057-specs.js | 2 +- useScroll/useScroll.js | 2 +- 84 files changed, 197 insertions(+), 197 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 928afb929c..47891fd5c5 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -320,7 +320,7 @@ No significant changes. ### Fixed -- `sandstone/Dropdown' to focus properly the first option and the last option via page up and page down +- `sandstone/Dropdown` to focus properly the first option and the last option via page up and page down - `sandstone/QuickGuidePanels` to not lose focus when the last view is displayed ## [2.7.5] - 2023-08-04 diff --git a/Checkbox/Checkbox.js b/Checkbox/Checkbox.js index 30b6aebf6d..162f23472c 100644 --- a/Checkbox/Checkbox.js +++ b/Checkbox/Checkbox.js @@ -146,7 +146,7 @@ const CheckboxBase = kind({ computed: { className: ({indeterminate, selected, standalone, styler}) => styler.append({selected, standalone, indeterminate}), - children: ({indeterminate, indeterminateIcon, children}) => (indeterminate ? indeterminateIcon : children) // This controls which icon to use, an not that icon's visual presence. + children: ({indeterminate, indeterminateIcon, children}) => (indeterminate ? indeterminateIcon : children) // This controls which icon to use, and not that icon's visual presence. }, render: ({children, css, disabled, selected, ...rest}) => { diff --git a/Checkbox/Checkbox.module.less b/Checkbox/Checkbox.module.less index e813f266c0..5f2f35293d 100644 --- a/Checkbox/Checkbox.module.less +++ b/Checkbox/Checkbox.module.less @@ -128,7 +128,7 @@ line-height: 0; .icon { - margin: 0; // Needed to apply here insetad of on `.checkbox` due to .icon.size precedence :( + margin: 0; // Needed to apply here instead of on `.checkbox` due to .icon.size precedence :( border-radius: @sand-checkbox-border-radius; position: relative; text-align: center; diff --git a/ContextualMenuDecorator/ContextualMenuDecorator.js b/ContextualMenuDecorator/ContextualMenuDecorator.js index 237fae1c33..b8c42c8527 100644 --- a/ContextualMenuDecorator/ContextualMenuDecorator.js +++ b/ContextualMenuDecorator/ContextualMenuDecorator.js @@ -62,7 +62,7 @@ const ScrollingRepeater = ({className, ...rest}) => ( ); const ContextualMenuDecoratorBase = hoc(defaultConfig, (config, Wrapped) => { - // we might not need Skinnable at all here. If we want to skin the popup and it's defined as a + // we might not need Skinnable at all here. If we want to skin the popup, and it's defined as a // private component in this module, we can wrap it with skinnable and style it as needed there. const Component = Skinnable( ContextualPopupDecorator( diff --git a/ContextualPopupDecorator/ContextualPopupDecorator.js b/ContextualPopupDecorator/ContextualPopupDecorator.js index abdcb51ffb..d3bb7ca85a 100644 --- a/ContextualPopupDecorator/ContextualPopupDecorator.js +++ b/ContextualPopupDecorator/ContextualPopupDecorator.js @@ -313,7 +313,7 @@ const Decorator = hoc(defaultConfig, (config, Wrapped) => { snapshot.shouldSpotActivator = ( // isn't set !current || - // is on the activator and we want to re-spot it so a11y read out can occur + // is on the activator, and we want to re-spot it so a11y read out can occur current === prevState.activator || // is within the popup this.containerNode.contains(current) @@ -358,7 +358,7 @@ const Decorator = hoc(defaultConfig, (config, Wrapped) => { } generateId = () => { - return Math.random().toString(36).substr(2, 8); + return Math.random().toString(36).substring(2, 10); }; getContainerNodeWidth () { diff --git a/DayPicker/DayPicker.js b/DayPicker/DayPicker.js index 1a136fe734..b1f0481a04 100644 --- a/DayPicker/DayPicker.js +++ b/DayPicker/DayPicker.js @@ -54,7 +54,7 @@ const DayPickerBase = kind({ disabled: PropTypes.bool, /** - * Called when an day is selected or unselected. + * Called when a day is selected or unselected. * * The event payload will be an object with the following members: * * `selected` - An array of numbers representing the selected days, 0 indexed where Sunday diff --git a/DayPicker/DaySelectorDecorator.js b/DayPicker/DaySelectorDecorator.js index c2eb106cfb..e173c6bfed 100644 --- a/DayPicker/DaySelectorDecorator.js +++ b/DayPicker/DaySelectorDecorator.js @@ -281,7 +281,7 @@ const DaySelectorDecorator = hoc((config, Wrapped) => { locale: PropTypes.string, /** - * Called when an day is selected or unselected. + * Called when a day is selected or unselected. * * The event payload will be an object with the following members: * * `selected` - An array of numbers representing the selected days, 0 indexed diff --git a/DayPicker/tests/DayPicker-specs.js b/DayPicker/tests/DayPicker-specs.js index cf62d46943..b6fcafdb61 100644 --- a/DayPicker/tests/DayPicker-specs.js +++ b/DayPicker/tests/DayPicker-specs.js @@ -29,7 +29,7 @@ describe('DayPicker', () => { // We need to change the locale to firstDayOfWeek !== 0. // If firstDayOfWeek === 0, the number type check conditional statement // is skipped due to the fast execution path of localizeSelected(), which - // reduces code coverage.. + // reduces code coverage. ilib.setLocale('es-ES'); render(); diff --git a/FlexiblePopupPanels/FlexiblePopupPanels.module.less b/FlexiblePopupPanels/FlexiblePopupPanels.module.less index 136b87f34a..eea8378181 100644 --- a/FlexiblePopupPanels/FlexiblePopupPanels.module.less +++ b/FlexiblePopupPanels/FlexiblePopupPanels.module.less @@ -140,7 +140,7 @@ box-sizing: border-box; } - &.auto { /* Availble for export */ } + &.auto { /* Available for export */ } // These calculations are here (rather than the variables file) because they're // implementation-specific rather than being generally useful due to the container diff --git a/IconItem/IconItem.js b/IconItem/IconItem.js index 2d8eca058a..c407cb6f11 100644 --- a/IconItem/IconItem.js +++ b/IconItem/IconItem.js @@ -96,7 +96,7 @@ const IconItemBase = kind({ bordered: PropTypes.bool, /** - * The custom component rendred as the content of this IconItem. + * The custom component rendered as the content of this IconItem. * * @type {Component} * @public @@ -152,7 +152,7 @@ const IconItemBase = kind({ /** * Source and size for the image. - * See the datails in {@link sandstone/IconItem.ImageShape|ImageShape} + * See the details in {@link sandstone/IconItem.ImageShape|ImageShape} * * @type {sandstone/IconItem.ImageShape} * @public diff --git a/Input/InputField.js b/Input/InputField.js index 2a18daa3b1..146a47e011 100644 --- a/Input/InputField.js +++ b/Input/InputField.js @@ -26,7 +26,7 @@ import componentCss from './InputField.module.less'; /** * A Sandstone styled input component. * - * It supports start and end icons but it does not support Spotlight. Apps should use + * It supports start and end icons, but it does not support Spotlight. Apps should use * {@link sandstone/Input.InputField}. * * @class InputFieldBase diff --git a/Input/Keypad.js b/Input/Keypad.js index fd97c026b2..f8b2059cd3 100644 --- a/Input/Keypad.js +++ b/Input/Keypad.js @@ -95,7 +95,7 @@ const Keypad = kind({ } }); -// Setup a keypress handler for window that monitors each of the number keys and the backspace key +// Set up a keypress handler for window that monitors each of the number keys and the backspace key const handleWindowKeyPress = handle( oneOf( [forKey('number'), forward('onAdd')], diff --git a/MediaPlayer/MediaControls.js b/MediaPlayer/MediaControls.js index 1e501b4738..3646d693f8 100644 --- a/MediaPlayer/MediaControls.js +++ b/MediaPlayer/MediaControls.js @@ -116,7 +116,7 @@ const MediaControlsBase = kind({ actionGuideLabel: PropTypes.string, /** - * These components are placed below the action guide. Typically these will be media playlist controls. + * These components are placed below the action guide. Typically, these will be media playlist controls. * * @type {Node} * @public @@ -446,7 +446,7 @@ const MediaControlsDecorator = hoc((config, Wrapped) => { actionGuideLabel: PropTypes.string, /** - * These components are placed below the children. Typically these will be media playlist items. + * These components are placed below the children. Typically, these will be media playlist items. * * @type {Node} * @public @@ -568,7 +568,7 @@ const MediaControlsDecorator = hoc((config, Wrapped) => { playPauseButtonDisabled: PropTypes.bool, /** - * Disables the media playback-rate control via rewind and fast forward keys + * Disables the media playback-rate control via rewind and fast-forward keys * * @type {Boolean} * @public diff --git a/MediaPlayer/Times.js b/MediaPlayer/Times.js index 120ababae8..5047d71120 100644 --- a/MediaPlayer/Times.js +++ b/MediaPlayer/Times.js @@ -22,7 +22,7 @@ const TimesBase = kind({ /** * An instance of a Duration Formatter from i18n. * - * Must has a `format()` method that returns a string. + * Must have a `format()` method that returns a string. * * @type {Object} * @required diff --git a/PageViews/tests/PageViews-specs.js b/PageViews/tests/PageViews-specs.js index 01998eb022..d61a3b5f68 100644 --- a/PageViews/tests/PageViews-specs.js +++ b/PageViews/tests/PageViews-specs.js @@ -10,8 +10,8 @@ describe('PageViews Specs', () => { () => { render( - I gots contents - I gots contents + I got contents + I got contents ); @@ -28,8 +28,8 @@ describe('PageViews Specs', () => { () => { render( - I gots contents - I gots contents + I got contents + I got contents ); @@ -46,8 +46,8 @@ describe('PageViews Specs', () => { () => { render( - I gots contents - I gots contents + I got contents + I got contents ); @@ -64,8 +64,8 @@ describe('PageViews Specs', () => { () => { render( - I gots contents - I gots contents + I got contents + I got contents ); @@ -84,8 +84,8 @@ describe('PageViews Specs', () => { let index = 0; const {rerender} = render( - I gots contents - I gots contents2 + I got contents + I got contents2 ); @@ -94,8 +94,8 @@ describe('PageViews Specs', () => { rerender( - I gots contents - I gots contents2 + I got contents + I got contents2 ); @@ -115,8 +115,8 @@ describe('PageViews Specs', () => { let index = 0; const {rerender} = render( - I gots contents - I gots contents2 + I got contents + I got contents2 ); @@ -125,8 +125,8 @@ describe('PageViews Specs', () => { rerender( - I gots contents - I gots contents2 + I got contents + I got contents2 ); diff --git a/Picker/Picker.js b/Picker/Picker.js index bca2b76008..4d9e6a42de 100644 --- a/Picker/Picker.js +++ b/Picker/Picker.js @@ -270,7 +270,7 @@ const PickerBase = kind({ /** * Allows picker to continue from the start of the list after it reaches the end and - * vice-versa. + * vice versa. * * @type {Boolean} * @public diff --git a/Popup/Popup.js b/Popup/Popup.js index ab3ab660a7..3ba06ac2f9 100644 --- a/Popup/Popup.js +++ b/Popup/Popup.js @@ -311,7 +311,7 @@ class Popup extends Component { /** * Indicates that the popup will not trigger `onClose` when the user presses the cancel/back (e.g. `ESC`) key or - * taps outside of the popup. + * taps outside the popup. * * @type {Boolean} * @default false @@ -616,7 +616,7 @@ class Popup extends Component { off('keydown', this.handleKeyDown); - // if there is no currently-spotted control or it is wrapped by the popup's container, we + // if there is no currently-spotted control, or it is wrapped by the popup's container, we // know it's safe to change focus if (!current || (containerNode && containerNode.contains(current))) { // attempt to set focus to the activator, if available @@ -647,7 +647,7 @@ class Popup extends Component { if (!Spotlight.isPaused() && !Spotlight.focus(containerId)) { const current = Spotlight.getCurrent(); - // In cases where the container contains no spottable controls or we're in pointer-mode, focus + // In cases where the container contains no spottable controls, or we're in pointer-mode, focus // cannot inherently set the active container or blur the active control, so we must do that // here. if (current) { diff --git a/PopupTabLayout/PopupTabLayout.js b/PopupTabLayout/PopupTabLayout.js index c4b3b28134..3668324729 100644 --- a/PopupTabLayout/PopupTabLayout.js +++ b/PopupTabLayout/PopupTabLayout.js @@ -28,7 +28,7 @@ import Popup from '../Popup'; import componentCss from './PopupTabLayout.module.less'; -// List all of the props from PopupTabLayout that we want to move from this component's root onto PopupTabLayout. +// List all the props from PopupTabLayout that we want to move from this component's root onto PopupTabLayout. const popupPropList = ['noAutoDismiss', 'onHide', 'onKeyDown', 'onShow', 'open', 'position', 'scrimType', 'spotlightId', 'spotlightRestrict', 'id', 'className', 'style', 'noAnimation', 'onClose']; @@ -78,7 +78,7 @@ const PopupTabLayoutBase = kind({ /** * Specify dimensions for the layout areas. * - * All 4 combinations must me supplied: each of the elements, tabs and content in both + * All 4 combinations must be supplied: each of the elements, tabs and content in both * collapsed and expanded state. * * @type {{tabs: {collapsed: Number, normal: Number}, content: {expanded: number, normal: number}}} diff --git a/QuickGuidePanels/QuickGuidePanels.js b/QuickGuidePanels/QuickGuidePanels.js index 7b6e704a77..c5e1924483 100644 --- a/QuickGuidePanels/QuickGuidePanels.js +++ b/QuickGuidePanels/QuickGuidePanels.js @@ -22,7 +22,7 @@ import Steps from '../Steps'; import css from './QuickGuidePanels.module.less'; /** - * A QuickGuidePaenls that has steps with corresponding panels and panels have full screen size content. + * A QuickGuidePanels that has steps with corresponding panels and panels have full screen size content. * * @example * diff --git a/QuickGuidePanels/tests/QuickGuidePanels-specs.js b/QuickGuidePanels/tests/QuickGuidePanels-specs.js index 14db09d795..962aab52e6 100644 --- a/QuickGuidePanels/tests/QuickGuidePanels-specs.js +++ b/QuickGuidePanels/tests/QuickGuidePanels-specs.js @@ -11,8 +11,8 @@ describe('QuickGuidePanels Specs', () => { () => { render( - I gots contents - I gots contents2 + I got contents + I got contents2 Last! ); @@ -33,8 +33,8 @@ describe('QuickGuidePanels Specs', () => { async () => { render( - I gots contents - I gots contents2 + I got contents + I got contents2 Last! ); @@ -55,8 +55,8 @@ describe('QuickGuidePanels Specs', () => { async () => { render( - I gots contents - I gots contents2 + I got contents + I got contents2 Last! ); @@ -76,8 +76,8 @@ describe('QuickGuidePanels Specs', () => { let index = 0; const {rerender} = render( - I gots contents - I gots contents2 + I got contents + I got contents2 ); @@ -86,8 +86,8 @@ describe('QuickGuidePanels Specs', () => { rerender( - I gots contents - I gots contents2 + I got contents + I got contents2 ); @@ -108,9 +108,9 @@ describe('QuickGuidePanels Specs', () => { let index = 0; const {rerender} = render( - I gots contents - I gots contents2 - I gots contents3 + I got contents + I got contents2 + I got contents3 ); @@ -121,9 +121,9 @@ describe('QuickGuidePanels Specs', () => { rerender( - I gots contents - I gots contents2 - I gots contents3 + I got contents + I got contents2 + I got contents3 ); @@ -143,8 +143,8 @@ describe('QuickGuidePanels Specs', () => { let index = 0; const {rerender} = render( - I gots contents - I gots contents2 + I got contents + I got contents2 ); @@ -153,8 +153,8 @@ describe('QuickGuidePanels Specs', () => { rerender( - I gots contents - I gots contents2 + I got contents + I got contents2 ); diff --git a/RangePicker/RangePicker.js b/RangePicker/RangePicker.js index 599b400967..8015765fb0 100644 --- a/RangePicker/RangePicker.js +++ b/RangePicker/RangePicker.js @@ -269,7 +269,7 @@ const RangePickerBase = kind({ /** * Allows picker to continue from the start of the list after it reaches the end and - * vice-versa. + * vice versa. * * @type {Boolean} * @public diff --git a/Scroller/EditableWrapper.js b/Scroller/EditableWrapper.js index 6cd68fc4fa..6982c671da 100644 --- a/Scroller/EditableWrapper.js +++ b/Scroller/EditableWrapper.js @@ -767,7 +767,7 @@ const EditableWrapper = (props) => { const orders = finalizeOrders(); finalizeEditing(orders); } else if (!mutableRef.current.isDragging) { - // Cancel mouse event to select a item when it is tapped + // Cancel mouse event to select an item when it is tapped ev.preventDefault(); const targetItemNode = findItemNode(ev.target); @@ -785,10 +785,10 @@ const EditableWrapper = (props) => { const handleDragStart = useCallback((ev) => { const {selectedItem} = mutableRef.current; // Index of dragged item - const dragTagetItemIndex = getNextIndexFromPosition(ev.x, 0); + const dragTargetItemIndex = getNextIndexFromPosition(ev.x, 0); mutableRef.current.isDragging = true; - if (selectedItem && Number(selectedItem.style.order) - 1 === dragTagetItemIndex) { + if (selectedItem && Number(selectedItem.style.order) - 1 === dragTargetItemIndex) { mutableRef.current.isDraggingItem = true; } }, [getNextIndexFromPosition]); @@ -968,7 +968,7 @@ EditableWrapper.propTypes = /** @lends sandstone/Scroller.EditableWrapper.protot /** * Enables editing items in the scroller. * You can specify props for editable scroller as an object. - * See the datails in {@link sandstone/Scroller.EditableShape|EditableShape} + * See the details in {@link sandstone/Scroller.EditableShape|EditableShape} * * @type {sandstone/Scroller.EditableShape} * @public diff --git a/Scroller/useThemeScroller.js b/Scroller/useThemeScroller.js index 75b034099d..a451bc7377 100644 --- a/Scroller/useThemeScroller.js +++ b/Scroller/useThemeScroller.js @@ -304,7 +304,7 @@ const useSpottable = (props, instances) => { // `focusedItem` that has a width that is bigger than `scrollBounds.clientWidth`. nextScrollLeft += rtlDirection * ((newItemLeft + itemWidth) - (clientWidth + currentScrollLeft)); } else if (newItemLeft < currentScrollLeft) { - // If focus is outside of the view area to the left, move scroller to the left accordingly. + // If focus is outside the view area to the left, move scroller to the left accordingly. nextScrollLeft += rtlDirection * (newItemLeft - currentScrollLeft); } diff --git a/Spinner/Spinner.js b/Spinner/Spinner.js index db557ca697..1931fce088 100644 --- a/Spinner/Spinner.js +++ b/Spinner/Spinner.js @@ -78,7 +78,7 @@ const SpinnerCore = kind({ }); /** - * The base component, defining all of the properties. + * The base component, defining all the properties. * * @class SpinnerBase * @memberof sandstone/Spinner diff --git a/Sprite/Sprite.js b/Sprite/Sprite.js index d67155bf3c..1acdb09131 100644 --- a/Sprite/Sprite.js +++ b/Sprite/Sprite.js @@ -1,7 +1,7 @@ /** * A "sprite" is an animated sequence of cells. * - * Cells, like a film-strip, are pre-arranged in one or more rows and one or more columns. + * Cells, like a filmstrip, are pre-arranged in one or more rows and one or more columns. * This component steps through those cells to create an animation. * * Example: @@ -172,7 +172,7 @@ const SpriteBase = kind({ rows: PropTypes.number, /** - * The sprite-sheet image with all of the cells on it. + * The sprite-sheet image with all the cells on it. * * @see {@link ui/Image.ImageBase.src} * @type {String|Object} @@ -254,7 +254,7 @@ const SpriteBase = kind({ for (let i = 0; i < sets; i++) { const axisValue = (i * axis * -1); - const dimentionValue = ((steps - 0) * dimension * -1); + const dimensionValue = ((steps - 0) * dimension * -1); framesets.push( createKeyframe({ offset: (i / sets), @@ -265,7 +265,7 @@ const SpriteBase = kind({ createKeyframe({ offset: ((i + 1) / sets), vertical, - dimension: dimentionValue, + dimension: dimensionValue, axis: axisValue }) ); diff --git a/TabLayout/TabLayout.js b/TabLayout/TabLayout.js index 64f4bbc963..2bb1eff581 100644 --- a/TabLayout/TabLayout.js +++ b/TabLayout/TabLayout.js @@ -115,7 +115,7 @@ const TabLayoutBase = kind({ /** * Specify dimensions for the layout areas. * - * All 4 combinations must me supplied: each of the elements, tabs and content in both + * All 4 combinations must be supplied: each of the elements, tabs and content in both * collapsed and expanded state. * * @type {{tabs: {collapsed: Number, normal: Number}, content: {expanded: number, normal: number}}} @@ -450,7 +450,7 @@ const TabLayoutDecorator = compose( RefocusDecorator, SpotlightContainerDecorator({ // using last-focused so we return to the last focused if it exists but fall through to - // default element if no focus has ocurred yet (e.g. on mount) + // default element if no focus has occurred yet (e.g. on mount) enterTo: 'last-focused', // favor the content when collapsed and the tabs otherwise defaultElement: [`.${componentCss.horizontal} .${componentCss.tabs} *`, `.${componentCss.collapsed} .${componentCss.content} *`, `.${componentCss.tabsExpanded} *`] diff --git a/TabLayout/tests/TabLayout-specs.js b/TabLayout/tests/TabLayout-specs.js index b8d55015d8..aedc379c33 100644 --- a/TabLayout/tests/TabLayout-specs.js +++ b/TabLayout/tests/TabLayout-specs.js @@ -169,7 +169,7 @@ describe('TabLayout specs', () => { expect(actual).toMatchObject(expected); }); - test('should call \'onSelect\' with \'onSelect\' type when pressing \'Enter\' on a tab while \'vertial\' and \'ltr\'', () => { + test('should call \'onSelect\' with \'onSelect\' type when pressing \'Enter\' on a tab while \'vertical\' and \'ltr\'', () => { const spy = jest.fn(); render( @@ -192,7 +192,7 @@ describe('TabLayout specs', () => { expect(actual).toMatchObject(expected); }); - test('should call \'onSelect\' with \'onSelect\' type when pressing \'Enter\' on a tab while \'vertial\' and \'rtl\'', () => { + test('should call \'onSelect\' with \'onSelect\' type when pressing \'Enter\' on a tab while \'vertical\' and \'rtl\'', () => { const spy = jest.fn(); render( diff --git a/ThemeDecorator/fontGenerator.js b/ThemeDecorator/fontGenerator.js index a4371932c5..5fc95c0312 100644 --- a/ThemeDecorator/fontGenerator.js +++ b/ThemeDecorator/fontGenerator.js @@ -3,7 +3,7 @@ * * _This is not intended to be directly included by external developers._ The purpose of this is to * override the existing "Sandstone" font family with a new typeface, conditionally when the system - * locale matches the corrosponding locale for the font (defined in this component's code). + * locale matches the corresponding locale for the font (defined in this component's code). * */ let {addLocalizedFont, generateFontRules, generateFontOverrideRules} = require('@enact/ui/internal/localized-fonts'); diff --git a/TooltipDecorator/Tooltip.js b/TooltipDecorator/Tooltip.js index d5ada818ff..21f6424215 100644 --- a/TooltipDecorator/Tooltip.js +++ b/TooltipDecorator/Tooltip.js @@ -85,7 +85,7 @@ const TooltipBase = kind({ * other. This prop accepts a value between -0.5 and 0.5 (representing 50% to the left or * right). This defaults to 0 offset (centered). It also automatically caps the value so it * never positions the tooltip label past the anchored arrow. If the tooltip label or arrow - * has non-rectangular geometry (rounded corners, a wide tail, etc), you'll need to manually + * has non-rectangular geometry (rounded corners, a wide tail, etc.), you'll need to manually * account for that in your provided offset value. * * @type {Number} @@ -97,7 +97,7 @@ const TooltipBase = kind({ /** * Allows the tooltip to marquee. * - * Specifying a {@link sandstone/TooltipDecorator.TooltipBase.width|width} restrects + * Specifying a {@link sandstone/TooltipDecorator.TooltipBase.width|width} restricts * the marquee to that size. * * @type {Boolean} diff --git a/TooltipDecorator/TooltipDecorator.js b/TooltipDecorator/TooltipDecorator.js index 3c322b2e58..1711ce325a 100644 --- a/TooltipDecorator/TooltipDecorator.js +++ b/TooltipDecorator/TooltipDecorator.js @@ -116,7 +116,7 @@ const TooltipDecorator = hoc(defaultConfig, (config, Wrapped) => { * Allows the tooltip to marquee. * * Specifying a {@link sandstone/TooltipDecorator.TooltipDecorator.tooltipWidth|tooltipWidth} - * restrects the marquee to that size. + * restricts the marquee to that size. * * @type {Boolean} * @public diff --git a/TooltipDecorator/useTooltip.js b/TooltipDecorator/useTooltip.js index e6587a51ff..f8ca955994 100644 --- a/TooltipDecorator/useTooltip.js +++ b/TooltipDecorator/useTooltip.js @@ -10,7 +10,7 @@ import {useCallback, useEffect, useLayoutEffect, useRef, useState} from 'react'; import {Tooltip, defaultArrowAnchor, defaultDirection} from './Tooltip'; import {adjustDirection, adjustAnchor, calcOverflow, getLabelOffset, getPosition} from './util'; -let currentTooltip; // needed to know whether or not we should stop a showing job when unmounting +let currentTooltip; // needed to know whether we should stop a showing job when unmounting const getTooltipDirection = (tooltipPosition, tooltipType) => { const position = tooltipPosition || (defaultDirection(tooltipType) + ' ' + defaultArrowAnchor(tooltipType)); diff --git a/TooltipDecorator/util.js b/TooltipDecorator/util.js index b24b56ccbc..cb55899357 100644 --- a/TooltipDecorator/util.js +++ b/TooltipDecorator/util.js @@ -21,9 +21,9 @@ const getLabelUnavailableSpace = function (tooltipWidth) { const tooltipUnavailableEdge = ri.scale((108 / 2) + arrowWidth); // cap the offset at 50% - that percentage - const tooltipUnavaliablePercentage = 0.5 - (tooltipUnavailableEdge / tooltipWidth); + const tooltipUnavailablePercentage = 0.5 - (tooltipUnavailableEdge / tooltipWidth); - return tooltipUnavaliablePercentage; + return tooltipUnavailablePercentage; }; /** @@ -114,7 +114,7 @@ const calcOverflow = function (tooltipNode, clientNode, tooltipDirection, edgeKe const windowHeight = window.innerHeight; const clientHorizontalCenter = clientNode.left + (clientNode.width / 2); const tooltipSafeWidth = tooltipNode.width + edgeKeepout; - const tooltipCenterdSafeWidth = (tooltipNode.width / 2) + edgeKeepout; + const tooltipCenteredSafeWidth = (tooltipNode.width / 2) + edgeKeepout; const rightDelta = tooltipSafeWidth > clientHorizontalCenter; const leftDelta = tooltipSafeWidth > windowWidth - clientHorizontalCenter; const isTooltipWide = (tooltipSafeWidth > windowWidth) || (leftDelta && rightDelta); @@ -124,8 +124,8 @@ const calcOverflow = function (tooltipNode, clientNode, tooltipDirection, edgeKe const isOverBottom = clientNode.bottom + tooltipNode.height + edgeKeepout; const isOverLeft = clientHorizontalCenter - tooltipSafeWidth; const isOverRight = clientHorizontalCenter + tooltipSafeWidth; - const isOverCenterLeft = clientHorizontalCenter - tooltipCenterdSafeWidth; - const isOverCenterRight = clientHorizontalCenter + tooltipCenterdSafeWidth; + const isOverCenterLeft = clientHorizontalCenter - tooltipCenteredSafeWidth; + const isOverCenterRight = clientHorizontalCenter + tooltipCenteredSafeWidth; return { isOverTop: (isOverTop < 0) ? isOverTop : false, isOverBottom: (isOverBottom > windowHeight) ? (isOverBottom - windowHeight) : false, @@ -208,7 +208,7 @@ const getPosition = function (clientNode, tooltipDirection) { * @private */ const getLabelOffset = function (tooltipNode, tooltipDirection, tooltipPosition, overflow) { - // This method is irrelevent to left and right anchored tooltips, skip entirely. + // This method is irrelevant to left and right anchored tooltips, skip entirely. if (tooltipDirection !== 'left' && tooltipDirection !== 'right') { const tooltipWidth = tooltipNode.width; diff --git a/VideoPlayer/FeedbackTooltip.js b/VideoPlayer/FeedbackTooltip.js index 7de3847a03..0195cd2475 100644 --- a/VideoPlayer/FeedbackTooltip.js +++ b/VideoPlayer/FeedbackTooltip.js @@ -118,7 +118,7 @@ const FeedbackTooltipBase = kind({ /** * Set a thumbnail image source to show on VideoPlayer's Slider knob. This is a standard - * {@link sandstone/Image} component so it supports all of the same options for the `src` + * {@link sandstone/Image} component so it supports all the same options for the `src` * property. If no `thumbnailSrc` is set, no tooltip will display. * * @type {String|Object} diff --git a/VideoPlayer/MediaTitle.js b/VideoPlayer/MediaTitle.js index 7caf4d3fc2..206996db25 100644 --- a/VideoPlayer/MediaTitle.js +++ b/VideoPlayer/MediaTitle.js @@ -31,7 +31,7 @@ const MediaTitleBase = kind({ id: PropTypes.string.isRequired, /** - * Anything supplied to `children` will be rendered. Typically this will be informational + * Anything supplied to `children` will be rendered. Typically, this will be informational * badges indicating aspect ratio, audio channels, etc., but it could also be a description. * * @type {Node} diff --git a/VideoPlayer/Video.js b/VideoPlayer/Video.js index d236aa193d..1607d28fd6 100644 --- a/VideoPlayer/Video.js +++ b/VideoPlayer/Video.js @@ -184,9 +184,9 @@ const VideoBase = class extends Component { if (playPromise) { playPromise.then(() => { - // Auto-play started + // Autoplay started }).catch(() => { - // Auto-play was prevented + // Autoplay was prevented }); } } diff --git a/VideoPlayer/VideoPlayer.js b/VideoPlayer/VideoPlayer.js index 6895238a48..3697e760c2 100644 --- a/VideoPlayer/VideoPlayer.js +++ b/VideoPlayer/VideoPlayer.js @@ -139,10 +139,10 @@ const AnnounceState = { // The title should be announced TITLE: 1, - // The title has been announce + // The title has been announced TITLE_READ: 2, - // The infoComponents should be announce + // The infoComponents should be announced INFO: 3, // All announcements have been made @@ -250,7 +250,7 @@ const VideoPlayerBase = class extends Component { /** * Components placed below the title. * - * Typically these will be media descriptor icons, like how many audio channels, what codec + * Typically, these will be media descriptor icons, like how many audio channels, what codec * the video uses, but can also be a description for the video or anything else that seems * appropriate to provide information about the video to the user. * @@ -388,7 +388,7 @@ const VideoPlayerBase = class extends Component { noAutoShowMediaControls: PropTypes.bool, /** - * Hides media slider feedback when fast forward or rewind while media controls are hidden. + * Hides media slider feedback when fast-forward or rewind while media controls are hidden. * * @type {Boolean} * @default false @@ -499,7 +499,7 @@ const VideoPlayerBase = class extends Component { * the current playback position. * * It is passed an object with a `seconds` key (float value) to indicate the current time - * index. It can be used to update the `thumbnailSrc` to the reflect the current scrub + * index. It can be used to update the `thumbnailSrc` to reflect the current scrub * position. * * @type {Function} @@ -593,7 +593,7 @@ const VideoPlayerBase = class extends Component { /** * Pauses the video when it reaches either the start or the end of the video during rewind, - * slow rewind, fast forward, or slow forward. + * slow rewind, fast-forward, or slow forward. * * @type {Boolean} * @default false @@ -701,7 +701,7 @@ const VideoPlayerBase = class extends Component { /** * Thumbnail image source to show on the slider knob. * - * This is a standard {@link sandstone/Image} component so it supports all of the same + * This is a standard {@link sandstone/Image} component so it supports all the same * options for the `src` property. If no `thumbnailComponent` and no `thumbnailSrc` is set, * no tooltip will display. * @@ -1694,7 +1694,7 @@ const VideoPlayerBase = class extends Component { }; /** - * Changes playbackRate to a valid value when initiating fast forward or rewind. + * Changes playbackRate to a valid value when initiating fast-forward or rewind. * * @param {Number} idx - The index of the desired playback rate. * @private @@ -1807,7 +1807,7 @@ const VideoPlayerBase = class extends Component { // More props from `ev` may be added here as needed, but a full copy via `...ev` // overloads Storybook's Action Logger and likely has other perf fallout. type: ev.type, - // Specific state variables are included in the outgoing calback payload, not all of them + // Specific state variables are included in the outgoing callback payload, not all of them ...this.getMediaState() }; }; @@ -1924,7 +1924,7 @@ const VideoPlayerBase = class extends Component { forwardToggleMore(ev, this.props); if (!showMoreComponents) { - this.startAutoCloseTimeout(); // Restore the timer since we are leaving "more. + this.startAutoCloseTimeout(); // Restore the timer since we are leaving "more". // Restore the title-hide now that we're finished with "more". this.startDelayedTitleHide(); } else { diff --git a/VirtualList/VirtualList.js b/VirtualList/VirtualList.js index 2b20efdfbc..b4d9f0c8fc 100644 --- a/VirtualList/VirtualList.js +++ b/VirtualList/VirtualList.js @@ -415,7 +415,7 @@ VirtualList.propTypes = /** @lends sandstone/VirtualList.VirtualList.prototype * }), /** - * When `true`, the list will scroll by page. Otherwise the list will scroll by item. + * When `true`, the list will scroll by page. Otherwise, the list will scroll by item. * * @type {Boolean} * @default false @@ -890,7 +890,7 @@ VirtualGridList.propTypes = /** @lends sandstone/VirtualList.VirtualGridList.pro }), /** - * When `true`, the list will scroll by page. Otherwise the list will scroll by item. + * When `true`, the list will scroll by page. Otherwise, the list will scroll by item. * * @type {Boolean} * @default false diff --git a/VirtualList/useSpotlight.js b/VirtualList/useSpotlight.js index 0c407262e9..48583ad761 100644 --- a/VirtualList/useSpotlight.js +++ b/VirtualList/useSpotlight.js @@ -43,7 +43,7 @@ const useSpotlightConfig = (props, instances) => { lastFocusedPersist, /* - * Restores the data-index into the placeholder if its the only element. Tries to find a + * Restores the data-index into the placeholder if it's the only element. Tries to find a * matching child otherwise. */ lastFocusedRestore, diff --git a/VirtualList/useThemeVirtualList.js b/VirtualList/useThemeVirtualList.js index 6a2500e8cd..19407f90e1 100644 --- a/VirtualList/useThemeVirtualList.js +++ b/VirtualList/useThemeVirtualList.js @@ -228,7 +228,7 @@ const useSpottable = (props, instances) => { let returnVal = false; if (!itemNode && index >= 0 && index < props.dataSize) { - // Item is valid but since the the dom doesn't exist yet, we set the index to focus after the ongoing update + // Item is valid but since the dom doesn't exist yet, we set the index to focus after the ongoing update setPreservedIndex(index, direction); } else { const @@ -297,7 +297,7 @@ const useSpottable = (props, instances) => { } // Since the result is used as a target position to be scrolled, - // scrondaryPosition should be 0 here. + // secondaryPosition should be 0 here. gridPosition.secondaryPosition = 0; return scrollContentHandle.current.gridPositionToItemPosition(gridPosition); diff --git a/WizardPanels/tests/WizardPanels-specs.js b/WizardPanels/tests/WizardPanels-specs.js index 0c438c097a..38bd025026 100644 --- a/WizardPanels/tests/WizardPanels-specs.js +++ b/WizardPanels/tests/WizardPanels-specs.js @@ -109,8 +109,8 @@ describe('WizardPanels Specs', () => { async () => { render( - I gots contents - I gots contents2 + I got contents + I got contents2 Last ); @@ -129,8 +129,8 @@ describe('WizardPanels Specs', () => { () => { render( - I gots contents - I gots contents2 + I got contents + I got contents2 Last! ); @@ -150,8 +150,8 @@ describe('WizardPanels Specs', () => { async () => { render( - I gots contents - I gots contents2 + I got contents + I got contents2 Last! ); @@ -175,8 +175,8 @@ describe('WizardPanels Specs', () => { async () => { render( - I gots contents - I gots contents2 + I got contents + I got contents2 Last! ); @@ -418,8 +418,8 @@ describe('WizardPanels Specs', () => { let index = 0; const {rerender} = render( - I gots contents - I gots contents2 + I got contents + I got contents2 ); @@ -428,8 +428,8 @@ describe('WizardPanels Specs', () => { rerender( - I gots contents - I gots contents2 + I got contents + I got contents2 ); @@ -450,9 +450,9 @@ describe('WizardPanels Specs', () => { let index = 0; const {rerender} = render( - I gots contents - I gots contents2 - I gots contents3 + I got contents + I got contents2 + I got contents3 ); @@ -463,9 +463,9 @@ describe('WizardPanels Specs', () => { rerender( - I gots contents - I gots contents2 - I gots contents3 + I got contents + I got contents2 + I got contents3 ); @@ -485,8 +485,8 @@ describe('WizardPanels Specs', () => { let index = 0; const {rerender} = render( - I gots contents - I gots contents2 + I got contents + I got contents2 ); @@ -495,8 +495,8 @@ describe('WizardPanels Specs', () => { rerender( - I gots contents - I gots contents2 + I got contents + I got contents2 ); diff --git a/docs/virtual-list-scroller.md b/docs/virtual-list-scroller.md index c4d34275ef..f6f0be9736 100644 --- a/docs/virtual-list-scroller.md +++ b/docs/virtual-list-scroller.md @@ -155,7 +155,7 @@ This document describes VirtualList, VirtualGridList, and Scroller. * Do not change `cbScrollTo` prop after a list or a scroller is initially rendered. It does not have any effect. * The binding of a callback function * Please make sure the context of a callback function is properly bound. In general, your app component instance would be the right one. - * We recommend to use ECMAScript 2015 (a.k.a ECMAScript 6 or ES6) arrow functions to handle binding. + * We recommend to use ECMAScript 2015 (a.k.a. ECMAScript 6 or ES6) arrow functions to handle binding. * [https://googlechrome.github.io/samples/arrows-es6/](https://googlechrome.github.io/samples/arrows-es6/) * [https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions) * Example: diff --git a/internal/$L/$L.js b/internal/$L/$L.js index 96f91b11fa..2dc42491b0 100644 --- a/internal/$L/$L.js +++ b/internal/$L/$L.js @@ -18,7 +18,7 @@ function getResBundle () { /** * Creates a new ilib.ResBundle for string translation * - * @param {ilib.Locale} locale Locale for ResBundle + * @param {ilib.Locale} options Locale for ResBundle * * @returns {Promise|ResBundle} Resolves with a new ilib.ResBundle */ @@ -54,7 +54,7 @@ function clearResBundle () { * Set the locale for the strings that $L loads. This may reload the * string resources if necessary. * - * @param {string} spec the locale specifier + * @param {string} bundle the locale specifier * @returns {ilib.ResBundle} Current ResBundle */ function setResBundle (bundle) { diff --git a/internal/Panels/Arrangers.js b/internal/Panels/Arrangers.js index 315a3333e3..bae4f3bd41 100644 --- a/internal/Panels/Arrangers.js +++ b/internal/Panels/Arrangers.js @@ -171,7 +171,7 @@ const FadeAndSlideArranger = { }; /** - * Arranger that cross fade between panels. + * Arranger that cross-fade between panels. * * @type {Arranger} * @private diff --git a/internal/Panels/PanelsRouter.js b/internal/Panels/PanelsRouter.js index 5e2e966234..7423f63c40 100644 --- a/internal/Panels/PanelsRouter.js +++ b/internal/Panels/PanelsRouter.js @@ -16,7 +16,7 @@ const PanelsContext = createContext(null); function useReverseTransition (index = -1, rtl) { const prevIndex = useRef(index); const reverse = useRef(rtl); - // If the index was changed, the panel transition is occured on the next cycle by `Panel` + // If the index was changed, the panel transition occurs on the next cycle by `Panel` const prev = {reverseTransition: reverse.current, prevIndex: prevIndex.current}; if (prevIndex.current !== index) { diff --git a/internal/Panels/PopupDecorator.js b/internal/Panels/PopupDecorator.js index 98ece8526f..aea422a42c 100644 --- a/internal/Panels/PopupDecorator.js +++ b/internal/Panels/PopupDecorator.js @@ -12,7 +12,7 @@ import Popup from '../../Popup'; import CancelDecorator from './CancelDecorator'; import TransitionDecorator from './TransitionDecorator'; -// List all of the props from Popup that we want to move from this component's root onto Popup. +// List all the props from Popup that we want to move from this component's root onto Popup. const popupPropList = ['noAutoDismiss', 'onHide', 'onKeyDown', 'onShow', 'open', 'position', 'scrimType', 'spotlightId', 'spotlightRestrict']; @@ -111,7 +111,7 @@ const PopupDecorator = hoc(defaultConfig, (config, Wrapped) => { * * Normally, the popup will flex in height to match the size of the content until the * screen bounds are met. Use this if you require a full-height popup but don't have - * enough content to fill the space or you want a component to stretch to the edges. + * enough content to fill the space, or you want a component to stretch to the edges. * * @type {Boolean} * @public diff --git a/internal/Panels/util.js b/internal/Panels/util.js index 33192f5bca..efc4414038 100644 --- a/internal/Panels/util.js +++ b/internal/Panels/util.js @@ -59,9 +59,9 @@ const ContextAsDefaults = hoc(defaultConfig, (config, Wrapped) => { const {contextProps, provideContextAsDefaults} = useContextAsDefaults(props, sharedProps); - // The following generates a complete list of all of the props expected by Wrapped + // The following generates a complete list of all the props expected by Wrapped // Using `pick`, add the specifically requested shared context props - // Using `omit`, exclude all of the shared props + // Using `omit`, exclude all the shared props return provideContextAsDefaults( { ); const pickerItem = screen.getByLabelText('2 next item').nextElementSibling.nextElementSibling; // there is a dummy sibling by Spottable - // I chosen this and not getByText because in get by test you have to go 4 parentElements up + // I chose this and not getByText because in get by test you have to go 4 parentElements up const expectedAttribute = 'aria-valuetext'; const expectedValue = '2'; @@ -662,7 +662,7 @@ describe('Picker Specs', () => { ); const picker = screen.getByLabelText('2 press ok button to change the value').children.item(0); - // I chosen this and not getByText because in get by test you have to go 4 parentElements up + // I chose this and not getByText because in get by test you have to go 4 parentElements up const expectedAttribute = 'aria-hidden'; const expectedValue = 'true'; @@ -680,7 +680,7 @@ describe('Picker Specs', () => { ); const picker = screen.getByLabelText('2 press ok button to change the value').children.item(0); - // I chosen this and not getByText because in get by test you have to go 4 parentElements up + // I chose this and not getByText because in get by test you have to go 4 parentElements up fireEvent.focus(picker); diff --git a/internal/WindowEventable/WindowEventable.js b/internal/WindowEventable/WindowEventable.js index 347185548e..1a5dc5a3d8 100644 --- a/internal/WindowEventable/WindowEventable.js +++ b/internal/WindowEventable/WindowEventable.js @@ -26,7 +26,7 @@ const defaultConfig = { globalNode: 'window' }; -// In config, extract all of the config stuff we know about. Everything else is an event. +// In config, extract all the config stuff we know about. Everything else is an event. const WindowEventable = hoc(defaultConfig, ({globalNode, ...events}, Wrapped) => { return class extends Component { @@ -44,7 +44,7 @@ const WindowEventable = hoc(defaultConfig, ({globalNode, ...events}, Wrapped) => this.events = {}; for (let [evName, fn] of Object.entries(events)) { - // Tailored event names (convert from react style to browser style naming) + // Tailored event names (convert from React style to browser style naming) if (evName.indexOf('on') === 0) evName = evName.substr(2).toLowerCase(); if (typeof fn === 'function') { diff --git a/samples/event-logger/src/components/InputBoard/Filter/Filter.js b/samples/event-logger/src/components/InputBoard/Filter/Filter.js index 9992c51acf..040bb9bccc 100644 --- a/samples/event-logger/src/components/InputBoard/Filter/Filter.js +++ b/samples/event-logger/src/components/InputBoard/Filter/Filter.js @@ -43,7 +43,7 @@ const Filter = () => { }, [onIsSyntheticEventOn]); const - timergroup = ['3 sec', '5 sec', '10 sec'], + timerGroup = ['3 sec', '5 sec', '10 sec'], eventItems = eventCategory.map((e, i) => { const handler = handleEventCategory(i); return ( @@ -109,7 +109,7 @@ const Filter = () => { title="Timer" value={timerIndex} > - {timergroup} + {timerGroup} diff --git a/samples/event-logger/src/components/InputBoard/InputBoard.js b/samples/event-logger/src/components/InputBoard/InputBoard.js index 8d1e054099..ec0555172e 100644 --- a/samples/event-logger/src/components/InputBoard/InputBoard.js +++ b/samples/event-logger/src/components/InputBoard/InputBoard.js @@ -103,7 +103,7 @@ const InputBoard = ({className}) => { const [showFilter, setShowFilter] = useState(true); const sendEventLog = useCallback((ev, isDOMElement, eventObject, isCapturing) => { - const timergroup = [3000, 5000, 10000]; + const timerGroup = [3000, 5000, 10000]; if (eventLogsRef.current && eventLogsRef.current.length > 0) { const lastLog = eventLogsRef.current[eventLogsRef.current.length - 1]; @@ -117,7 +117,7 @@ const InputBoard = ({className}) => { beforeTimeoutId.current = eventLogsRef.current[index].timeoutId; window.clearTimeout(eventLogsRef.current[index].timeoutId); - const newTimeoutId = window.setTimeout(() => onRemoveEventLog(ev.type, isDOMElement, isCapturing), timergroup[timerIndexRef.current]); + const newTimeoutId = window.setTimeout(() => onRemoveEventLog(ev.type, isDOMElement, isCapturing), timerGroup[timerIndexRef.current]); onUpdateEventLog(eventObject, eventLogsRef.current[index].timeoutId, newTimeoutId); return; @@ -136,7 +136,7 @@ const InputBoard = ({className}) => { const timeoutId = window.setTimeout( () => onRemoveEventLog(ev.type, isDOMElement, isCapturing), - timergroup[timerIndexRef.current]); + timerGroup[timerIndexRef.current]); onAddEventLog(ev.type, eventObject, isDOMElement, isCapturing, timeoutId); }, [onAddEventLog, onRemoveEventLog, onUpdateEventLog]); diff --git a/samples/perf-virtuallist-native-with-different-item-size/src/views/VerticalDifferentHeightItemList.js b/samples/perf-virtuallist-native-with-different-item-size/src/views/VerticalDifferentHeightItemList.js index 69885df4d9..cba9dd0ee2 100644 --- a/samples/perf-virtuallist-native-with-different-item-size/src/views/VerticalDifferentHeightItemList.js +++ b/samples/perf-virtuallist-native-with-different-item-size/src/views/VerticalDifferentHeightItemList.js @@ -28,7 +28,7 @@ const itemStyleDefault = { lineHeight }; -const DifferenctHeightItem = ({index, items, style: itemStyleFromList, ...rest}) => { +const DifferentHeightItem = ({index, items, style: itemStyleFromList, ...rest}) => { const {title: children, height} = items[index], itemStyle = {...itemStyleDefault, ...itemStyleFromList, height}; @@ -39,7 +39,7 @@ const DifferenctHeightItem = ({index, items, style: itemStyleFromList, ...rest}) ); }; -DifferenctHeightItem.propTypes = { +DifferentHeightItem.propTypes = { index: PropTypes.number, items: PropTypes.array }; @@ -68,7 +68,7 @@ const VerticalDifferentHeightItemList = (props) => { }, []); const renderItem = useCallback((renderProps) => { - return ; + return ; }, []); return ( diff --git a/samples/qa-a11y/src/App/App.module.less b/samples/qa-a11y/src/App/App.module.less index 8d60e061be..ba27bd2dbe 100644 --- a/samples/qa-a11y/src/App/App.module.less +++ b/samples/qa-a11y/src/App/App.module.less @@ -60,7 +60,7 @@ h2 { .debug.aria, .debug.aria + div { // If the position of the `div` element including the following aria attributes, is `static`. - // Then the before and after psudo element will position incorrectly. + // Then the before and after pseudo element will position incorrectly. [aria-label], [aria-valuetext], [aria-checked], @@ -74,7 +74,7 @@ h2 { } } - // The before and after psudo element position and look + // The before and after pseudo element position and look [aria-label]::after, [aria-valuetext]::after, [aria-checked]::after, @@ -137,7 +137,7 @@ h2 { color: black; } - // The before and after psudo element content + // The before and after pseudo element content [role]::before { content: "Role:\0020" attr(role); diff --git a/samples/qa-a11y/src/components/Section.js b/samples/qa-a11y/src/components/Section.js index 2794e2f030..8cca109007 100644 --- a/samples/qa-a11y/src/components/Section.js +++ b/samples/qa-a11y/src/components/Section.js @@ -12,7 +12,7 @@ import css from './Section.module.less'; // // Accepts a title for the section, and children components. // Each child component should have an "alt" attribute that will be used as the title. -// Typically this is a 1-2 word "state" description of what the child is representing, +// Typically, this is a 1-2 word "state" description of what the child is representing, // like "Normal", "Selected", or "With Icon" // // This is a cell, so `size` prop may be supplied as well, to fit multiple on one line diff --git a/samples/qa-a11y/src/views/VirtualGridList.module.less b/samples/qa-a11y/src/views/VirtualGridList.module.less index 201b1f50d5..f6258dec85 100644 --- a/samples/qa-a11y/src/views/VirtualGridList.module.less +++ b/samples/qa-a11y/src/views/VirtualGridList.module.less @@ -1,4 +1,4 @@ -// VitualGridList.module.less +// VirtualGridList.module.less // @import '~@enact/sandstone/styles/mixins.less'; diff --git a/samples/qa-i18n/src/App/attachErrorHandler.js b/samples/qa-i18n/src/App/attachErrorHandler.js index 357d9a9d8e..e30d3586ae 100644 --- a/samples/qa-i18n/src/App/attachErrorHandler.js +++ b/samples/qa-i18n/src/App/attachErrorHandler.js @@ -7,7 +7,7 @@ const handleError = (ev) => { let stack = ev.error && ev.error.stack || null; if (stack && stack.length > 512) { - // JSON must be limitted to 1024 characters so we truncate the stack to 512 for safety + // JSON must be limited to 1024 characters so we truncate the stack to 512 for safety stack = ev.error.stack.substring(0, 512); } diff --git a/samples/qa-virtualgridlistnative-preserve-focus/README.md b/samples/qa-virtualgridlistnative-preserve-focus/README.md index ceb5c5aefa..2535b247d9 100644 --- a/samples/qa-virtualgridlistnative-preserve-focus/README.md +++ b/samples/qa-virtualgridlistnative-preserve-focus/README.md @@ -2,7 +2,7 @@ Run `npm install` then `npm run serve` to have the app running on [http://localhost:8080](http://localhost:8080), where you can view it in your browser. -To test focus preservere in VirtualGridListNative, you may use this sample. +To test focus preserve in VirtualGridListNative, you may use this sample. --- diff --git a/samples/sampler/src/ThemeEnvironment/ThemeEnvironment.module.less b/samples/sampler/src/ThemeEnvironment/ThemeEnvironment.module.less index 80f2030968..a164430d8e 100644 --- a/samples/sampler/src/ThemeEnvironment/ThemeEnvironment.module.less +++ b/samples/sampler/src/ThemeEnvironment/ThemeEnvironment.module.less @@ -1,4 +1,4 @@ -// ThemeEnvrionment.module.less +// ThemeEnvironment.module.less // @import "~@enact/sandstone/styles/variables.less"; @import "~@enact/sandstone/styles/mixins.less"; @@ -51,7 +51,7 @@ body { // overflow: hidden; // } -// Storybook injects the info overlay within the ThemeEnvrionment panel itself, so we need to +// Storybook injects the info overlay within the ThemeEnvironment panel itself, so we need to // assign the z-index of `section` to be at the same level as the X button to allow the info overlay // to overlap it (hiding the X while the info overlay is open). // diff --git a/samples/sampler/stories/default/Button.js b/samples/sampler/stories/default/Button.js index 909364ece0..432bb3dd78 100644 --- a/samples/sampler/stories/default/Button.js +++ b/samples/sampler/stories/default/Button.js @@ -7,7 +7,7 @@ import {Fragment} from 'react'; import iconNames from '../helper/icons'; -// Button's prop `minWidth` defaults to true and we only want to show `minWidth={false}` in the JSX. In order to hide `minWidth` when `true`, we use the normal storybook boolean control and return `void 0` when `true`. +// Button's prop `minWidth` defaults to true, and we only want to show `minWidth={false}` in the JSX. In order to hide `minWidth` when `true`, we use the normal storybook boolean control and return `void 0` when `true`. Button.displayName = 'Button'; const Config = mergeComponentMetadata('Button', UIButtonBase, UIButton, ButtonBase, Button); diff --git a/samples/sampler/stories/default/Image.js b/samples/sampler/stories/default/Image.js index ebe23e20cf..e34e4cdb64 100644 --- a/samples/sampler/stories/default/Image.js +++ b/samples/sampler/stories/default/Image.js @@ -45,7 +45,7 @@ export const _Image = (args) => ( fontSize: '32px' }} > - Image Boundry + Image Boundary ); diff --git a/samples/sampler/stories/default/Panel.js b/samples/sampler/stories/default/Panel.js index 09ec2c6eed..573065b95a 100644 --- a/samples/sampler/stories/default/Panel.js +++ b/samples/sampler/stories/default/Panel.js @@ -88,7 +88,7 @@ export const PanelsPanel = (args) => ( { const customizedConfig = Object.assign({}, Config); // Shallow copy this fn into a normal object customizedConfig.defaultProps = Object.assign( - {}, // Fresh new defaltProps object + {}, // Fresh new defaultProps object clone(Config.defaultProps), // Deep copy the defaultProps from our object into a fresh defaultProps object of our shallow copy (preserving all other props as their original references) { backButtonBackgroundOpacity: 'opaque', diff --git a/samples/sampler/stories/qa/components/KitchenSinkSection/KitchenSinkSection.js b/samples/sampler/stories/qa/components/KitchenSinkSection/KitchenSinkSection.js index 2bfe91fb78..1cd49d5b42 100644 --- a/samples/sampler/stories/qa/components/KitchenSinkSection/KitchenSinkSection.js +++ b/samples/sampler/stories/qa/components/KitchenSinkSection/KitchenSinkSection.js @@ -12,7 +12,7 @@ import css from './KitchenSinkSection.module.less'; // // Accepts a title for the section, and children components. // Each child component should have an "alt" attribute that will be used as the title. -// Typically this is a 1-2 word "state" description of what the child is representing, +// Typically, this is a 1-2 word "state" description of what the child is representing, // like "Normal", "Selected", or "With Icon" // // This is a cell, so `size` prop may be supplied as well, to fit multiple on one line diff --git a/tests/screenshot/apps/Sandstone-View.js b/tests/screenshot/apps/Sandstone-View.js index 276baeb465..a31d69ceec 100644 --- a/tests/screenshot/apps/Sandstone-View.js +++ b/tests/screenshot/apps/Sandstone-View.js @@ -45,7 +45,7 @@ function prepareTest (componentName, testId) { const componentMetadata = components[componentName][testId]; let component; - // If test wants focus, set mode to 5-way so auto-focus works + // If test wants focus, set mode to 5-way so autofocus works if (componentMetadata.focus) { spotlight.setPointerMode(false); } @@ -150,7 +150,7 @@ const ExportedApp = (props) => { locale = components[props.component][props.testId].locale; textSize = components[props.component][props.testId].textSize; focusRing = components[props.component][props.testId].focusRing; - // If focus enabled by test, use auto-focus to set an initial focus + // If focus enabled by test, use autofocus to set an initial focus noAutoFocus = !components[props.component][props.testId].focus; // Test can override values from the test runner diff --git a/tests/screenshot/apps/components/Button.js b/tests/screenshot/apps/components/Button.js index 1e3e8ce964..9414d93da5 100644 --- a/tests/screenshot/apps/components/Button.js +++ b/tests/screenshot/apps/components/Button.js @@ -30,7 +30,7 @@ const ButtonTests = [ // end [QWTC-1837] // iconPosition = before (Default) + small (default) + large - // Leaving size small here as example but it is not required since it is the default. + // Leaving size small here as example, but it is not required since it is the default. , , // iconPosition = before (Default) + icon + iconPosition + small (default) + large @@ -162,7 +162,7 @@ const ButtonTests = [ , // iconPosition = before (Default) + small (default) + large - // Leaving size small here as example but it is not required since it is the default. + // Leaving size small here as example, but it is not required since it is the default. , , // iconPosition = before (Default) + icon + iconPosition + small (default) + large @@ -315,7 +315,7 @@ const ButtonTests = [ , // iconPosition = before (Default) + small (default) + large - // Leaving size small here as example but it is not required since it is the default. + // Leaving size small here as example, but it is not required since it is the default. , , // Icon only - iconPosition = before (Default) + icon + iconPosition + small (default) + large diff --git a/tests/screenshot/apps/components/Icon.js b/tests/screenshot/apps/components/Icon.js index 1e5afbca1e..46469e83c4 100644 --- a/tests/screenshot/apps/components/Icon.js +++ b/tests/screenshot/apps/components/Icon.js @@ -8,7 +8,7 @@ const IconTests = [ minus, minus, minus, - 💣, // [QWTC-2251] testing 'custom-icon' using unicode character + 💣, // [QWTC-2251] testing 'custom-icon' using Unicode character 💣, 💣, 💣, @@ -76,7 +76,7 @@ const IconTests = [ { locale: 'ar-SA', - component: 💣 // testing 'custom-icon' using unicode character + component: 💣 // testing 'custom-icon' using Unicode character }, { locale: 'ar-SA', diff --git a/tests/screenshot/apps/components/InputField.js b/tests/screenshot/apps/components/InputField.js index 013b6636ad..be1592b652 100644 --- a/tests/screenshot/apps/components/InputField.js +++ b/tests/screenshot/apps/components/InputField.js @@ -116,7 +116,7 @@ const InputFieldTests = [ component: }, - // InputField field of type 'number'should be empty with letters as input + // InputField field of type 'number' should be empty with letters as input { locale: 'ar-SA', component: diff --git a/tests/screenshot/apps/components/Picker.js b/tests/screenshot/apps/components/Picker.js index a1b4fc7ee3..8a92216e8c 100644 --- a/tests/screenshot/apps/components/Picker.js +++ b/tests/screenshot/apps/components/Picker.js @@ -52,7 +52,7 @@ const PickerTests = [ }, // end of [QWTC-2214] - // tallCharacters: Change 'width', 'wrap', 'joined', 'changedBy', 'noAnimation', 'disabled', 'incrementIcon', decrementIcon' dynamically - [QWTC-2215] + // tallCharacters: Change 'width', 'wrap', 'joined', 'changedBy', 'noAnimation', 'disabled', 'incrementIcon', 'decrementIcon' dynamically - [QWTC-2215] // Marked part automated due to Marquee and Click actions // Step 2: Tall characters display fully and are not truncated. {pickerList.tall}, @@ -133,7 +133,7 @@ const PickerTests = [ {pickerList.oneAirport}, // end of [QWTC-2212] - // Change 'width', 'wrap', 'joined', 'noAnimation', 'disabled', 'incrementIcon', decrementIcon' dynamically - Vertical Picker - [QWTC-2213] + // Change 'width', 'wrap', 'joined', 'noAnimation', 'disabled', 'incrementIcon', 'decrementIcon' dynamically - Vertical Picker - [QWTC-2213] , // step 3: Picker displays vertically. {pickerList.vegetables}, @@ -232,7 +232,7 @@ const PickerTests = [ component: {pickerList.oneAirport} }, - // tallCharacters: Change 'width', 'wrap', 'joined', 'changedBy', 'noAnimation', 'disabled', 'incrementIcon', decrementIcon' dynamically - [QWTC-2215] + // tallCharacters: Change 'width', 'wrap', 'joined', 'changedBy', 'noAnimation', 'disabled', 'incrementIcon', 'decrementIcon' dynamically - [QWTC-2215] // Marked part automated due to Marquee and Click actions // Step 2: Tall characters display fully and are not truncated. { @@ -442,7 +442,7 @@ const PickerTests = [ }, // end of [QWTC-2212] - // Change 'width', 'wrap', 'joined', 'noAnimation', 'disabled', 'incrementIcon', decrementIcon' dynamically - Vertical Picker - [QWTC-2213] + // Change 'width', 'wrap', 'joined', 'noAnimation', 'disabled', 'incrementIcon', 'decrementIcon' dynamically - Vertical Picker - [QWTC-2213] { locale: 'ar-SA', component: diff --git a/tests/screenshot/apps/components/RangePicker.js b/tests/screenshot/apps/components/RangePicker.js index 386ee364df..c37c666085 100644 --- a/tests/screenshot/apps/components/RangePicker.js +++ b/tests/screenshot/apps/components/RangePicker.js @@ -28,7 +28,7 @@ const RangePickerTests = [ , , - // 'orientation" changed to 'vertical' and 'width' changed to 'large' - [QWTC-2143] + // 'orientation' changed to 'vertical' and 'width' changed to 'large' - [QWTC-2143] , , , @@ -214,7 +214,7 @@ const RangePickerTests = [ component: }, - // Start of [QWTC-2143] - 'orientation" changed to 'vertical' and 'width' changed to 'large' + // Start of [QWTC-2143] - 'orientation' changed to 'vertical' and 'width' changed to 'large' { locale: 'ar-SA', component: diff --git a/tests/ui/apps/Slider/Slider-View.js b/tests/ui/apps/Slider/Slider-View.js index 7657c2c24a..6080447936 100644 --- a/tests/ui/apps/Slider/Slider-View.js +++ b/tests/ui/apps/Slider/Slider-View.js @@ -11,7 +11,7 @@ const app = (props) =>
Slider default - Slider activate on slect + Slider activate on select Slider custom progressAnchor diff --git a/tests/ui/specs/ContextualMenuDecorator/ContextualMenuDecorator-specs.js b/tests/ui/specs/ContextualMenuDecorator/ContextualMenuDecorator-specs.js index 8aa1b35f60..ca34c3b4eb 100644 --- a/tests/ui/specs/ContextualMenuDecorator/ContextualMenuDecorator-specs.js +++ b/tests/ui/specs/ContextualMenuDecorator/ContextualMenuDecorator-specs.js @@ -50,7 +50,7 @@ describe('ContextualMenuDecorator', function () { }); it('should not dismiss the menu on 5-way up from first menu item - [QWTC-1915]', async function () { - // The *Contextual Button* menu does not close. Spotlight is on on same item (verify step 6) + // The *Contextual Button* menu does not close. Spotlight is on same item (verify step 6) await Page.spotlightUp(); expect(await menu2.isMenuExist).toBe(true); expect(await menu2.item(0).isFocused()).toBe(true); diff --git a/tests/ui/specs/ContextualPopupDecorator/ContextualPopupDecorator-specs.js b/tests/ui/specs/ContextualPopupDecorator/ContextualPopupDecorator-specs.js index ac75c7dc78..5ef206baca 100644 --- a/tests/ui/specs/ContextualPopupDecorator/ContextualPopupDecorator-specs.js +++ b/tests/ui/specs/ContextualPopupDecorator/ContextualPopupDecorator-specs.js @@ -71,7 +71,7 @@ describe('ContextualPopupDecorator', function () { expect(await button2.self.isFocused()).toBe(false); }); - it('should close the popup and should have Spotlight on button on auto dismiss the ContexualPopup2', async function () { + it('should close the popup and should have Spotlight on button on auto dismiss the ContextualPopup2', async function () { let popupButton = $('#popupButton2'); await button2.focus(); @@ -83,7 +83,7 @@ describe('ContextualPopupDecorator', function () { expect(await button2.self.isFocused()).toBe(true); }); - it('should not close the popup and should not have Spotlight on button on auto dismiss the ContexualPopup3', async function () { + it('should not close the popup and should not have Spotlight on button on auto dismiss the ContextualPopup3', async function () { let popupButton = $('#popupButton3'); await button3.focus(); diff --git a/tests/ui/specs/Item/Item-specs.js b/tests/ui/specs/Item/Item-specs.js index bd44de234f..70ff259cb9 100644 --- a/tests/ui/specs/Item/Item-specs.js +++ b/tests/ui/specs/Item/Item-specs.js @@ -49,7 +49,7 @@ describe('Item', function () { }); describe('pointer', function () { - // Step 3 - Focus on the Item title. In sampler, the item has no label. Here we focusing on an item with a label. + // Step 3 - Focus on the Item title. In sampler, the item has no label. Here we focus on an item with a label. it('should focus the third item when hovered - [QWTC-1820]', async function () { await item3WithLabel.hover(); expect(await item3WithLabel.self.isFocused()).toBe(true); diff --git a/tests/ui/specs/Popup/Popup-specs.js b/tests/ui/specs/Popup/Popup-specs.js index 05e5f24508..64ad801908 100644 --- a/tests/ui/specs/Popup/Popup-specs.js +++ b/tests/ui/specs/Popup/Popup-specs.js @@ -755,7 +755,7 @@ describe('Popup', function () { // Check spotlight is NOT on buttons outside popup (verify step 4) expect(await popup.buttonOK.isFocused()).toBe(true); - // Hover the button 'spotlightRestrict - self-only' outside of the popup (step 6) + // Hover the button 'spotlightRestrict - self-only' outside the popup (step 6) await Page.spotlightSelect(); // wait for popup closes await Page.delay(1000); diff --git a/tests/ui/specs/RangePicker/RangePicker-RTL-specs.js b/tests/ui/specs/RangePicker/RangePicker-RTL-specs.js index 608611eb7f..a1fd1e5998 100644 --- a/tests/ui/specs/RangePicker/RangePicker-RTL-specs.js +++ b/tests/ui/specs/RangePicker/RangePicker-RTL-specs.js @@ -26,7 +26,7 @@ describe('RangePicker RTL', function () { }); it('should decrement/increment in ar-SA [QWTC-2145]', async function () { - // Check that the disabled pciker's position is left from defaultPicker.(in case LTR, it is right from defaultPicker.) + // Check that the disabled picker's position is left from defaultPicker.(in case LTR, it is right from defaultPicker.) await Page.spotlightLeft(); await Page.spotlightLeft(); expect(await rangePickerDisabled.incrementer(rangePickerDisabled.self).isFocused()).toBe(true); diff --git a/tests/ui/specs/Scroller/ListOfThings/Scroller-QWTC-2086-specs.js b/tests/ui/specs/Scroller/ListOfThings/Scroller-QWTC-2086-specs.js index 03e8188b62..aaeb38ef30 100644 --- a/tests/ui/specs/Scroller/ListOfThings/Scroller-QWTC-2086-specs.js +++ b/tests/ui/specs/Scroller/ListOfThings/Scroller-QWTC-2086-specs.js @@ -20,7 +20,7 @@ describe('Scroller List Of Things', function () { expectFocusedItem(0); // Step 4-2: 5-way Right. await ScrollerPage.spotlightRight(); - // Step 4 Verify: Spotlight is on the verticalScrollbar Scroll Thumb.. + // Step 4 Verify: Spotlight is on the verticalScrollbar Scroll Thumb. expect(await ScrollerPage.verticalScrollThumb.isFocused()).toBe(true); // Step 5-1: Knobs > Scroller > spotlightDisabled > check diff --git a/tests/ui/specs/Scroller/ListOfThings/Scroller-QWTC-2134-specs.js b/tests/ui/specs/Scroller/ListOfThings/Scroller-QWTC-2134-specs.js index 1ae9053a43..97b6bebe35 100644 --- a/tests/ui/specs/Scroller/ListOfThings/Scroller-QWTC-2134-specs.js +++ b/tests/ui/specs/Scroller/ListOfThings/Scroller-QWTC-2134-specs.js @@ -21,7 +21,7 @@ describe('Scroller List Of Things', function () { // Step 4-3: Press 5-way Up. await ScrollerPage.spotlightUp(); - // Step 4 Verify: Spotlight is on the Close (x) button. (x) button is replaced top botton in this view. + // Step 4 Verify: Spotlight is on the Close (x) button. (x) button is replaced top bottom in this view. expect(await ScrollerPage.buttonTop.isFocused()).toBe(true); }); }); diff --git a/tests/ui/specs/TabLayout/ControlledIndex/TabLayoutControlledIndex-specs.js b/tests/ui/specs/TabLayout/ControlledIndex/TabLayoutControlledIndex-specs.js index b2c13d539a..a34cd7950b 100644 --- a/tests/ui/specs/TabLayout/ControlledIndex/TabLayoutControlledIndex-specs.js +++ b/tests/ui/specs/TabLayout/ControlledIndex/TabLayoutControlledIndex-specs.js @@ -43,7 +43,7 @@ describe('TabLayout', function () { await $$('.TabLayout_TabGroup_tab')[0].click(); // Step 7-1 Verify: Icons expand to tabs on the left side. expect(await Page.tabLayout.isCollapsed).toBe(false); - // Step 7-2: Click on the Change to 2rd tab button. + // Step 7-2: Click on the Change to 2nd tab button. await $('#button1').click(); await Page.delay(500); // Step 7-2 Verify: Tool icon with Button tab are selected. @@ -83,7 +83,7 @@ describe('TabLayout', function () { // Step 12-1: Click on the home icon. await $$('.TabLayout_TabGroup_tab')[0].click(); - // Step 12-2: Click on the Change to 2rd tab. + // Step 12-2: Click on the Change to 2nd tab. await $('#button1').click(); await Page.delay(500); // Step 12-1 Verify: Content of second navigation tab displays. diff --git a/tests/ui/specs/VideoPlayer/VideoPlayer-specs.js b/tests/ui/specs/VideoPlayer/VideoPlayer-specs.js index 158da1418d..d1edbe7a00 100644 --- a/tests/ui/specs/VideoPlayer/VideoPlayer-specs.js +++ b/tests/ui/specs/VideoPlayer/VideoPlayer-specs.js @@ -119,7 +119,7 @@ describe('VideoPlayer', function () { } expect(await videoPlayerDefault.playButton.isFocused()).toBe(true); - // Setp 4-2: 5-way Down again. + // Step 4-2: 5-way Down again. // Step 4-3: 5-way Select. // Step 4-4: 5-way Up quickly. await Page.spotlightDown(); diff --git a/tests/ui/specs/VirtualList/VirtualGridList/qa-VirtualGridList/VirtualGridList-QWTC-2301-specs.js b/tests/ui/specs/VirtualList/VirtualGridList/qa-VirtualGridList/VirtualGridList-QWTC-2301-specs.js index 1271f160d1..4b3a7fd4da 100644 --- a/tests/ui/specs/VirtualList/VirtualGridList/qa-VirtualGridList/VirtualGridList-QWTC-2301-specs.js +++ b/tests/ui/specs/VirtualList/VirtualGridList/qa-VirtualGridList/VirtualGridList-QWTC-2301-specs.js @@ -20,14 +20,14 @@ describe('qa-VirtualGridList', function () { await Page.spotlightDown(); await Page.delay(200); } - // Step 3-1 Verify: Spotlight is on the 101th item with 'Image 100'. + // Step 3-1 Verify: Spotlight is on the 101st item with 'Image 100'. await expectFocusedItem(100); // Wait for scroll animation await Page.delay(300); expect(await Page.scrollThumbPosition()).toBe('1'); - // Step 3-2 Verify: The 101th item with 'Image 100' sticks to the bottom in the list. + // Step 3-2 Verify: The 101st item with 'Image 100' sticks to the bottom in the list. await Page.spotlightDown(); expect(await Page.buttonBottom.isFocused()).toBe(true); }); diff --git a/tests/ui/specs/VirtualList/VirtualList/Samples/VirtualList-QWTC-2512-specs.js b/tests/ui/specs/VirtualList/VirtualList/Samples/VirtualList-QWTC-2512-specs.js index 98754c8d4c..fcad24a509 100644 --- a/tests/ui/specs/VirtualList/VirtualList/Samples/VirtualList-QWTC-2512-specs.js +++ b/tests/ui/specs/VirtualList/VirtualList/Samples/VirtualList-QWTC-2512-specs.js @@ -30,7 +30,7 @@ describe('VirtualList Samples', function () { // Wait for scroll animation await Page.delay(500); - // Verify Spotlight displays on the 21st item ('Itme 20'); + // Verify Spotlight displays on the 21st item ('Item 20'); await expectFocusedItem(Number((await Page.bottomVisibleItemId()).slice(4))); await expectFocusedItem(20); }); diff --git a/tests/ui/specs/VirtualList/VirtualList/VirtualList-specs.js b/tests/ui/specs/VirtualList/VirtualList/VirtualList-specs.js index b013549e7e..fcf01272de 100644 --- a/tests/ui/specs/VirtualList/VirtualList/VirtualList-specs.js +++ b/tests/ui/specs/VirtualList/VirtualList/VirtualList-specs.js @@ -102,7 +102,7 @@ describe('VirtualList', function () { await Page.fiveWayToItem(0); // Verify Step 6: 1. The list Scroll Down. 2. The Spotted item is placed on the Top. await expectFocusedItem(Number((await Page.topVisibleItemId()).slice(4)), 'focus Item 00'); - await expectFocusedItem(0, 'focus Item 00'); // to double check it is really top item + await expectFocusedItem(0, 'focus Item 00'); // to double-check it is really top item }); it('should Spotlight with Pointer wave [QWTC-2055]', async function () { diff --git a/tests/ui/specs/VirtualList/VirtualList/onScroll/VirtualList-QWTC-2057-specs.js b/tests/ui/specs/VirtualList/VirtualList/onScroll/VirtualList-QWTC-2057-specs.js index 9d306c191f..e632a83a6b 100644 --- a/tests/ui/specs/VirtualList/VirtualList/onScroll/VirtualList-QWTC-2057-specs.js +++ b/tests/ui/specs/VirtualList/VirtualList/onScroll/VirtualList-QWTC-2057-specs.js @@ -10,7 +10,7 @@ describe('Navigate Items with Channel Up/Down', function () { // UI Test's item default size is 100. // Step 4-1: Position the pointer on an item. await (await Page.item(0)).moveTo(); - // Step 4-2: 5-way Left to change into 5-way key mode. In this case, If press 5-way left, spotlight move to left button. Therefore 5-way left is replaced 5-way down. + // Step 4-2: 5-way Left to change into 5-way key mode. In this case, If press 5-way left, spotlight move to left button. Therefore, 5-way left is replaced 5-way down. await Page.spotlightDown(); await expectFocusedItem(1); // Step 5: Press Channel Down continuously until the bottom of the list displays. diff --git a/useScroll/useScroll.js b/useScroll/useScroll.js index 1ee2fb1100..1041644487 100644 --- a/useScroll/useScroll.js +++ b/useScroll/useScroll.js @@ -237,7 +237,7 @@ const useThemeScroll = (props, instances) => { } // oddly, Scroller manages scrollContainerHandle.current.bounds so if we don't update it here (it is also - // updated in calculateAndScrollTo but we might not have made it to that point), it will be + // updated in calculateAndScrollTo, but we might not have made it to that point), it will be // out of date when we land back in this method next time. scrollContainerHandle.current.bounds.scrollHeight = scrollContainerHandle.current.getScrollBounds().scrollHeight; }