Brand Header breaks chat reply box on mobile (iOS)

Contribute Bug
,

AI-generated summary

On iOS, the Brand Header theme component breaks the chat reply box, rendering chat unusable in both the Discourse Hub app and PWA ludwikc. After testing in Safe Mode, ludwikc confirmed the issue stems from the Brand Header component ludwikc, and disabling it resolves the problem ludwikc. Moin suggested Safe Mode for troubleshooting, and Lilly acknowledged the report and flagged it for review. chapoi questioned whether the brand header should appear in chat at all, given the distinction between forum and chat modes, and implemented a fix to hide it on mobile via https://github.com/discourse/discourse-brand-header/pull/54 chapoi. However, merefield noted the same issue persists on Meta due to the Job ad banner — dismissing it restores chat functionality. chapoi confirmed the team is working on a more comprehensive solution.

1

On iOS chat doesn’t show the reply box, making chat useless.

Here’s the video documenting the bug:
https://link.siadlak.com/oYK6

I’m on iPhone 12 Pro Max. I’ve tried both Discourse Hub app and PWA.

Edit: changed the title for clarity.

2 Likes
2

Did you try disabling theme customizations with safe mode?

2 Likes
3

Boom, that’s the case. How do I find out what breaks the chat, which works fine in Safe Mode?

2 Likes
4

How did you add that section below the header which is also visible in chat?

1 Like
5

It’s Branded Header theme component.

6

Disabling Brand Header component fixed the issue.

Solution is to disable displaying the Brand Header on mobile for now.

Thank you @Moin for reminding me of the Safe Mode in the first place.
Thank you @Lilly for adding the proper tags to this Bug report.

2 Likes
7

I was going to reply but @Moin was doing a good job of reading my mind and replying faster :sweat_smile:

I’ll get someone to have a look at this. thanks for letting us know.

4 Likes
9

I’m questioning if a brand header should appear on chat to begin with tbh. We make a distinction between forum/chat, so when you pivot to chat “mode”, it makes sense to me to not show the forum header.

I think I’ll follow that solution route for now.

2 Likes
11

Done in UX: hide component on mobile chat pages by chapoi · Pull Request #54 · discourse/discourse-brand-header · GitHub

2 Likes
12

Is the change reflected on meta yet?

This also happens on Meta right now because of the Job ad banner.

With the Job ad un-dismissed Chat Ui breaks on mobile and is useless.

Dismiss the job ad and the problem goes away.

2 Likes
13

Hm, you are right :slightly_frowning_face: We’re looking into a more comprehensive solution for this atm.

1 Like