-
Notifications
You must be signed in to change notification settings - Fork 79
Closed
Labels
4 - verifiedIssues that have been tested, confirmed as mitigated, and are ready to close.Issues that have been tested, confirmed as mitigated, and are ready to close.calcite-componentsIssues specific to the @esri/calcite-components package.Issues specific to the @esri/calcite-components package.estimate - 5A few days of work, definitely requires updates to tests.A few days of work, definitely requires updates to tests.p - mediumIssue is non core or affecting less that 60% of people using the libraryIssue is non core or affecting less that 60% of people using the libraryrefactorIssues tied to code that needs to be significantly reworked.Issues tied to code that needs to be significantly reworked.
Description
Description
In addition to the migration notes added by #10310, the following should be addressed as well:
- refactor
filter#filter()
to use the debounced function (see ad054fd) - revisit internal event emitting flag used in
list
(see ad054fd). - ✅ ensure all open comboboxes are rendered in the snapshot (Combobox: With Description Short Label And Content End Slot) – fix(combobox): fix sporadic change event emitted on initialization #10952
- ✅ fix GH pages content – chore: fix storybook GH pages deployment #10807
- ✅ simplify config (see https://github.com/Esri/calcite-design-system/pull/10482/files/57fefcd2747528235d727a17fd947f0c4b4f6a96..04af92c28583f05b22db71f111717bf68e24afb6#r1844236385) - build: simplify version placeholder replacement #11266
Stabilize
☑️ – no longer shows up
@esri/calcite-components:test: FAIL src/components/combobox/combobox.e2e.ts > calcite-combobox > reflects > %p scale
@esri/calcite-components:test: Error: vitest-fail-on-console > Expected test not to call console.error().
@esri/calcite-components:test: If the error is expected, test for it explicitly by mocking it out using:
@esri/calcite-components:test: vi.spyOn(console, 'error').mockImplementation(() => {})
@esri/calcite-components:test: and test that the warning occurs.
@esri/calcite-components:test:
@esri/calcite-components:test: "chevronDown16" is not a valid calcite-ui-icon name
@esri/calcite-components:test: Stack:
@esri/calcite-components:test: http://localhost:5173/src/components/icon/utils.ts:23:14
@esri/calcite-components:test:
@esri/calcite-components:test: at Console.l [as error] (file:///home/runner/work/calcite-design-system/calcite-design-system/node_modules/vitest-fail-on-console/src/index.ts:90:31)
@esri/calcite-components:test: at ae (file:///home/runner/work/calcite-design-system/calcite-design-system/node_modules/@arcgis/lumina-compiler/dist/puppeteerTesting/index.js:2:1926)
@esri/calcite-components:test: ❯ Console.l [as error] ../../node_modules/vitest-fail-on-console/src/index.ts:90
✅ – #10818
@esri/calcite-components:test:
@esri/calcite-components:test: FAIL src/components/segmented-control/segmented-control.e2e.ts > calcite-segmented-control > when multiple items are checked, last one wins
@esri/calcite-components:test: AssertionError: expected '1' to be '3' // Object.is equality
@esri/calcite-components:test:
@esri/calcite-components:test: Expected: "3"
@esri/calcite-components:test: Received: "1"
Additional fixes
Documentation Updates
- Add docs package export (fix(docs): add docs package export #11221)
- Use updated translation bundle paths to generate
translations.json
(fix(docs): use updated translation bundle paths to generatetranslations.json
#11219)
UI & Behavior Fixes
- Avoid showing scroller when
max-items
equals items in dropdown (fix(dropdown): avoid showing scroller when max-items equals items #11480) - Fix
has-children
logic in tree (fix(tree): fix has-children logic #11347) - Keep initial focus on newly selected item in stepper-item (fix(stepper-item): keep initial focus on newly selected item #11482)
- Fix error in color-picker caused by initial value with a different format (fix(color-picker): fix error caused by initial value with a different format #11351)
- Improve prop update times in combobox (fix(combobox): improve prop update times #11383)
- Make
readOnly
prop writable in text-area (fix(text-area): makereadOnly
prop writable #11218) - Fix sporadic change event emitted on initialization in combobox (fix(combobox): fix sporadic change event emitted on initialization #10952)
- Update internal state after custom value is added in combobox (fix(combobox): update internal state after custom value is added #11405)
- Drop relative-positioning in popover & tooltip to reduce clipping risk (fix(popover, tooltip): drop relative-positioning to reduce risk of clipping #11373)
- Toggle focus-trap on DOM connect/disconnect in dialog (fix(dialog): toggle focus-trap on DOM connect and disconnect #11429)
- Fix rounded corners in sheet when
displayMode="float"
(fix(sheet): fix rounded corners whendisplayMode="float"
#11086) - Stretch slotted actions only in accordion-item (fix(accordion-item): stretch slotted actions only #11082)
- Fix toggle logic in action-menu when reconnected (fix(action-menu): fix toggle logic when
action-menu
is reconnected #11139) - Fix selection indicator placement in RTL for list-item (fix(list-item): fix selection indicator placement in RTL #11288)
- Ensure change & select events are emitted after state updates in list & list-item (fix(list, list-item): ensure change and select events are emitted after state is updated #11079)
- Restore deactivating focus traps on outside click in dialog, modal, popover & sheet (fix(dialog, modal, popover, sheet): restore deactivating focus traps on outside click #11058)
- Prevent
beforeClose
from being invoked during initialization in dialog & panel (fix(dialog, panel): preventbeforeClose
from being invoked during initialization #11038) - Disable sort-handle when no move-to items are present and item count is one (fix(list, sort-handle): disable sort-handle when no move to items are present and item count is one #11073)
- Avoid using refs that have been nulled in input-date-picker, modal & tile-select (fix(input-date-picker, modal, tile-select): avoid using refs that have been nulled #11090)
- Fix year-select width logic in date-picker (fix(date-picker): fix year-select width logic #11071)
- Skip ref setup logic on component removal in popover & tooltip (fix(popover, tooltip): skip ref setup logic on component removal #11132)
- Fix initial item selection delay in combobox & input-time-zone (fix(combobox, input-time-zone): fix initial item selection delay #11326)
Performance & Core Fixes
- Fix initialization performance regression in combobox (perf(combobox): fix initialization performance regression #11265)
- Floating UI elements set
max-content
forinline-size
(fix: floating ui elements set max-content for inline-size #11095) - Prevent transpilation issues by using
this
to reference static props (fix: prevent transpilation issues by usingthis
to reference static props #11088) - Use
inert
on host elements instead ofaria-hidden
when hidden (fix: use inert on host elements instead of aria-hidden when host element is hidden #11056)
Proposed Advantages
Cleaner, simpler code.
Which Component
N/A
Relevant Info
No response
Calcite package
- @esri/calcite-components
- @esri/calcite-components-react
- @esri/calcite-design-tokens
- @esri/calcite-ui-icons
- @esri/eslint-plugin-calcite-components
Metadata
Metadata
Assignees
Labels
4 - verifiedIssues that have been tested, confirmed as mitigated, and are ready to close.Issues that have been tested, confirmed as mitigated, and are ready to close.calcite-componentsIssues specific to the @esri/calcite-components package.Issues specific to the @esri/calcite-components package.estimate - 5A few days of work, definitely requires updates to tests.A few days of work, definitely requires updates to tests.p - mediumIssue is non core or affecting less that 60% of people using the libraryIssue is non core or affecting less that 60% of people using the libraryrefactorIssues tied to code that needs to be significantly reworked.Issues tied to code that needs to be significantly reworked.