Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Table] Change label for batch action toolbar cancel action #3737

Closed
1 of 2 tasks
heinsj opened this issue Feb 27, 2023 · 1 comment · Fixed by #3780
Closed
1 of 2 tasks

[Table] Change label for batch action toolbar cancel action #3737

heinsj opened this issue Feb 27, 2023 · 1 comment · Fixed by #3780
Assignees
Labels
status: needs priority 📥 Enhancements without a `val #` label status: needs triage 🔍 type: enhancement 💡 New feature or request

Comments

@heinsj
Copy link

heinsj commented Feb 27, 2023

What package is this for?

  • React
  • Angular

Summary

In the table batch action toolbar, change the "Cancel" label to "Clear selections".
It was discussed and agreed on that "Cancel" does not represent the action that actually happens when clicked.

Here is how the updated action will look:
image

Specific timeline issues / requests

This request is coming from feedback of an interface that is in Beta and we hope to release in two months.

NB: The core contributors will try to work with your timeline, but it's not
guaranteed. The earlier you make a request in advance of a desired delivery
date, the better!

Available extra resources

What resources do you have to assist this effort? None

Carbon add-ons for Watson IoT is a collaborative system. We encourage teams to
build components and submit them for integration.

@heinsj heinsj added status: needs priority 📥 Enhancements without a `val #` label status: needs triage 🔍 type: enhancement 💡 New feature or request labels Feb 27, 2023
@heinsj
Copy link
Author

heinsj commented Mar 8, 2023

@davidicus Here is the story I created to update the batch action Cancel button text.

sls-ca pushed a commit that referenced this issue May 17, 2023
* feat(table): change batch cancel button label

#3737

* test(pagetitlebar): update tests

* test(table): update snapshots

* test(pagetitlebar): revert test updates

* test(table): update cypress snapshot

* test(pagetitlebar): update e2e test
sls-ca added a commit that referenced this issue Oct 11, 2023
* fix(testappyml): fetch from next

* v2.152.4-next.1

* fix(tablecard): update images

* fix(datetimepickerv2): show no tooltip when closed inline

* fix(header): handle server-side rendering  #3742

* v2.152.6-next.0

* fix(walkme): Defining the required Walkme global variable and function in a useEffect hook because they were not being set inside the script tag for some reason.

* fix(walkme): Running yarn format

* test(walkme): Adding some tests to make sure the global walkme_get_language() function was set.

* v2.152.6-next.1

* fix(proptypes): adding defaultDateFormatPattern to the PropType

* v2.153.0-next.0

* revert: revert the defaultDateFormatPattern to old format

* revert(barchartutils): reverting the tooltipDateFormatPattern format

* docs(tablecard): updating the tablecard docs to have different defaultDateFormatPattern

* fix(tablecardutils): formatting the file

* fix(idlelogoutconfirmationmodal): Assembling the `originHref` with the current url excluding the query parameters to fix an issue where query parameters may be added to the page without a page reload, which would cause the `useEffect` to restart because it has the logout and inactivityLogout URLs as dependencies.

* v2.153.0-next.1

* test(suiteheader): Fixing some unit tests.

* v2.153.0-next.2

* fix(suiteheader): Adding a null check in `surveyData` to avoid runtime errors if `surveyData` becomes null after in subsequent component re-renders

* fix(sidepanel): add tabindex and aria-hidden

* fix(sidepanel): add tabindex and aria-hidden

* feat(breadcrumb): add option to disable truncation

* test(breadcrumb): add e2e tests

* test(breadcrumb): update snapshots

* chore(breadcrumb): add story with truncation

#3700

* test(breadcrumb): update e2e test

* feat(barchartcard): add maximum data points input field

* test(barchartcard): add test for maximum data points field

* chore(cardeditor): update storybook

* chore(cardeditor): update snapshots

* chore(table): refactor pin column mixin

* chore(table): update snapshot

* chore(mixin): run prettier

* chore(table): remove additional classes for pin column in tbody

* feat(table): initial table drag and drop #3751

Initial drag and drop support for the table. Allows rows to be dragged over other rows. Only supports dragging a single row currently (the selection set is ignored).

* feat(table): add multi row dragging

If the dragged row is in the selected rows, then drag all selected rows.
Hide all drag handles if any selected rows are undraggable.

* feat(table): add option to pin header and footer

#3696

* docs(table): update docs for pinned header and footer

* chore(table): add story with pinned header and footer

* test(table): add e2e test for pinned header and footer

* chore(publicapi): update snapshot

* test(table): update snapshots

* test(table): update flaky snapshots

* feat(table): add table drag and drop tests

* v2.153.0-next.3

* feat(table): align row overlay in rtl

* feat(table): DnD can fail on macOS

activeDropRowIdRef.current was null if accessed after a state change.

* feat(table): correct hook return value docs

* feat(table): fix position of avatar, row overlay in rtl

* feat(table): safari misses drop sometimes

Safari can fire mousemove events with buttons:e before it fires the mouse up event. Need to delay that last mousemove to given mouseup a chance to fire first, otherwise the drop would be canceled. Happens if moving the mouse fast will dropping, which is easy to do.

Also found a test case that failed sometimes because a mouseup was missing in it, so it was in an invalid state for the next step.

* feat(table): handle event order problems on macOS

If the mouse goes outside the window and releases the button then the mouseup is missed and the drag is stuck. If we see a few moves without a mouse button down then that's probably the case. We don't do this on the first bad move since Safari and Firefox on macOS can fire a mousemove without buttons BEFORE a mouse up sometimes, which would cancel the drop too early. Waiting a few moves proves this really is stuck, not just an unexpected event order.

* fix(headeraction): close panel on other action click

#3755

* test(suiteheader): header action panel click

* fix(sidepanel): add tabindex for content elements

* fix(sidepanel): update snapshots

* feat(table): drag and drop with expandable rows

Drag and drop assumed all rows were in the top level `data` array. Now all child rows are consulted too. This required moving some logic into the row components.

Some CSS changes were needed to overcome the styles on expandable rows.

* feat(cardtoolbar): add max width to extra action overflow menu

#3733

* test(card): add test for extra action overflow menu

* chore(card): update storybook knobs

* chore(breadcrumb): update storybook know for truncation

* fix(datetimepickerv2): add onClear callback

* feat(table): hide row expando when no children

After a drag and drop, if a row has not children the expando icon would still show. The class to hide it was only applied to closed rows, but a row is open after dragging its last child. Add the class to hide it in that case so it disappears as soon as there are no children.

* feat(table): review fixes 1

- In StatefulTable use `callbackParent` for consistency and to protect against calling a null callback (not really a concern here, but doesn't hurt).
- Use the ${prefix} sass var instead of `bx`.
- Remove `td` element name from selector where not needed.

* feat(table): correct docs about dragging multiple rows

* fix(headeraction): close panel on click outside

* test(suiteheader): add test for click outside

* feat(table): support z-index on table for DnD, Graphite

Add a `zIndex` prop on the `Table`. This is needed if the caller places the `Table` in a modal dialog or similar with z-index > 0 and enables row drag and drop. The drag image needs to know the min z-index it can use to be drawn "over" the table.

* feat(table): improve drag and drop performance

When dragging, so make any React state updates, since that will rerender the whole table, which is slow once there are enough rows. Instead, just update the dom element styles (the avatar position and the "onDrop" row) directly.

* fix(header): add tooltip to action menus

#3763

* test(headeraction): update tests for tooltips

* test(header): update snapshots

* test(header): flaky snapshots

* fix(header): decrease border width for menu button focus

* chore(breadcrumb): add truncation login to css

* test(breadcrumb): update snapshots

* chore(header): remove onkeydown event for action menu

* test(header): update unit test

* test(header): update snapshots

* Update codeowner

* fix(filterheaderrow): display filter icon without input

* test(filterheaderrow): test filter icon without input

* fix(datetimepickerv2): new error message

* fix(datetimepickerv2): update snashots

* v2.153.0-next.4

* test(table): update snapshots

* test(table): update snapshot

* test(table): update snapshots

* test(table): update snapshot

* chore(table): run linters

* chore(table): update snapshot

* chore(table): update snapshots

* chore(table): update snapshots

* v2.153.0-next.5

* chore(tablebody): rename classnames import

* v2.153.0-next.6

* chore(tablebody): add conditional props for className

* test(table): update snapshots

* chore(table): update snapshots

* test(table): update snapshots

* test(table): update snapshots

* test(table): update snapshots

* v2.153.0-next.7

* test(table): update snapshots

* test(table): update snapshots

* v2.153.0-next.8

* v2.153.0-next.9

* test(table): update snapshots

* test(table): update snapshots

* fix(headeraction): handle click outside safari

* test(suiteheader): add tests for header menu

* fix(breadcrumb): add min width to item

* chore(repo): trigger github actions

* v2.153.0-next.10

* v2.153.0-next.11

* v2.153.0-next.12

* docs(table): update pinColumn prop

* chore(table): update snapshots

* chore(suiteheader): removed unnecessary test

* fix(datetimepickerv2): make sure locale is 2 letters

* fix(table): scrollbar shows when drag and drop starts

* fix(table): scrollbar shows when drag and drop starts

* fix(datetimepickerv2): update tests

* v2.153.0-next.13

* fix(datetimepickerv2): make sure changes only apply to single select

* fix(datetimepickerv2): update tests

* fix(datetimepickerv2): fix typos

* v2.153.0-next.14

* v2.153.0-next.15

* v2.153.0-next.16

* fix(datetimepickerv2): update placeholder text (#3778)

* v2.153.0-next.17

* Update SECURITY.md

* chore(codeowners): add amritha as codeowners for this piece (#3779)

* feat(table): update batch cancel button label (#3780)

* feat(table): change batch cancel button label

#3737

* test(pagetitlebar): update tests

* test(table): update snapshots

* test(pagetitlebar): revert test updates

* test(table): update cypress snapshot

* test(pagetitlebar): update e2e test

* v2.153.0-next.18

* fix(table): add column for DnD in the filter/manage columns headers (#3782)

* fix(table): add column for DnD  in the filter/manage columns headers

When drag and drop is enabled, rows get an extra leading column for the drag and drop handle. The base header gets and extra blank column to line up right. The Filter and Manage Columns headers were missing that extra column, so are one column less than the table body when dtag and drop was enabled, causing them to line up wrong. This just adds that DnD blank column to those header rows as well.

* fix(table): update header tests for DnD

* v2.153.0-next.19

* fix: allowing description outside tooltip when content is provided

* feat: added desc tooltip to current breadcrumb item for condensed header

* chore: page title bar tests

* chore: snapshots updated

* chore: page title bar e2e tests updated

* fix(datetimepickerv2): auto position inf loop (#3786)

* v2.153.0-next.20

* fix: removing record flag

* fix: adding record false flag

* chore: reverting changes in page title bar e2e tests

* v2.153.0-next.21

* fix(suiteheader): app switcher icon title (#3788)

* fix(suiteheader): disable browser native tooltip

* chore(suiteheader): update app switcher tooltip label

* test(suiteheader): update snapshots

* test(publicapi): update snapshot

* v2.153.0-next.22

* feat(repo): allow node 18 for consumers (#3792)

* feat(repo): allow node 18 for consumers

* chore(repo): setup node v16 for development

* v2.153.0-next.23

* fix(table): remove unnecessary space (#3797)

* v2.153.0-next.24

* fix(datetimeinput): accessibility violation in date time input (#3802)

#3800

* v2.153.0-next.25

* fix(table): select all checkbox state on initial render (#3801)

* fix(table): select all checkbox state on initial render

* test(table): select all checkbox state on initial render

* fix(tablecellrenderer): add min width for truncation (#3799)

* fix(tablecellrenderer): add min width for truncation

* chore(table): update cypress image snapshot

* feat(SuiteHeader): add unique id to action items (#3796)

* feat(suiteheader): add id to action items

* test(suiteheader): add test fir unique id for action items

* test(suiteheader): update snapshots

* feat(SuiteHeader): add support for on click callback for application name (#3795)

* feat(suiteheader): add onClick callback for application name

* docs(suiteheader): update docs

* test(suiteheader): add unit test for onClick callback

* test(suiteheader): update snapshots

* chore(suiteheader): add storybook example

* test(suiteheader): update snapshots

* v2.153.0-next.26

* feat(table) simple pagination (#3805)

* feat(table): display simple pagination if items exceed maxPages

* test(table): update pagination tests

* docs(table): update pagination docs

* chore(header): update snapshots

* chore(table): change simple pagination render condition

* v2.153.0-next.27

* 3804 suite header and header action fix accessibility scan violations (#3807)

* fix(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suitheader and headeraction): fix accessibility scan violations

#3804

* Update PageTitleBar.test.e2e.jsx

* fix(SuiteHeader): header action panel proptypes  (#3803)

* fix(suiteheader): proptypes for header action panel

* test(suiteheader): update snapshot

* v2.153.0-next.28

* fix(table): connect pagination to totalItems prop

* test(table): update pagination tests

* v2.153.0-next.29

* fix(table): change filter tag close button title

* fix(headeractionpanel): remove unnecessary aria-label

* fix(headeractionpanel): update tests

* fix(appswitcher): firefox disable focus

* fix(appswitcher): update snapshots

* fix(headeractionpanel): update e2e tests

* v2.153.0-next.30

* feat(SuiteHeader): add option to hide menu (#3815)

* feat(suiteheader): add prop to hide menu button

* test(suiteheader): add test to hide menu button

* docs(suiteheader): update props info

* chore(suiteheader): update storybook

* fix(table): add min width for pinned last column (#3816)

* v2.153.0-next.31

* feat(ValueCard): disable preview in edit mode (#3818)

* feat(valuecard): disable preview in edit mode

* test(valuecard): update unit test

* chore(valuecard): update isEditing story

* chore(valuecard): update snapshots

* v2.153.0-next.32

* fix: dashboard layout constant fix (#3821)

* v2.153.0-next.33

* fix(IdleTimer): Preventing the IdleTimer logic to start its timer if timeout is 0. (#3825)

* fix(IdleTimer): Preventing the IdleTimer logic to start its timer if timeout is 0.

* test(IdleTimer): Adding a unit tests to make sure that IdleTimer does not start a timer when timeout is zero.

---------

Co-authored-by: Marcelo Blechner <>

* v2.153.0-next.34

* feat: close header side nav on navigation (#3826)

* v2.153.0-next.35

---------

Co-authored-by: vaffonso <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: XUE YAN <[email protected]>
Co-authored-by: David Conner <[email protected]>
Co-authored-by: balint.lendvai <[email protected]>
Co-authored-by: Amritha raj herle <[email protected]>
Co-authored-by: Marcelo Blechner <>
Co-authored-by: Amritha raj herle <[email protected]>
Co-authored-by: Balint Lendvai <[email protected]>
Co-authored-by: Erzhan-Temir Mamyrov <[email protected]>
Co-authored-by: Jay Gillibrand <[email protected]>
Co-authored-by: jessieyan <[email protected]>
Co-authored-by: Jessie Yan <[email protected]>
Co-authored-by: Taylor Jones <[email protected]>
Co-authored-by: Erzhan-Temir Mamyrov <[email protected]>
Co-authored-by: Hector Davis <[email protected]>
Co-authored-by: abpaul1993 <[email protected]>
Co-authored-by: Kalyani Raut <[email protected]>
Co-authored-by: blechner <[email protected]>
sls-ca added a commit that referenced this issue Oct 11, 2023
* Mas8.11 release (#3828)

* fix(testappyml): fetch from next

* v2.152.4-next.1

* fix(tablecard): update images

* fix(datetimepickerv2): show no tooltip when closed inline

* fix(header): handle server-side rendering  #3742

* v2.152.6-next.0

* fix(walkme): Defining the required Walkme global variable and function in a useEffect hook because they were not being set inside the script tag for some reason.

* fix(walkme): Running yarn format

* test(walkme): Adding some tests to make sure the global walkme_get_language() function was set.

* v2.152.6-next.1

* fix(proptypes): adding defaultDateFormatPattern to the PropType

* v2.153.0-next.0

* revert: revert the defaultDateFormatPattern to old format

* revert(barchartutils): reverting the tooltipDateFormatPattern format

* docs(tablecard): updating the tablecard docs to have different defaultDateFormatPattern

* fix(tablecardutils): formatting the file

* fix(idlelogoutconfirmationmodal): Assembling the `originHref` with the current url excluding the query parameters to fix an issue where query parameters may be added to the page without a page reload, which would cause the `useEffect` to restart because it has the logout and inactivityLogout URLs as dependencies.

* v2.153.0-next.1

* test(suiteheader): Fixing some unit tests.

* v2.153.0-next.2

* fix(suiteheader): Adding a null check in `surveyData` to avoid runtime errors if `surveyData` becomes null after in subsequent component re-renders

* fix(sidepanel): add tabindex and aria-hidden

* fix(sidepanel): add tabindex and aria-hidden

* feat(breadcrumb): add option to disable truncation

* test(breadcrumb): add e2e tests

* test(breadcrumb): update snapshots

* chore(breadcrumb): add story with truncation

#3700

* test(breadcrumb): update e2e test

* feat(barchartcard): add maximum data points input field

* test(barchartcard): add test for maximum data points field

* chore(cardeditor): update storybook

* chore(cardeditor): update snapshots

* chore(table): refactor pin column mixin

* chore(table): update snapshot

* chore(mixin): run prettier

* chore(table): remove additional classes for pin column in tbody

* feat(table): initial table drag and drop #3751

Initial drag and drop support for the table. Allows rows to be dragged over other rows. Only supports dragging a single row currently (the selection set is ignored).

* feat(table): add multi row dragging

If the dragged row is in the selected rows, then drag all selected rows.
Hide all drag handles if any selected rows are undraggable.

* feat(table): add option to pin header and footer

#3696

* docs(table): update docs for pinned header and footer

* chore(table): add story with pinned header and footer

* test(table): add e2e test for pinned header and footer

* chore(publicapi): update snapshot

* test(table): update snapshots

* test(table): update flaky snapshots

* feat(table): add table drag and drop tests

* v2.153.0-next.3

* feat(table): align row overlay in rtl

* feat(table): DnD can fail on macOS

activeDropRowIdRef.current was null if accessed after a state change.

* feat(table): correct hook return value docs

* feat(table): fix position of avatar, row overlay in rtl

* feat(table): safari misses drop sometimes

Safari can fire mousemove events with buttons:e before it fires the mouse up event. Need to delay that last mousemove to given mouseup a chance to fire first, otherwise the drop would be canceled. Happens if moving the mouse fast will dropping, which is easy to do.

Also found a test case that failed sometimes because a mouseup was missing in it, so it was in an invalid state for the next step.

* feat(table): handle event order problems on macOS

If the mouse goes outside the window and releases the button then the mouseup is missed and the drag is stuck. If we see a few moves without a mouse button down then that's probably the case. We don't do this on the first bad move since Safari and Firefox on macOS can fire a mousemove without buttons BEFORE a mouse up sometimes, which would cancel the drop too early. Waiting a few moves proves this really is stuck, not just an unexpected event order.

* fix(headeraction): close panel on other action click

#3755

* test(suiteheader): header action panel click

* fix(sidepanel): add tabindex for content elements

* fix(sidepanel): update snapshots

* feat(table): drag and drop with expandable rows

Drag and drop assumed all rows were in the top level `data` array. Now all child rows are consulted too. This required moving some logic into the row components.

Some CSS changes were needed to overcome the styles on expandable rows.

* feat(cardtoolbar): add max width to extra action overflow menu

#3733

* test(card): add test for extra action overflow menu

* chore(card): update storybook knobs

* chore(breadcrumb): update storybook know for truncation

* fix(datetimepickerv2): add onClear callback

* feat(table): hide row expando when no children

After a drag and drop, if a row has not children the expando icon would still show. The class to hide it was only applied to closed rows, but a row is open after dragging its last child. Add the class to hide it in that case so it disappears as soon as there are no children.

* feat(table): review fixes 1

- In StatefulTable use `callbackParent` for consistency and to protect against calling a null callback (not really a concern here, but doesn't hurt).
- Use the ${prefix} sass var instead of `bx`.
- Remove `td` element name from selector where not needed.

* feat(table): correct docs about dragging multiple rows

* fix(headeraction): close panel on click outside

* test(suiteheader): add test for click outside

* feat(table): support z-index on table for DnD, Graphite

Add a `zIndex` prop on the `Table`. This is needed if the caller places the `Table` in a modal dialog or similar with z-index > 0 and enables row drag and drop. The drag image needs to know the min z-index it can use to be drawn "over" the table.

* feat(table): improve drag and drop performance

When dragging, so make any React state updates, since that will rerender the whole table, which is slow once there are enough rows. Instead, just update the dom element styles (the avatar position and the "onDrop" row) directly.

* fix(header): add tooltip to action menus

#3763

* test(headeraction): update tests for tooltips

* test(header): update snapshots

* test(header): flaky snapshots

* fix(header): decrease border width for menu button focus

* chore(breadcrumb): add truncation login to css

* test(breadcrumb): update snapshots

* chore(header): remove onkeydown event for action menu

* test(header): update unit test

* test(header): update snapshots

* Update codeowner

* fix(filterheaderrow): display filter icon without input

* test(filterheaderrow): test filter icon without input

* fix(datetimepickerv2): new error message

* fix(datetimepickerv2): update snashots

* v2.153.0-next.4

* test(table): update snapshots

* test(table): update snapshot

* test(table): update snapshots

* test(table): update snapshot

* chore(table): run linters

* chore(table): update snapshot

* chore(table): update snapshots

* chore(table): update snapshots

* v2.153.0-next.5

* chore(tablebody): rename classnames import

* v2.153.0-next.6

* chore(tablebody): add conditional props for className

* test(table): update snapshots

* chore(table): update snapshots

* test(table): update snapshots

* test(table): update snapshots

* test(table): update snapshots

* v2.153.0-next.7

* test(table): update snapshots

* test(table): update snapshots

* v2.153.0-next.8

* v2.153.0-next.9

* test(table): update snapshots

* test(table): update snapshots

* fix(headeraction): handle click outside safari

* test(suiteheader): add tests for header menu

* fix(breadcrumb): add min width to item

* chore(repo): trigger github actions

* v2.153.0-next.10

* v2.153.0-next.11

* v2.153.0-next.12

* docs(table): update pinColumn prop

* chore(table): update snapshots

* chore(suiteheader): removed unnecessary test

* fix(datetimepickerv2): make sure locale is 2 letters

* fix(table): scrollbar shows when drag and drop starts

* fix(table): scrollbar shows when drag and drop starts

* fix(datetimepickerv2): update tests

* v2.153.0-next.13

* fix(datetimepickerv2): make sure changes only apply to single select

* fix(datetimepickerv2): update tests

* fix(datetimepickerv2): fix typos

* v2.153.0-next.14

* v2.153.0-next.15

* v2.153.0-next.16

* fix(datetimepickerv2): update placeholder text (#3778)

* v2.153.0-next.17

* Update SECURITY.md

* chore(codeowners): add amritha as codeowners for this piece (#3779)

* feat(table): update batch cancel button label (#3780)

* feat(table): change batch cancel button label

#3737

* test(pagetitlebar): update tests

* test(table): update snapshots

* test(pagetitlebar): revert test updates

* test(table): update cypress snapshot

* test(pagetitlebar): update e2e test

* v2.153.0-next.18

* fix(table): add column for DnD in the filter/manage columns headers (#3782)

* fix(table): add column for DnD  in the filter/manage columns headers

When drag and drop is enabled, rows get an extra leading column for the drag and drop handle. The base header gets and extra blank column to line up right. The Filter and Manage Columns headers were missing that extra column, so are one column less than the table body when dtag and drop was enabled, causing them to line up wrong. This just adds that DnD blank column to those header rows as well.

* fix(table): update header tests for DnD

* v2.153.0-next.19

* fix: allowing description outside tooltip when content is provided

* feat: added desc tooltip to current breadcrumb item for condensed header

* chore: page title bar tests

* chore: snapshots updated

* chore: page title bar e2e tests updated

* fix(datetimepickerv2): auto position inf loop (#3786)

* v2.153.0-next.20

* fix: removing record flag

* fix: adding record false flag

* chore: reverting changes in page title bar e2e tests

* v2.153.0-next.21

* fix(suiteheader): app switcher icon title (#3788)

* fix(suiteheader): disable browser native tooltip

* chore(suiteheader): update app switcher tooltip label

* test(suiteheader): update snapshots

* test(publicapi): update snapshot

* v2.153.0-next.22

* feat(repo): allow node 18 for consumers (#3792)

* feat(repo): allow node 18 for consumers

* chore(repo): setup node v16 for development

* v2.153.0-next.23

* fix(table): remove unnecessary space (#3797)

* v2.153.0-next.24

* fix(datetimeinput): accessibility violation in date time input (#3802)

#3800

* v2.153.0-next.25

* fix(table): select all checkbox state on initial render (#3801)

* fix(table): select all checkbox state on initial render

* test(table): select all checkbox state on initial render

* fix(tablecellrenderer): add min width for truncation (#3799)

* fix(tablecellrenderer): add min width for truncation

* chore(table): update cypress image snapshot

* feat(SuiteHeader): add unique id to action items (#3796)

* feat(suiteheader): add id to action items

* test(suiteheader): add test fir unique id for action items

* test(suiteheader): update snapshots

* feat(SuiteHeader): add support for on click callback for application name (#3795)

* feat(suiteheader): add onClick callback for application name

* docs(suiteheader): update docs

* test(suiteheader): add unit test for onClick callback

* test(suiteheader): update snapshots

* chore(suiteheader): add storybook example

* test(suiteheader): update snapshots

* v2.153.0-next.26

* feat(table) simple pagination (#3805)

* feat(table): display simple pagination if items exceed maxPages

* test(table): update pagination tests

* docs(table): update pagination docs

* chore(header): update snapshots

* chore(table): change simple pagination render condition

* v2.153.0-next.27

* 3804 suite header and header action fix accessibility scan violations (#3807)

* fix(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suiteheader and headeraction): fix accessibility scan violations

#3804

* chore(suitheader and headeraction): fix accessibility scan violations

#3804

* Update PageTitleBar.test.e2e.jsx

* fix(SuiteHeader): header action panel proptypes  (#3803)

* fix(suiteheader): proptypes for header action panel

* test(suiteheader): update snapshot

* v2.153.0-next.28

* fix(table): connect pagination to totalItems prop

* test(table): update pagination tests

* v2.153.0-next.29

* fix(table): change filter tag close button title

* fix(headeractionpanel): remove unnecessary aria-label

* fix(headeractionpanel): update tests

* fix(appswitcher): firefox disable focus

* fix(appswitcher): update snapshots

* fix(headeractionpanel): update e2e tests

* v2.153.0-next.30

* feat(SuiteHeader): add option to hide menu (#3815)

* feat(suiteheader): add prop to hide menu button

* test(suiteheader): add test to hide menu button

* docs(suiteheader): update props info

* chore(suiteheader): update storybook

* fix(table): add min width for pinned last column (#3816)

* v2.153.0-next.31

* feat(ValueCard): disable preview in edit mode (#3818)

* feat(valuecard): disable preview in edit mode

* test(valuecard): update unit test

* chore(valuecard): update isEditing story

* chore(valuecard): update snapshots

* v2.153.0-next.32

* fix: dashboard layout constant fix (#3821)

* v2.153.0-next.33

* fix(IdleTimer): Preventing the IdleTimer logic to start its timer if timeout is 0. (#3825)

* fix(IdleTimer): Preventing the IdleTimer logic to start its timer if timeout is 0.

* test(IdleTimer): Adding a unit tests to make sure that IdleTimer does not start a timer when timeout is zero.

---------

Co-authored-by: Marcelo Blechner <>

* v2.153.0-next.34

* feat: close header side nav on navigation (#3826)

* v2.153.0-next.35

---------

Co-authored-by: vaffonso <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: XUE YAN <[email protected]>
Co-authored-by: David Conner <[email protected]>
Co-authored-by: balint.lendvai <[email protected]>
Co-authored-by: Amritha raj herle <[email protected]>
Co-authored-by: Marcelo Blechner <>
Co-authored-by: Amritha raj herle <[email protected]>
Co-authored-by: Balint Lendvai <[email protected]>
Co-authored-by: Erzhan-Temir Mamyrov <[email protected]>
Co-authored-by: Jay Gillibrand <[email protected]>
Co-authored-by: jessieyan <[email protected]>
Co-authored-by: Jessie Yan <[email protected]>
Co-authored-by: Taylor Jones <[email protected]>
Co-authored-by: Erzhan-Temir Mamyrov <[email protected]>
Co-authored-by: Hector Davis <[email protected]>
Co-authored-by: abpaul1993 <[email protected]>
Co-authored-by: Kalyani Raut <[email protected]>
Co-authored-by: blechner <[email protected]>

* v2.153.0

* chore: fix package.json for linux

---------

Co-authored-by: vaffonso <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Co-authored-by: carbon-bot <[email protected]>
Co-authored-by: XUE YAN <[email protected]>
Co-authored-by: David Conner <[email protected]>
Co-authored-by: balint.lendvai <[email protected]>
Co-authored-by: Amritha raj herle <[email protected]>
Co-authored-by: Amritha raj herle <[email protected]>
Co-authored-by: Balint Lendvai <[email protected]>
Co-authored-by: Erzhan-Temir Mamyrov <[email protected]>
Co-authored-by: Jay Gillibrand <[email protected]>
Co-authored-by: jessieyan <[email protected]>
Co-authored-by: Jessie Yan <[email protected]>
Co-authored-by: Taylor Jones <[email protected]>
Co-authored-by: Erzhan-Temir Mamyrov <[email protected]>
Co-authored-by: Hector Davis <[email protected]>
Co-authored-by: abpaul1993 <[email protected]>
Co-authored-by: Kalyani Raut <[email protected]>
Co-authored-by: blechner <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs priority 📥 Enhancements without a `val #` label status: needs triage 🔍 type: enhancement 💡 New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants