push-protocol / push-dapp

The Push Protocol Dapp
https://app.push.org
Other
42 stars 36 forks source link

Update old and add new icons #1673

Closed kalashshah closed 1 week ago

kalashshah commented 1 week ago

Pull Request Template

Ticket Number

Description

List of icons updated:

List of new icons added:

Type of Change

Checklist

Frontend Guidelines

Build & Testing

Screenshots/Video with Explanation

Screenshot 2024-06-26 at 3 26 01 PM

Additional Context

Review & Approvals

Notes

NA

github-actions[bot] commented 1 week ago

The code in the provided files appears to have various mistakes:

  1. In all SVG components, the attribute stroke-width should be changed to strokeWidth.
  2. In Add.tsx, the second d property should be inside the path tag.
  3. In AddEmoji.tsx, there are multiple instances where the d property should be enclosed within the path tags and the comments should be removed.
  4. In Back.tsx, the second d property should be inside the path tag.
  5. In BellSimple.tsx, the comments should be removed and the d properties should be enclosed within the path tags.
  6. In CalendarBlank.tsx, the second and third d properties should be inside the path tags.
  7. In Copy.tsx, the clip-path attribute should be changed to clipPath.
  8. In Copy.tsx, the second d property should be inside the path tag.
  9. In Dashboard.tsx, the second x property should be inside the rect tag.
  10. In Front.tsx, the second d property should be inside the path tag.

After addressing the above issues, the SVG components should display correctly.

github-actions[bot] commented 1 week ago

PR Preview Action v1.4.7 :---: Preview removed because the pull request was closed. 2024-06-27 14:33 UTC