Skip to content

Conversation

iccicci
Copy link
Contributor

@iccicci iccicci commented Sep 22, 2025

Checklist

  • JIRA - LW-13584
  • Proper tests implemented
  • Screenshots added.

Proposed solution

Add Notifications Center bell icon and badge.
The notifications count and the click handler are clearly tests version to be substituted with actual implementation when Notification Center Service will be ready.

@iccicci iccicci requested a review from a team as a code owner September 22, 2025 09:14
@iccicci iccicci force-pushed the feat/LW-13584-notifications-bell-badge branch from 6c9b610 to 2cf1ae4 Compare September 22, 2025 09:43
@lace-bot
Copy link
Collaborator

lace-bot commented Sep 22, 2025

Allure Report

allure-report-publisher generated test report!

processReports: ✅ test report for c683420f

passed failed skipped flaky total result
Total 33 0 4 0 37

@iccicci iccicci force-pushed the feat/LW-13584-notifications-bell-badge branch from 2cf1ae4 to a31e90b Compare September 22, 2025 13:27
Copy link
Contributor

@mirceahasegan mirceahasegan left a comment

Choose a reason for hiding this comment

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

Great work @iccicci

@iccicci iccicci force-pushed the feat/LW-13584-notifications-bell-badge branch from 64717e8 to c683420 Compare September 23, 2025 10:46
@mirceahasegan
Copy link
Contributor

Just one more thing here. The light theme bell icon badge count color seems off. It should look like this:
image
But it looks like this
image

Copy link

@iccicci iccicci merged commit 0903080 into main Sep 23, 2025
27 checks passed
@iccicci iccicci deleted the feat/LW-13584-notifications-bell-badge branch September 23, 2025 11:10
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.

4 participants