Kademi / keditor

KEditor is a jQuery plugin which provides a content editor with drag n drop, configurable contents
http://kademi.github.io/keditor/
MIT License
319 stars 153 forks source link

[BUG] Bootstrap 4 Column Resize #194

Closed hitaloramon closed 5 years ago

hitaloramon commented 5 years ago

Description: The previous bug reported has been fixed in part. I realized that when resizing a column in Desktop view and then switching to Laptop, tablet or mobile view, the same column can no longer be resized, but the others work normally. Using the bootstrap 3 example this does not occur.

How to reproduce the problem: 1 - Resize a column in Desktop View 2 - Switch to Laptop View 3 - Try resizing the same previous column

Live Example: https://www.cmsgeek.m7web.com.br/bootstrap4.html

hitaloramon commented 5 years ago

Any news on this? @ducdhm

ducdhm commented 5 years ago

@hitaloramon The main issue here is styles of BS4. I'm investigating...

hitaloramon commented 5 years ago

@ducdhm, I think find out why this problem happens. The bootstrap 4 grid system has an additional class prefix, which is col-xl for desktop, and when resizing it changes in a lower view. So I tried changing the device constants to:

const DEVICE_MODE = {      MOBILE: 'sm',      TABLET: 'md',      LAPTOP: 'lg',      DESKTOP: 'xl', };

and it seems that solved the problem.

ducdhm commented 5 years ago

@hitaloramon I think the DEVICE_MODE need to be configurable. Let me update it in keditor

ducdhm commented 5 years ago

New options: image