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

Addition the menubar position in props #133

Closed
wants to merge 3 commits into from
Closed

Conversation

isv9
Copy link

@isv9 isv9 commented Jun 30, 2020

Hi, I needed to set menubar in the bottom position and I did it by css order. Can you look it and if you think that is ok you will add it to your project. Thanks

Repository owner deleted a comment from isv9 Jun 30, 2020
Repository owner deleted a comment from isv9 Jun 30, 2020
Repository owner deleted a comment from isv9 Jun 30, 2020
@Leecason
Copy link
Owner

Leecason commented Jul 2, 2020

Thank you for your pr❤️.
I think editorClass, editorContentClass, editorMenubarClass props also can reposition the menubar, instead of menubarPosition . I don't want to add menubarPosition prop, it may be difficult to add left and right menubar in the future.

@isv9
Copy link
Author

isv9 commented Jul 2, 2020

You can add menubarPosition with ['top', 'bottom'] only and don't add 'left' and 'right'

@Leecason
Copy link
Owner

Leecason commented Jul 3, 2020

Does editorClass, editorContentClass, editorMenubarClass can implement position menubar to bottom? At the moment I don't want to add menubarPosition prop to element-tiptap.

Leecason added a commit that referenced this pull request Jul 5, 2020
Leecason pushed a commit that referenced this pull request Jul 5, 2020
# [1.25.0](https://github.com/Leecason/element-tiptap/compare/@[email protected]) (2020-07-05)

### Bug Fixes

* 🎨 type definings ([9cc1274](9cc1274))
* 🐛 show bubble menu when all select (#97) ([b495bec](b495bec))

### Features

* ✨ add `defaultWidth`, `defaultDisplay` options for `Image` extension ([64bc1b6](64bc1b6))
* ✨ add `editorClass`, `editorContentClass`, `editorMenubarClass`, `editorBubbleMenuClass`, `editorFooterClass` props (#133) ([d43d61e](d43d61e))
* ✨ add `lang` prop to support config editor i18n language locally (#62) ([e28b6bd](e28b6bd))
* ✨ add `openInNewTab` option to `Link` extension, default to `true` ([f66ff5e](f66ff5e))
* ✨ add `tooltip` prop to enable/disable tooltip (#121) ([08db78e](08db78e))
* ✨ Add new language `zh-tw` ([be52f57](be52f57))
* ✨ default export `ElementTiptapPlugin` ([bf1726f](bf1726f))
* ✨ enable back to default menu in link bubble menu (#117) ([1805987](1805987))
@Leecason
Copy link
Owner

Leecason commented Jul 5, 2020

I add editorClass, editorContentClass, editorMenubarClass, editorFooterClass props, is these help you?

@isv9
Copy link
Author

isv9 commented Jul 6, 2020

yes, thanks

@Leecason Leecason closed this Aug 5, 2020
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.

2 participants