Skip to content
This repository has been archived by the owner on Jun 11, 2020. It is now read-only.

Status dot icon #126

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Status dot icon #126

wants to merge 4 commits into from

Conversation

Schlumpf
Copy link
Contributor

Simple implementation of the status icons of the mockup images.

@nurupo
Copy link
Owner

nurupo commented Aug 1, 2014

Any reason against using image files from the mockup UI repo?

@nurupo
Copy link
Owner

nurupo commented Aug 1, 2014

To be honest, I don't really like that status indicator also being used to indicate unread messages, but I guess following the mockup for now it's fine, we always can change things if needed.

If you want to work on styling, one of the steps of styling would be porting the way Quassel does themeing, with those Qt style sheet files (Qt doc, Quassel themes). I think Quassel just uses a custom parser which feeds things in Qt, so there is not that much code (I might be wrong).

painter->setBrush(Qt::NoBrush);
painter->drawEllipse(dotPos, 4, 4);
if (getUnreadMessages(index)) {
painter->setPen(QPen(QColor("#CEBE44"), 1.5));
Copy link
Owner

Choose a reason for hiding this comment

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

Also, I believe this should be #C74F50.

snapshot16-

@Schlumpf
Copy link
Contributor Author

Schlumpf commented Aug 1, 2014

I just didn't know, that the mockup repo exists, so there is no reason. I will change do those icons.
I like the combination of status and notification icon. It saves space and looks fine, in my opinion ;)

Currently I'm still working on smileys, emoji and chat view. But I'm very busy with my masters, and I needed a simple task to do.

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.

2 participants