Skip to content

Conversation

@josh-ramos-22
Copy link
Contributor

Why the changes are required

Currently, the buttons on the sidebar have no icons. By adding universally recognised icons, users will be able to instantly know the functionality of each button.

Changes

In frontend/src/packages/dashboard/components/Sidebar/Sidebar.tsx:

  • Edited styled component SidebarButton to have display: flex, in order to align the button icons and text properly.
  • Created new styled components ButtonIcon and ButtonText to style the individual components of each button.
  • Imported icons for each of the three buttons from the MUI Icon Library that most closely resembled those specified in the ticket. Integrated them into each button as required.

Screenshots

Screenshot_20230226_023458

Copy link
Contributor

@lauraw0 lauraw0 left a comment

Choose a reason for hiding this comment

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

lgtm you can merge now! :D

@josh-ramos-22 josh-ramos-22 merged commit f5d7c19 into main Mar 4, 2023
@josh-ramos-22 josh-ramos-22 deleted the WEB-3_button_icons branch March 4, 2023 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants