Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Smuxi emoticons #178

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

Smuxi emoticons #178

wants to merge 1 commit into from

Conversation

jamesaxl
Copy link
Contributor

Add emoticons to Smuxi, hope that they will make it more gorgeous 👍

@knocte
Copy link
Contributor

knocte commented Feb 27, 2014

Nice patch!

But I tested it and it doesn't work. I mean, yeah, it shows the faces and stuff, but if I choose one and I hit enter, it doesn't send the smiley to the channel. And if I write some text after the smiley, and hit enter, it sends the buffer to the channel but the smiley is not included in what is sent to the channel...

@jamesaxl
Copy link
Contributor Author

thank you very much i will resolve that problem

@jamesaxl
Copy link
Contributor Author

here you are :-)

@knocte
Copy link
Contributor

knocte commented Feb 27, 2014

Ok, it works now. But the images are not shown in the chat room, only when you select them. Is that expected?

@@ -357,6 +357,10 @@ public void AddMessage(MessageModel msg, bool addLinebreak, bool showTimestamps)
tags.Add(LinkTag);

buffer.InsertWithTags(ref iter, linkText, tags.ToArray());
} else if (msgPart is ImageMessagePartModel) {
var imgMsgPart = (ImageMessagePartModel) msgPart;
var imgPixbuf = Frontend.LoadIcon(imgMsgPart.ImageFileName, 16, imgMsgPart.ImageFileName + ".png");
Copy link
Owner

Choose a reason for hiding this comment

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

This assumption that any image is an emoticon can't be done. Either a special URI prefix needs to be used to at least the file name needs to be checked for the emoticon name pattern face-XXX-symbolic

@jamesaxl
Copy link
Contributor Author

The man who sold the world

@knocte
Copy link
Contributor

knocte commented Feb 27, 2014

The man who sold the world

Is that an answer to my question? Because I don't understand it.

@jamesaxl
Copy link
Contributor Author

sorry it was just a notification for meebey, you could test now, it works, thank you

@@ -0,0 +1,67 @@
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
Copy link
Owner

Choose a reason for hiding this comment

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

Move the .svg files to the same directory as the .png files (images/Smuxi-Emoticons/)

@jamesaxl
Copy link
Contributor Author

I am here , do you see me now :)

@@ -357,6 +358,10 @@ public void AddMessage(MessageModel msg, bool addLinebreak, bool showTimestamps)
tags.Add(LinkTag);

buffer.InsertWithTags(ref iter, linkText, tags.ToArray());
} else if (msgPart is ImageMessagePartModel) {
var imgMsgPart = (ImageMessagePartModel) msgPart;
var emoticonPixbuf = EmoticonStore.GetEmoticonImage(EmoticonStore[imgMsgPart.ImageFileName], 16);
Copy link
Owner

Choose a reason for hiding this comment

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

Don't always load ALL images from the emoticon store, it could also be a regular file, URL or whatever! Something NOT smiley

@jamesaxl
Copy link
Contributor Author

jamesaxl commented Apr 1, 2014

add new symbolic emoticons, create one instance for EmoticonsWindow call (to do not watse many resources)

@meebey meebey added the 0.12 label Apr 4, 2014
@meebey meebey added this to the 0.12 milestone Apr 4, 2014
@jamesaxl
Copy link
Contributor Author

jamesaxl commented Apr 8, 2014

Add new images with dimension 256x256

// private void OnDestroyed(object obj, EventArgs args)
// {
// Active = false;
// }ram is distributed in the hope that it will be useful,
Copy link
Owner

Choose a reason for hiding this comment

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

what happened here? :)

@knocte
Copy link
Contributor

knocte commented Nov 10, 2016

Was this PR superseded by some work already merged in the master branch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants