Skip to content

feat: Notification screen Implementation-Added Icons#919

Open
chith-git wants to merge 24 commits into
kubecon-2025from
feat/notification
Open

feat: Notification screen Implementation-Added Icons#919
chith-git wants to merge 24 commits into
kubecon-2025from
feat/notification

Conversation

@chith-git

Copy link
Copy Markdown
Contributor

Added icons for Recommendation List and Recommendation Details screens

asika-gadgeon
asika-gadgeon previously approved these changes Oct 8, 2025

@AbhishekA1509 AbhishekA1509 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Pls delete these files:
src/Shared/Components/SelectPicker/GroupedFilterSelectPicker.example.tsx
src/Shared/Components/SelectPicker/GroupedFilterSelectPicker-Usage-Guide.md

Comment thread src/Assets/IconV2/ic-coin.svg Outdated
@@ -0,0 +1,3 @@
<svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" fill="none" viewBox="0 0 14 14">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We can use ic-coins instead with sending color B500 as prop for Icon

@@ -0,0 +1,6 @@
<svg width="16" height="16" viewBox="0 0 16 16" fill="none" xmlns="http://www.w3.org/2000/svg">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We can use ic-circle-loader, pls check across the PR for other icons as well if there are any counterpart present like ic-speedometer, etc

Comment thread src/Common/API/CoreAPI.ts
Comment thread src/Common/Constants.ts Outdated
@@ -0,0 +1,3 @@
<svg width="12" height="13" viewBox="0 0 12 13" fill="none" xmlns="http://www.w3.org/2000/svg">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Pls delete this is ic-bug

@@ -0,0 +1,3 @@
<svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" fill="none" viewBox="0 0 14 14">
<path fill="#B23412" d="M3.063 7h-.876a.875.875 0 0 0-.874.875v3.5c0 .483.391.875.875.875h.874a.875.875 0 0 0 .876-.875v-3.5A.875.875 0 0 0 3.062 7Zm4.375-2.625h-.875a.875.875 0 0 0-.875.875v6.125c0 .483.391.875.875.875h.875a.875.875 0 0 0 .875-.875V5.25a.875.875 0 0 0-.876-.875Zm4.375-2.625h-.876a.875.875 0 0 0-.874.875v8.75c0 .483.391.875.874.875h.876a.875.875 0 0 0 .874-.875v-8.75a.875.875 0 0 0-.874-.875Z"/>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is ic-priority-medium-fill itself just send color prop

@@ -0,0 +1,4 @@
<svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" fill="none" viewBox="0 0 14 14">
<path fill="#106240" d="M3.063 7h-.876a.875.875 0 0 0-.874.875v3.5c0 .483.391.875.875.875h.874a.875.875 0 0 0 .876-.875v-3.5A.875.875 0 0 0 3.062 7Z"/>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is ic-priority-medium-fill itself just send color prop

@@ -0,0 +1,5 @@
<svg width="32" height="32" viewBox="0 0 32 32" fill="none" xmlns="http://www.w3.org/2000/svg">
<rect width="32" height="32" rx="4" fill="#1E1F28"/>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is ic-edit leverage that with centred in div

@@ -0,0 +1,3 @@
<svg width="2" height="14" viewBox="0 0 2 14" fill="none" xmlns="http://www.w3.org/2000/svg">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should be achieved with css instead have looked into design, say in case the text is multiline the height should increase subsequently

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For example:
Screenshot 2025-10-28 at 11 10 23 AM
In Design these are connected to avatar there is no gap

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants