SimpleMDE plugin issues in kibry 2.5.8 making panel field super wide

Kirby 2 Issues šŸ›
1

i found a bugā€¦ im using kirby version 2.5.8

When SimpleMDE field is on, it makes the whole edit area super wide.

22 PM

2

I also use the SimpleMDE field with Kirby 2.5.8 but I canā€™t reproduce your issue. What other plugins/fields do you have in this blueprint? Could you paste the blueprint content here?

2-5-8

3

i found itā€¦ if i add long text in textarea.

try this

<iframe src="https://www.google.com/maps/embed?pb=!1m18!1m12!1m3!1d3316.375740032557!2d151.05140795051838!3d-33.77679398058708!2m3!1f0!2f0!3f0!3m2!1i1024!2i768!4f13.1!3m3!1m2!1s0x6b12a405e7c99613%3A0xa2c479e698c86a74!2sCarlingford+Chiropractic+Centre!5e0!3m2!1sen!2sau!4v1508907657756" width="100%" height="488" frameborder="0" style="border:0" allowfullscreen></iframe>
4

I think this is a similar issue to that one:
https://github.com/lukaskleinschmidt/kirby-sortable/issues/37#issuecomment-326210365

Could you try the panel.css fix?

EDIT:
I tested it by adding the following to my panel.css:

.form > fieldset {
  min-width: 0;
}

This works and the field will no longer be super wide.

Iā€™m afraid I canā€™t really fix this issue with my plugin. There is a pull request by @lukaskleinschmidt fixing the panel issue. But I donā€™t have any information about the reasons why it hasnā€™t been merged.

1 Like
5

@texnixe @bastianallgeier some of us are watching this thread with great interest - Iā€™ve just replaced Enhanced Textarea with SimpleMDE in all my projectsā€¦ :flushed:

6

Hm, I donā€™t have an answer to why nothing has happened since the PR was added to the 2.5 milestone almost a year agoā€¦:thinking: Wondering why the minified js files have been changed?

7

that one works too @thguenther

 body.ltr .field-grid {
   min-width: 0px;
 }

thanks

8

I added a comment in the GitHub issue. :slight_smile:

Probably the commit that @lukaskleinschmidt based his PR on didnā€™t have properly compiled JS so another change got included in the PR. But thatā€™s alright, weā€™d need to merge this PR manually anyway because of the merge conflicts in the other build files.