PM tagging / tags no working on mobile

Contribute Bug

AI-generated summary

The PM tagging feature is not working on mobile devices. sok777 reported the issue, and JammyDodger confirmed it, providing a workaround by editing the PM title or OP to access the tag input box. pmusaraj found that this issue has existed since at least August 2022. According to nbianca, the issue was likely due to space constraints, but a recent layout change makes it possible to implement the feature again. A pull request (#22241) has been merged by nbianca, and updating the instance should resolve the issue.

1

I noticed the textarea/input form for adding PM tags isn’t showing up on mobile.

Any workaround for that?

1 Like
2

Hmmm. Yes, adding a tag to a PM when creating a message does not seem to be appearing on mobile. :thinking:

However, if you create a message, and then either edit the title or OP then the tag input box appears.

I’ll slide this over to Contribute > Bug. :+1:

  • Add a group (staff) to pm tags allowed for groups
  • Create a PM using a mobile
  • No tagging option available in composer
  • Send a PM
  • Edit the PM title or OP and the tagging option appears

I would expect the tagging option to appear in both create and edit

4 Likes
3

Thanks! Glad that’s useful and thank you for the workaround.

2 Likes
4

It looks like this is not a recent regression, it has been like this since at least August 2022: FIX: Show tag chooser if can_tag_pms (#18107) · discourse/discourse@f351200 · GitHub

Do you remember if this was for usability reasons, @nbianca?

1 Like
5

I do not remember the whole context, but I think it was done this way because of space constraints. It can get very cluttered for PMs because there are inputs for the recipients, title and tags too. I think something else changed in the layout because now it looks like it could work:

https://github.com/discourse/discourse/pull/22241

4 Likes
7

@nbianca I can’t see it on the list of commits?

8

The PR is still open and under review right now, but I expect it to be merged soon.

2 Likes
9

@sok777, I have just merged this commit. You can update your instance and you will get the fix. Thank you for the patience!

3 Likes
10

This topic was automatically closed after 4 days. New replies are no longer allowed.