[bug] emoji picker on mobile post modal is un-scrollable on iOS Safari #321

Closed
opened 2023-07-07 15:48:04 +00:00 by SukinoVerse · 4 comments
Contributor

Version

Akkoma: 3.9.3-0-g9d7c877
Akkoma-FE: e530c2b

What were you trying to do?

Using iOS Safari to create a new post with emoji

What did you expect to happen?

  • I should be able to scroll across emoji packs list
  • I should be able to scroll across emoji items list

What actually happened?

  • Emoji pack list is un-scrollable
  • Emoji item list is un-scrollable

This is make impossible for user to pick many emojis when create a new post from iOS Safari.

Noted: I'm not a frontend developer not sure what is the best way to fix this. Is this kind of code is acceptable anyone can open PR with it :)

Severity

I can manage

Have you searched for this issue?

  • I have double-checked and have not found this issue mentioned anywhere.
### Version Akkoma: 3.9.3-0-g9d7c877 Akkoma-FE: e530c2b ### What were you trying to do? Using iOS Safari to create a new post with emoji ### What did you expect to happen? - I should be able to scroll across emoji packs list - I should be able to scroll across emoji items list ### What actually happened? - Emoji pack list is un-scrollable - Emoji item list is un-scrollable This is make impossible for user to pick many emojis when create a new post from iOS Safari. Noted: I'm not a frontend developer not sure what is the best way to fix this. Is this kind of code is acceptable anyone can open PR with it :) ### Severity I can manage ### Have you searched for this issue? - [x] I have double-checked and have not found this issue mentioned anywhere.
Author
Contributor

Looks like it was fixed in Pleroma side week ago

Ref: https://git.pleroma.social/pleroma/pleroma-fe/-/merge_requests/1844

Looks like it was fixed in Pleroma side week ago Ref: https://git.pleroma.social/pleroma/pleroma-fe/-/merge_requests/1844

then raise a PR? as far as I know absolutely none of the current contributors use Safari and as such none of us can confirm this works for you

then raise a PR? as far as I know absolutely none of the current contributors use Safari and as such none of us can confirm this works for you
Author
Contributor

I try to adopt a Pleroma solution here but doesn't work due to different behavior of emoji-picker.

Yes, I'm not expert at frontend / vue but will create a workaround PR here perhaps someone need it :)
#323

I try to adopt a Pleroma solution here but doesn't work due to different behavior of emoji-picker. Yes, I'm not expert at frontend / vue but will create a workaround PR here perhaps someone need it :) https://akkoma.dev/AkkomaGang/akkoma-fe/pulls/323
Mergan added the
Bug
label 2023-07-17 19:10:13 +00:00
Author
Contributor

Fixed by #323

Fixed by https://akkoma.dev/AkkomaGang/akkoma-fe/pulls/323
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: AkkomaGang/akkoma-fe#321
No description provided.