Skip to content

Conversation

Coussecousse
Copy link

@Coussecousse Coussecousse commented Sep 24, 2025

Bob ask me to work on a functionnality that 'highlight' in blue the new items.
It used the same blue as selected items.
The isNewItem prop can be null without causing an issue.

Example :
image

Code in tdrive cozy/cozy-drive#3501 .

Copy link

bundlemon bot commented Sep 24, 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 (+113B +0.01%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@Coussecousse Coussecousse force-pushed the ean_newItems branch 2 times, most recently from 879590d to 810093c Compare September 24, 2025 13:41
…ms and highlight them

- IsNewItem check if the item is the same as in the list.
@lethemanh lethemanh merged commit d60d485 into master Oct 1, 2025
12 checks passed
@lethemanh lethemanh deleted the ean_newItems branch October 1, 2025 12:43
@cozy-bot
Copy link

cozy-bot commented Oct 1, 2025

🎉 This PR is included in version 130.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

className={dragCollect.isDragging ? 'virtualized u-o-50' : 'virtualized'}
className={cx(
dragCollect.isDragging ? 'virtualized u-o-50' : 'virtualized',
isNew ? 'u-bg-primaryColorLight' : ''
Copy link
Collaborator

@JF-Cozy JF-Cozy Oct 7, 2025

Choose a reason for hiding this comment

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

be careful we don't use u-classes to change color anymore, don't know if they are still relevant. For now I recommand to avoid this while someone didn't take a deep look on it.. For example it's not compliant with dark mode, see #2867 (comment)

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