-
Notifications
You must be signed in to change notification settings - Fork 1.9k
medium-editor-insert plugin css fixes on beagle theme #1361
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
Conversation
…ium-editor-insert-plugins´ buttons and toolbar - On the original one it would change to a square button on hover, and the last button wouldn't show correctly either
Changes Unknown when pulling 3fb46a6 on mnussbaumer:beagle_css_uploader into ** on yabwe:master**. |
Could you provide a jsfiddle reproducing the bug? |
@j0k3r do you have any working fiddle for medium-editor alone? I have screenshots, but no really sure how I can go on showing this on a fiddle. |
From a previous issue: https://jsfiddle.net/4xdom6ar/ |
Thank you @j0k3r, |
Could you please explain step by step what should I do to reproduce the bug using your jsfiddle? |
@j0k3r yes, click on the "Lorem ipsum..." editable content. Go to the end of the line, hit carriage return to go to a new line. It will show you the ´+´ sign from the insert plugin. Click the plus, it will show you another 2 circles, click the first one to upload a picture. Choose a picture from your local drive, select. It will display the picture on the editor area. Now click the picture itself to show the plugin's toolbar & the ´x´ icon on it. Hover over the ´x´ icon and the right-most item on the plugin toolbar, you'll see the border-radius change between them. |
And the plugin isn't attributing classes for the first and last item in the nav bar so I might as well do a pr on the plugin's repo to fix that, still, the ´x´ close icon isn't on an ordered list (meaning it's a single item) so it will not be fixed by that. |
src/sass/themes/beagle.scss
Outdated
|
||
.medium-editor-toolbar-actions { | ||
li, button { | ||
border-radius: 50px; |
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.
You shouldn't use the hardcoded value 50px
but $medium-editor-border-radius
instead
…medium-editor-toolbar-actions
Changes Unknown when pulling 137c8d7 on mnussbaumer:beagle_css_uploader into ** on yabwe:master**. |
Added .scss and dist/.css changes for beagle theme to accomodate ´med…ium-editor-insert-plugins´ buttons and toolbar - On the original one it would change to a square button on hover, and the last button wouldn't show correctly either
Description
When using medium-insert-plugin directly with beagle theme on medium-editor, buttons on the toolbar of medium-insert-plugin and the delete button (both appear when an image has been added to the dom for upload) would change border-radius on hover, showing squares and obviously looking wrong.
--