hotwax / receiving

HotWax Commerce Receiving App
https://receiving.hotwax.io/
Apache License 2.0
8 stars 40 forks source link

Fixed: the issue of line after button by improving the html structure(#294) #320

Closed ymaheshwari1 closed 8 months ago

ymaheshwari1 commented 8 months ago

Removed: ion-item and replaced it with ion-buttons as when declaring ion-button inside toolbar we should use ion-buttons Added attributes on the ion-button to match the styling with the figma design

Related Issues

Closes #294

Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

Before:

image

After:

image

IMPORTANT NOTICE - Remember to add changelog entry

Contribution and Currently Important Rules Acceptance

ymaheshwari1 commented 8 months ago

@dt2patel Sir, should I add styling to match the same with the previous UI / figma?

ymaheshwari1 commented 8 months ago

@dt2patel Sir, should I add styling to match the same with the previous UI / figma?

Discussed, we don't need to add styling for this.