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

[doodle_bear_dev]直播间弹幕emoji适配 #5

Closed
wants to merge 1 commit into from
Closed

[doodle_bear_dev]直播间弹幕emoji适配 #5

wants to merge 1 commit into from

Conversation

GyakuKaku
Copy link

@GyakuKaku GyakuKaku commented Jan 12, 2023

对直播间新出的弹幕emoji进行了适配,未验证二次开发的css样式的适配情况。 Modified By Gyaku

@DoodleBears
Copy link
Owner

DoodleBears commented Jan 12, 2023

感谢PR,才注意到B站新增了这个功能。

但是:https://www.yuque.com/doodle-irifi/ueaigm/laogg2#c73f7474

之前更新表情包功能时,希望使用者自行在表情包配置界面对应关键词和图片的关系,这样可以更方便自定义大小和想要的图片。

如果是要加入这个功能,会不会基于现有的表情包的显示方式,将B站新增的emoji关键词和url存为key-value加入到Trie比较好?这样不需要去写新的“渲染”部分的代码。

详见 frontend/src/views/Room.vuegetRichContent(data)frontend/src/utils/trie.js 里关于 Trie 的代码

如果你继续改的话,我之后合并。(跪谢)当然我自己改也可以。

@DoodleBears
Copy link
Owner

已经更新,感谢

@GyakuKaku GyakuKaku deleted the feature_emoji branch January 13, 2023 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants