Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@JF-Cozy
Copy link
Collaborator

@JF-Cozy JF-Cozy commented Oct 2, 2025

to disable closing when clicking away or waiting the timeout. I moved the handleClose func in a helper file to avoid styleguide messing with func name

to disable closing when clicking away or waiting the timeout.
I moved the handleClose func in a helper file to avoid styleguide
messing with func name
@JF-Cozy JF-Cozy requested review from Ldoppea and zatteo as code owners October 2, 2025 09:49
@bundlemon
Copy link

bundlemon bot commented Oct 2, 2025

BundleMon

Unchanged files (3)
Status Path Size Limits
dist/cozy-ui.min.css
20.02KB -
transpiled/react/stylesheet.css
17.74KB -
dist/cozy-ui.utils.min.css
10.97KB -

No change in files bundle size

Groups updated (1)
Status Path Size Limits
transpiled/react/**
1.03MB (+729B +0.07%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

message,
title,
duration,
noTimeOut, // eslint-disable-line no-unused-vars
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👮 🚨

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comment just above to explain that

variant="filled"
elevation={6}
onClose={handleClose(state, setState)}
onClose={ev => handleClose(state, setState)(ev, 'click')}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why click? if it's an arbitrary value it can also be undefined or null

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, because we use "clickaway" and "timeout", I think "click" is also explicit

@JF-Cozy JF-Cozy merged commit eee29d0 into master Oct 2, 2025
12 checks passed
@JF-Cozy JF-Cozy deleted the feat/alert branch October 2, 2025 12:34
@cozy-bot
Copy link

cozy-bot commented Oct 2, 2025

🎉 This PR is included in version 130.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants