fix(breadcrumbs): breadcrumbs ui improvement and minor bug fix

[Shubhdeep12]

• some UI improvement
• bug fix - if statement for checking empty screen in breadcrumbs was missing a condition

Fixes: #178

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
spotlightjs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 1, 2023 3:00pm

[codecov]

Codecov Report

Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (68ec4a9) 32.88% compared to head (ee979ed) 32.57%.
Report is 3 commits behind head on main.

❗ Current head ee979ed differs from pull request most recent head a5ed657. Consider uploading reports for the commit a5ed657 to get more accurate results

Files	Patch %	Lines
...ntegrations/sentry/components/EventBreadcrumbs.tsx	18.18%	9 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #194      +/-   ##
==========================================
- Coverage   32.88%   32.57%   -0.32%     
==========================================
  Files          37       37              
  Lines        1785     1793       +8     
  Branches       71       68       -3     
==========================================
- Hits          587      584       -3     
- Misses       1198     1209      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Lms24]

Thanks! I think this already looks much better! I have one more suggestion for a quick improvement:

Can we display the breadcrumb category above the timestamp and make it visually more distinctive? E.g. make the category bolder and a bit larger (or the time smaller). I'll leave the details up to you ;)

[Shubhdeep12]

Thanks! I think this already looks much better! I have one more suggestion for a quick improvement:

Can we display the breadcrumb category above the timestamp and make it visually more distinctive? E.g. make the category bolder and a bit larger (or the time smaller). I'll leave the details up to you ;)

Sure.

[Shubhdeep12]

cc @Lms24

[Lms24]

Awesome, thanks!