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

Render Custom :shortcode: emoji support NIP-30 #2003

Open
2 tasks
alltheseas opened this issue Feb 13, 2024 · 16 comments
Open
2 tasks

Render Custom :shortcode: emoji support NIP-30 #2003

alltheseas opened this issue Feb 13, 2024 · 16 comments
Assignees
Labels
feature New feature request NIP tracking Tracking issues

Comments

@alltheseas
Copy link
Collaborator

alltheseas commented Feb 13, 2024

user story

As a Damus and emoji aficionado, I would like to be able to see & share custom emojis, so that I can increase my enjoyment of nostr.

Acceptance criteria

  1. Damus user can read custom shortcode emoji
  2. Damus user can send custom shortcode emoji

implementation

Example note with emoji links in tags

https://gist.github.com/mattn/9b43595824750c3794902d77ec9deb71

NIP-30

List of generic shortcode emojis: https://emojibase.dev/shortcodes/?

Origin
image

https://damus.io/nevent1qqs0hw2euptpu5k6t6ruyx6wtvvp9czzjeyr0eyj9ltru3p6gl9z8jgpramhxue69uhkummnw3ez6un9d3shjtnwda4k7arpwfhjucm0d5hsz9mhwden5te0wfjkccte9ehx7um5wghxyctwvshszythwden5te0dehhxarj9emkjmn99uqsuamnwvaz7tmev9382tndv5hsvu7lx4

@alltheseas alltheseas added the feature New feature request label Feb 13, 2024
@alltheseas alltheseas changed the title :Shortcode: emoji support :Shortcode: emoji support NIP-30 Feb 13, 2024
@alltheseas alltheseas added the NIP label Feb 13, 2024
@alltheseas alltheseas changed the title :Shortcode: emoji support NIP-30 Custom :shortcode: emoji support NIP-30 Feb 13, 2024
@alltheseas
Copy link
Collaborator Author

alltheseas commented Feb 13, 2024

Clients that support

Nostter
noStrudel
Amethyst
Rabbit (I am not sure to whether it's only reactions or kind1 also)

@mattn
Copy link

mattn commented Feb 13, 2024

njump will fix by fiatjaf/njump#42

@alltheseas
Copy link
Collaborator Author

open question

How are inline emojis displayed in notecrumbs (e.g. in telegram or other places where damus.io/neventlinks are shared)?

@kernelkind
Copy link
Contributor

I'm working on this

@alltheseas
Copy link
Collaborator Author

Do you think you can fit this in for current sprint @kernelkind ?

@kernelkind
Copy link
Contributor

I believe I can do rendering this sprint

@kernelkind
Copy link
Contributor

Can we break this issue up into multiple? I recommend the following issues:

  • rendering custom emojis
  • Ability to add custom emojis from local cache/db to new post
  • uploading new custom emojis

@alltheseas alltheseas changed the title Custom :shortcode: emoji support NIP-30 Render Custom :shortcode: emoji support NIP-30 Mar 14, 2024
@alltheseas
Copy link
Collaborator Author

Can we break this issue up into multiple? I recommend the following issues:

  • rendering custom emojis
  • Ability to add custom emojis from local cache/db to new post
  • uploading new custom emojis

Yes, great suggestion

#2055
#2056

@alltheseas alltheseas added this to the 1.8 post Madeira milestone Mar 14, 2024
@jb55
Copy link
Collaborator

jb55 commented Mar 16, 2024 via email

@alltheseas
Copy link
Collaborator Author

@jb55 has performance worries about this. Can @kernelkind @danieldaquino help test performance impacts of this feature add?

@alltheseas
Copy link
Collaborator Author

  1. manual test first,
  2. go to profiler ("xcode instruments"). SwiftUI profiler is good. Shares info about SwiftUI views (e.g. if views getting invalidated too quickly). Can help test network bandwidth.

Should collect network stats on individual components. Will suspects images. Some people suspect relay queries.

@alltheseas
Copy link
Collaborator Author

related

measure and attribute data #1838

@alltheseas
Copy link
Collaborator Author

@jb55
Copy link
Collaborator

jb55 commented May 10, 2024

there's a lot involved to implement this well:

  • rendering custom emojis (@kernelkind has a patch for this)
  • collecting seen custom emojis into a local collection for later use
  • updating our emoji picker to display all collected custom emojis
  • exploring custom emoji sets

I can start putting together a tracking issue for this

@jb55
Copy link
Collaborator

jb55 commented May 10, 2024

I will use this as the tracking issue

@jb55 jb55 added the tracking Tracking issues label May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature request NIP tracking Tracking issues
Projects
Status: In Review
Development

No branches or pull requests

4 participants