Skip to content
This repository was archived by the owner on Sep 4, 2020. It is now read-only.

Conversation

@braxtonCoats
Copy link
Contributor

Added a forward Icon, updated components with missing icons and changed font name at @a-type suggestionl

Copy link
Contributor

@tato123 tato123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Filter icon appears to be incorrect as well, please see https://bandwidth.codefactori.com/browse/BWDB-5459

@braxtonCoats braxtonCoats requested a review from SmithPR as a code owner July 1, 2019 15:03
Copy link
Contributor

@tato123 tato123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Braxton and I discussed, the icon for BWDB-5459 appears to be present as part of this change

@zacklitzsinger
Copy link
Contributor

@braxtonCoats This has merge conflicts – can you resolve them? Should be able to pull master, then merge master, and update the files.

@braxtonCoats
Copy link
Contributor Author

fixed merging errors and should be good to go

@zacklitzsinger zacklitzsinger merged commit 77a3db0 into Bandwidth:master Jul 2, 2019
@ghost
Copy link

ghost commented Jul 5, 2019

Putting this here for reference:

Icons removed:
administrator amazon api apiSettings appointmentReminders ascendingSorting average2 businessman calendar1 calendar_1 calendar10 calendar_10 calendar11 calendar_11 calendar12 calendar_12 calendar13 calendar_13 calendar14 calendar_14 calendar15 calendar_15 calendar16 calendar_16 calendar17 calendar_17 calendar18 calendar_18 calendar19 calendar_19 calendar2 calendar_2 calendar20 calendar_20 calendar21 calendar_21 calendar22 calendar_22 calendar23 calendar_23 calendar24 calendar_24 calendar25 calendar_25 calendar26 calendar_26 calendar27 calendar_27 calendar28 calendar_28 calendar29 calendar_29 calendar3 calendar_3 calendar30 calendar_30 calendar31 calendar_31 calendar4 calendar_4 calendar5 calendar_5 calendar6 calendar_6 calendar7 calendar_7 calendar8 calendar_8 calendar9 calendar_9 callTransfer callTransferFilled cat cheap2 checkedCheckbox clearShoppingCart collaborator collapse columns computer controller courses coursesFilled crown delete_2 descendingSorting disclaimer down2 downloadFromCloud emptyBox error errorFilled exclamationMarkFilled exit expensive2 externalLink flashOn genderNeutralUser googleLogoFilled grid guest hashtag hashtagFilled help2 help_2 hide home insertTable journey journeyFilled link loginRounded logoutRounded lol lowPrice maintenance menu menu2 message minus moderator moveGrabber musicVideo okFilled onlineSupport openInWindow pdf2 pdf_2 phone phoneFilled picture pokemon portrait readMessage save searchFilled securityChecked server shoppingCart signature skypeFilled sms smsFilled sortingOptions speechBubble speechBubbleFilled speedometer spinnerFrame1 spinnerFrame_1 spinnerFrame2 spinnerFrame_2 spinnerFrame3 spinnerFrame_3 spinnerFrame4 spinnerFrame_4 spinnerFrame5 spinnerFrame_5 spinnerFrame6 spinnerFrame_6 spinnerFrame7 spinnerFrame_7 spinnerFrame8 spinnerFrame_8 stack stackOfPhotos thumbUp toggleOff toggleOnFilled toolbox tripadvisor tvShow uncheckedCheckbox unicorn up up2 user userFemaleBody userMaleCircle videoPlaylist visible voipGateway whatsappFilled worker

Icons Added:
arrowDown arrowUp bell caret cartEmpty cart envelope filter move newWindow pdf plus sliders table x

@braxtonCoats braxtonCoats deleted the Updated_Icons branch August 7, 2019 20:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants