Skip to content
This repository has been archived by the owner on Feb 12, 2023. It is now read-only.

feat(ui): new avatar icon #5148

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat(ui): new avatar icon #5148

wants to merge 1 commit into from

Conversation

PKEv
Copy link
Contributor

@PKEv PKEv commented May 18, 2018

as an alternative solution for #4826
2018-05-18 12-35-38
2018-05-18 12-35-53


This change is Reviewable

@sudden6
Copy link
Member

sudden6 commented May 18, 2018

This breaks on OSX apparently: https://travis-ci.org/qTox/qTox/jobs/380616340#L1366

@PKEv
Copy link
Contributor Author

PKEv commented May 18, 2018

there are ideas of other icons?

@PKEv
Copy link
Contributor Author

PKEv commented May 18, 2018

@sudden6 I see only an error "missing-override"? I'm right?

@sudden6
Copy link
Member

sudden6 commented May 18, 2018

For me this seems not to work as intended:
screenshot_20180518_215606

Yes the missing override error is what I meant, didn't research the source though.

as an alternative solution for qTox#4826
@PKEv
Copy link
Contributor Author

PKEv commented May 22, 2018

@sudden6 agree
Any suggestions?

@sudden6
Copy link
Member

sudden6 commented May 29, 2018

@PKEv I don't really have an idea what the problem is. :/

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.

None yet

2 participants