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

Switch from codemirror to ace.js #308

Closed
pabloko opened this issue Dec 22, 2015 · 10 comments
Closed

Switch from codemirror to ace.js #308

pabloko opened this issue Dec 22, 2015 · 10 comments

Comments

@pabloko
Copy link

pabloko commented Dec 22, 2015

Hey, i've worked on original codemirror implementation for filemin. as i see now authentic theme handles the assets for codemirror so i post this issue here.

I dindt had the good experience i've expected of codemirror, recently switched some projects to ace.js that is working way better and has a lot more lexers support and better integration.

Maybe switching it could be a good idea...

Ive also noticed that codemirror isnt working on standalone filemin access (no ajax) i'd like to have lexer on non ajaxed version, since its almost impossible to use the ajaxed one (works very slowly, almost unusable) maybe this settings could be configured on theme preferences

regards

@iliajie
Copy link
Collaborator

iliajie commented Dec 22, 2015

I also like Acejs but CodeMirror works also very stable for me in all browsers. Isn't it for you too?

However, I didn't understand what you meant about no ajax at the last paragraph?

@pabloko
Copy link
Author

pabloko commented Dec 22, 2015

Well, when you open filemin or edit a file in a new tab codemirror wont load. Also ajax loading not working. (fe. you access to webmin.url:10000/filemin)

For me its important the ability of editing several files at same time, thats why I always edit files on a new tab. Maybe having the ability of minimize and maximice edit dialogs could do the trick...

@iliajie
Copy link
Collaborator

iliajie commented Dec 22, 2015

That is for sure, I will make tabs and minimization in the future.

So far, right clicking on the top left switch will create new window of Webmin.

.. and yes, you should open the whole thing yo make scripts work properly.

@Real-Gecko
Copy link

I'm planning to create another Webmin module that will be central point for storing and managin all JS and CSS stuff. So maybe when it will be done selecting desired editor will become just a one click deal :)

@iliajie
Copy link
Collaborator

iliajie commented Dec 22, 2015

...ohh. That will be difficult!

@Real-Gecko
Copy link

Maybe :)

@iliajie
Copy link
Collaborator

iliajie commented Dec 23, 2015

Maybe having the ability of minimize and maximice edit dialogs could do the trick...

Will be added to 17.21

@iliajie
Copy link
Collaborator

iliajie commented Dec 23, 2015

+ hotkeys:

  • Added new hotkeys for Filemin:
    • Meta+F4 - close active editor window
    • Meta+Shift+F4 - minimize active editor window

@iliajie iliajie closed this as completed Dec 23, 2015
@pabloko
Copy link
Author

pabloko commented Dec 30, 2015

Superb project and owner, many thanks @qooob for your effort and quick enhancement craft. 10/10 will donate something by end of the month!

@iliajie
Copy link
Collaborator

iliajie commented Dec 30, 2015

Your are very welcome! :) Thank you and a Happy New Year!

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

No branches or pull requests

3 participants