Feat: Activity feed revamp #15716
Feat: Activity feed revamp #15716
Conversation
github-actions
bot
added
the
safe to test
Ashish8689
requested review from
a team,
harshach,
karanh37,
chirag-madlani and
Sachin-chaurasiya
as code owners
|
@Rupchavan classification icon doesn't look right |
...in/resources/ui/src/components/ActivityFeed/ActivityFeedCard/FeedCardBody/FeedCardBodyV1.tsx
Outdated
Show resolved
Hide resolved
...esources/ui/src/components/ActivityFeed/ActivityFeedCard/FeedCardBody/feed-card-body-v1.less
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Can we rename this to ActivityFeedCard?
There was a problem hiding this comment.
We still have the old file for ActivityFeedCard and it is being used, will remove once it dependency is removed!
| @@ -480,6 +487,7 @@ a[href].link-text-grey, | |||
| // breadcrumb | |||
|
|
|||
| .breadcrumb-item { | |||
| max-width: 100%; | |||
There was a problem hiding this comment.
Do we want this to be applied to all the breadcrumb?
There was a problem hiding this comment.
yes, so the case was a breadcrumb item was overflowing due the continuous string in it present. so this max-width will only used the width of the container and the split to the next line.
|
|
Describe your changes:
Fixes #13871
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>