Changed components to KComponents

[BabyElias]

Summary

Updated IconButton to use KComponents (KIcon & KIconButton) instead. This is the part of a series of changes, to be made under Issue #219 in Kolibri Design System.

Description of the change(s) you made

learningequality/kolibri-design-system#219

Screenshots (if applicable)

The icons that have been changed

Contributor's Checklist

Testing:

• Code is clean and well-commented
• Contributor has fully tested the PR manually
• If there are any front-end changes, before/after screenshots are included

Reviewer's Checklist

This section is for reviewers to fill out.

• Automated test coverage is satisfactory
• PR is fully functional
• PR has been tested for accessibility regressions
• External dependency files were updated if necessary (yarn and pip)
• Documentation is updated
• Contributor is in AUTHORS.md

[BabyElias]

The command yarn run lint-frontend:format is exiting with an error I am unable to figure anything about.
Any help to fix this linting error would be highly appreciated.
Thanks!

[akolson]

Changes look correct to me and manual QA checks out. The reported linting build issue looks to be an issue with the current version of kolibri-tools being used in studio. Updating it should resolve the issue. cc @ozer550

[ozer550]

Changes look correct to me and manual QA checks out. The reported linting build issue looks to be an issue with the current version of kolibri-tools being used in studio. Updating it should resolve the issue. cc @ozer550

I agree! Should be ready to go in, Just one suggestion @BabyElias, you could also use the pre-commit hooks for liniting which could help regarding linting issues before each commit. To install pre-commit hooks you could run pre-commit install in your studio directory with the virtual environment on and this should do the magic.

[BabyElias]

Changes look correct to me and manual QA checks out. The reported linting build issue looks to be an issue with the current version of kolibri-tools being used in studio. Updating it should resolve the issue. cc @ozer550

I agree! Should be ready to go in, Just one suggestion @BabyElias, you could also use the pre-commit hooks for liniting which could help regarding linting issues before each commit. To install pre-commit hooks you could run pre-commit install in your studio directory with the virtual environment on and this should do the magic.

Ohkay @ozer550! Will keep this handy from next PRs.

[BabyElias]

Hey!
Are any changes needed on this PR?

[rtibbles]

Hi @BabyElias - it looks like you just need to commit the formatting change in TreeViewBase after running linting. What looks like an error in the command is actually just a log message, the reason it is exiting with error 1 is because of the formatting change.

[BabyElias]

Thank you @rtibbles! Doing this just perfectly solved the problem I had been facing for long now :)

[akolson]

This should be good for a merge. Thanks @BabyElias!

[akolson]

Approved!