Issue 18896 Replace deprecated Button component with new Button component

[ujwalkumar1995]

Explanation

As part of this PR, I have replaced the depracated button component (ui/components/ui/button/button.component.js) with ui/components/component-library/button/button.js

• Fixes Replace deprecated Button component with new Button component #18896
• Fixes Deprecate old UrlIcon in favour of appropriate Avatar #18718

Screenshots/Screencaps

Before

After

Manual Testing Steps

Pre-merge author checklist

• I've clearly explained:
  • What problem this PR is solving
  • How this problem was solved
  • How reviewers can test my changes
• Sufficient automated test coverage has been added

Pre-merge reviewer checklist

• Manual testing (e.g. pull and build branch, run in browser, test code being changed)
• PR is linked to the appropriate GitHub issue
• IF this PR fixes a bug in the release milestone, add this PR to the release milestone

If further QA is required (e.g. new feature, complex testing steps, large refactor), add the Extension QA Board label.

In this case, a QA Engineer approval will be be required.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[ujwalkumar1995]

Hey @georgewrmarshall

I have submitted this PR for fixing one of the files for issue #18896
I can see in the screenshot of before and after that there is some styling difference between the old and new buttons. I am not sure if this is intended as per some new design as I have not changed any styling as part of my changes.
Please let me know if this needs to be fixed and I will try to explore further.

Thanks in advance!

[georgewrmarshall]

Hey @ujwalkumar1995, would you be able to update this PR I believe it is close so it would be great if we can get it merged

[ujwalkumar1995]

Hey @georgewrmarshall
Sorry I completely forgot about this PR. I could not see any open review commits but I could see a merge conflict. I have resolved it. Please review.

[codecov]

Codecov Report

Merging #19559 (4a365e5) into develop (d3c9d4a) will decrease coverage by 0.01%.
Report is 2 commits behind head on develop.
The diff coverage is 20.00%.

@@             Coverage Diff             @@
##           develop   #19559      +/-   ##
===========================================
- Coverage    68.67%   68.67%   -0.01%     
===========================================
  Files          986      986              
  Lines        37862    37862              
  Branches     10135    10135              
===========================================
- Hits         26001    25998       -3     
- Misses       11861    11864       +3     

Files Changed	Coverage Δ
.../component-library/modal-content/modal-content.tsx	96.15% <ø> (ø)
ui/components/ui/url-icon/url-icon.js	100.00% <ø> (ø)
...swaps/dropdown-search-list/dropdown-search-list.js	75.68% <0.00%> (ø)
...pages/swaps/prepare-swap-page/prepare-swap-page.js	69.38% <0.00%> (ø)
ui/pages/swaps/import-token/import-token.js	75.00% <33.33%> (-12.50%)	⬇️

... and 2 files with indirect coverage changes

[georgewrmarshall]

LGTM! Thanks for your contribution @ujwalkumar1995