Happy Face Icons

[DamnDramaQueen]

Hi,

This is my first timer putting request in, hope is right?

I would like to suggest a new feature be added for the rtmedia lightbox comment box. Currently the comment box only do writing and do not have icons. NOT the emoji I am talking about.

This is an i.e icons would like. birthday cake, smilely, gift, teddy bear. heart, etc etc.

People would like to insert icons along when making comments. Who doesn't?

Please send reply notification or if any update for this feature.

Thanks

[aviral-mittal]

Hi @robicse11127 can you please share an update on this?

[robicse11127]

@aviral-mittal It is in progress.... Configuring the Emojies on the lightbox comment area.

[aviral-mittal]

@pavanpatil1 Can you please verify these changes?

[kjnanda]

@pavanpatil1 Added emoji support works fine but still there is scope of improvement.

• emoji icon size is uneven
• spacing seems not uniform
• currently, we are only showing limited emoji. The requested emoji icons seems not been added.

[Utsav-Ladani]

Raised this PR to resolve this issue. Please check and let me know if there is any issue.

CC: @kjnanda, @pavanpatil1

[kjnanda]

@Utsav-Ladani We can give scroll event for emoji inside the child panel instead of whole parent window. This does not give good UX.

Please refer to this visual

---

I am yet to test this on Responsive devices. Will the share feedback soon.

I have tested this issue on mobile device and I'm unable to scroll when i open media in light box as shown below visual

RecordIt-B6D54D4F-9B31-401C-B791-C95110F28D2F.2.MP4

CC @pavanpatil1

[Utsav-Ladani]

I had fixed the UI, but this distortion is coming from emoji-picker.js minified files. I didn't find the source code of this minified file. So we have to create our custom library or use an existing third-party library. I will let you know once it is finalized and the issue is fixed.

[kjnanda]

@pavanpatil1 Please refer to comment added by Utsav ☝🏻

[pavanpatil1]

Hi @kjnanda @Utsav-Ladani, I have checked this and the scroller in the emoji picker seems to be fine now. We can merge the PR in pre-release

Record_2023-08-09-13-09-44.mp4

[kjnanda]

@pavanpatil1I've verified this problem on an iPhone, and it's not functioning as intended. The emoji picker is not triggering any scroll events. Do we are still want to progress with know bug?

[Utsav-Ladani]

I made a few changes yesterday. It is working fine for me on Android. Please check and let me know if something not working.

[kjnanda]

Cross verified this issue wrt dev branch and it is working as expected.

[kjnanda]

Tested after release and it works as expected.