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
Markdown Interface Updates #9267
Conversation
- "Editor" button text fixed to "Edit" - Edit and Preview buttons stay highlighted based on current mode - Added interface option for Editor and Preview Font type - Copied WYSIWYG CSS correct stylings to Markdown - Added custom block option placeholders for each input - Changed icon heading to use format_size icon - Added toolbar controls
- Edit translation string for name in placeholder - Wrapped preview hidden toolbars with a <template v-if> - Changed .indexOf to .includes - Added translation strings for Edit and Preview
@rclee91 Thanks for the prompt commit to resolve the requested changes! Also appreciate the "drive-by" changes to use the CSS variables where applicable. |
Thanks @rclee91!! We'll review! ❤️ |
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.
Ahh but those aren't the correct fonts though. We should make sure it's using the var(--family-sans-serif) et al. fonts to match the design language of the app 👍🏻 |
- "Editor" button text fixed to "Edit" - Edit and Preview buttons stay highlighted based on current mode - Added interface option for Editor and Preview Font type - Copied WYSIWYG CSS correct stylings to Markdown - Added custom block option placeholders for each input - Changed icon heading to use format_size icon - Added toolbar controls
- Edit translation string for name in placeholder - Wrapped preview hidden toolbars with a <template v-if> - Changed .indexOf to .includes - Added translation strings for Edit and Preview
Newest commit 874783a gives these styles now. |
Worked on issue #9167
Fixes #9167