Skip to content
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

medium-editor-insert plugin css fixes on beagle theme #1361

Merged
merged 4 commits into from
Aug 16, 2017
Merged

medium-editor-insert plugin css fixes on beagle theme #1361

merged 4 commits into from
Aug 16, 2017

Conversation

mnussbaumer
Copy link
Contributor

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

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
New tests added? not needed
License MIT

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.

--

…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
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 3fb46a6 on mnussbaumer:beagle_css_uploader into ** on yabwe:master**.

@j0k3r
Copy link
Contributor

j0k3r commented Aug 15, 2017

Could you provide a jsfiddle reproducing the bug?
Thanks.

@mnussbaumer
Copy link
Contributor Author

@j0k3r do you have any working fiddle for medium-editor alone?
I've tried setting JS to be jQuery, added medium and stylesheets as external resources but it doesn't seem to be able to load them? And then jquery upload doesn't really have a CDN dist, nor does medium-insert.

I have screenshots, but no really sure how I can go on showing this on a fiddle.
Thanks

@j0k3r
Copy link
Contributor

j0k3r commented Aug 16, 2017

From a previous issue: https://jsfiddle.net/4xdom6ar/

@mnussbaumer
Copy link
Contributor Author

Thank you @j0k3r,
https://jsfiddle.net/4fzwcftx/1/

@j0k3r
Copy link
Contributor

j0k3r commented Aug 16, 2017

Could you please explain step by step what should I do to reproduce the bug using your jsfiddle?

@mnussbaumer
Copy link
Contributor Author

@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.

@mnussbaumer
Copy link
Contributor Author

mnussbaumer commented Aug 16, 2017

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.


.medium-editor-toolbar-actions {
li, button {
border-radius: 50px;
Copy link
Contributor

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

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 137c8d7 on mnussbaumer:beagle_css_uploader into ** on yabwe:master**.

@j0k3r j0k3r merged commit 462fff0 into yabwe:master Aug 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants