New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UX: Maximize the preview space in composer #15188
Conversation
This pull request has been mentioned on Discourse Meta. There might be relevant details there: |
# Conflicts: # app/assets/stylesheets/mobile/compose.scss
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pulled the branch locally and we now have some inconsistencies (not necessarily all due to this PR, specifically).
Composer input and dropdown border colors don't match border colors elsewhere:
Focused border size is 1px for input/dropdown, 2px for textarea:
I think we should stay consistent here, I am struggling to see the justification for why we want to introduce a different aesthetic to only the inputs/dropdowns in the composer.
I restored original input border colors and fixed focus border on the title input. I tried it out with the following:
|
# Conflicts: # app/assets/javascripts/discourse/app/templates/composer.hbs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure whether the test failures are pertinent, but otherwise, the last few changes look good.
A follow-up to #15117 and #15141. Applies the previous changes to PM-specific fields, makes the preview area take the all the available height of the composer, and unifies more spacing between composer elements.
Since this PR changes
composer
,composer-editor
andd-editor
templates, some theme components need to be updated:d-editor
composer-editor
: discourse-googlebooks (not maintained anymore?) and discourse-basic-editorcomposer
: discourse-basic-editor and one internalI also have to do a spot check of random (popular) themes to see nothing regressed there.