zhukov / webogram

Telegram web application, GPL v3
https://web.telegram.org
GNU General Public License v3.0
7.96k stars 2.35k forks source link

Clean up gemoji resources, fix emoji menu scrollbar #118

Closed staltz closed 10 years ago

staltz commented 10 years ago

@zhukov : "The only thing left is to replace gemoji icons inside richText with ones from new spritesheet." https://github.com/zhukov/webogram/pull/111#issuecomment-36456193

@nukeador : "I found a small issue. When you change from category to category, scroll bar seems to remember state, so if you where at the bottom on the previous category, you ended at the bottom of the new active one." https://github.com/zhukov/webogram/issues/56#issuecomment-36460649

I can take this one, doesn't seem to be as big as #56.

zhukov commented 10 years ago

Good.

I would suggest modifying RichTextProcessor service to move emoji to new spritesheets. (I didn't understand .im_history restriction in PR comments)

staltz commented 10 years ago

Well, I can find my way once I look at all the code. I think it's enough that I replace all requests for separated emoji icons into request for spritesheet icons, right?

zhukov commented 10 years ago

Yes, that's right. I was just confused a little by ".im_history", while these emojis are also used in user names etc.