Sorry, no I didn’t do a pull request. Just commenting that the original pull request probably won’t work.
It's hard to tell if you haven't also applied the js changes if this is a bug with the PR or not:
Original solution above, using fixed, for me completely hides composer now.
This may be best. This ensures the composer is at the bottom, similar to how it is on desktop.
That looks nice on your mobile screen, but how does it behave on smaller mobile screens where there is no space for both the keyboard and the composer ? I imagine you wouldn't be able to see the title field anymore.
Also, fixed positioning fixes a problem where the composer is stuck at a specific position in the screen, creating issues when closing the keyboard and other scenarios: https://discuss.flarum.org/d/24831-ui-bug-mobile-postdiscussion-is-not-on-the-screen-sometime
Another way of looking at this is: Unless a better fix can be found relatively quickly.. Should Flarum be a good experience on super tiny screens or a good experience on very popular iPhone screens?
I don't understand what you mean, we are not forced to choose between both, this pull request fixes all issues.
I’m sure there are really good reasons. But, I wonder why the post box on mobile isn’t just like the post box on desktop? Where it is responsive and slides up into view.