Skip to content

Commit

Permalink
chore: Ditch a few console logs from useActionBarModel that can clutt…
Browse files Browse the repository at this point in the history
…er up test logs (#2295)

I noticed some console logs when I was running a unit test with some components that relied on CK internally that I figured I could go ahead and declutter:

![Screenshot 2023-07-19 at 17 20 44](https://github.com/Workday/canvas-kit/assets/8914051/70df7f4c-91ca-4c7a-b97c-11bf22057d83)

Feel free to close this if you want to keep them around!

[category:Components]
  • Loading branch information
ahayes91 authored Jul 19, 2023
1 parent e05ca04 commit 53de16a
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions modules/react/action-bar/lib/useActionBarModel.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,6 @@ export const useActionBarModel = createModelHook({
let nonInteractiveIds = model.state.nonInteractiveIds;
const totalSize = model.state.items.length;

console.log(config.maximumVisible);

// Only show maximumVisible buttons
const maximumVisible: number =
!config.maximumVisible || config.maximumVisible < 1
Expand All @@ -54,8 +52,6 @@ export const useActionBarModel = createModelHook({
? totalSize
: config.maximumVisible;

console.log('maximumVisible', maximumVisible);

if (totalSize - hiddenIds.length >= maximumVisible) {
hiddenIds = items.slice(maximumVisible, totalSize).map(getId);
}
Expand Down

0 comments on commit 53de16a

Please sign in to comment.