Angular catalog holdings view suffers from material icon formatting issue

[Bill Erickson]

Evergreen 3.3

Part of the work of bug #1830912 was identifying material icons that behave oddly when served locally instead of from the Google servers -- a handful of icons add extra padding. Most of them were replaced, but I just found another that affects the Angular catalog Holdings view.

When copy rows are expanded, barcode values and sometimes call number values fail to render under the "Location/Barcode" column. Closer inspection shows the data is there, it's just pushed outside the scope of the grid cell because of the extra icon padding.

Replacing "arrow_right" with keyboard_arrow_right (and arrow_drop_down with keyboard_arrow_down for consistency) fixes the issue.

Branch en route.

[Bill Erickson (berick)]

Fix pushed:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1840982-holdings-icon-fix

To test, apply patch and confirm the "Location/Barcode" column in the Holdings grid contains values for every row, copy barcodes in particular.

[Jane Sandberg (sandbergja)]

Thanks for this fix, Bill. The holdings view was looking very odd without it. It works well. I've pushed it to master, rel_3_4, and rel_3_3.