Support » Plugin: WPIDE - File Manager & Code Editor » Various Issues

  • Resolved Oliver Campion

    (@domainsupport)


    Hi,

    The new version of WPIDE has some issues that the previous incarnation did not have.

    1. Scrolling long sections of code is very slow
    2. Drag selecting sections of code that requires scrolling further down the page is not possible. You can only drag select upwards.
    3. A fatal-error-handler.php drop-in plugin is installed automatically when you activate. I’m not sure why this is required (WordPress already handles errors) but forcibly installing a drop-in and then hiding it from your own file explorer is a bit suspicious?
    4. You are using your own ajax.php. WordPress already has admin-ajax.php which works brilliantly … why would you open up another potential security risk?
    5. Navigating tabbed space at the beginning of a line of code would jump across tabs before, now it very slowly moves space by space
    6. When searching, you used to be able to press return to find the next match. Return does nothing now.

    Please advise.

    Thank you,

    Oliver

Viewing 14 replies - 1 through 14 (of 14 total)
  • Plugin Author XplodedThemes

    (@xplodedthemes)

    Hi Oliver,

    Thanks for your feedback!

    1. I am not able to reproduce this from my end. How many lines of codes are you trying with ? https://d.pr/v/ppn5zh

    2. Same with this: https://d.pr/v/aEyMJw

    3. The problem is, if you get a fatal error after modifying a PHP file, then you won’t have access to the backend at all. The only way to fix the problem is by FTP. With our drop-in, it makes it much easier. Whenever a fatal error is triggered, our file recovery wizard appears. It will show you the error message and error line, as well as the recovery option. With a hit of a button, your site is back online. The drop-in is uninstalled automatically when the plugin is disabled.

    4. Yes, the ajax.php file is the exact copy of WordPress admin-ajax.php, so no security issues at all. However, it will be faster, since it is dedicated to our plugin and performance won’t be affected if let say a site has many active plugins that use Ajax.

    5. I am not sure if I understand this, I am able to use the tab correct from my end: https://d.pr/v/nAdYQU . What OS / browser / version are you using ?

    6. Yes, thanks for reporting, this will be implemented soon. For now, the search works by clicking on the search icon.

    Thank you

    Georges

    Thread Starter Oliver Campion

    (@domainsupport)

    Thank you for the fast reply!

    OK, so …

    1. I’m using a MacBook Pro MacOS v12.4 2.3 GHz Quad-Core Intel Core i5 with Safari. To scroll I’m just using two finger gesture on the trackpad. Several thousand lines of code but that doesn’t seem to make a difference to the speed.
    2. I see in your video it looks like you are scrolling with a mouse wheel to click drag? I’m using a single click-hold on the Mac trackpad and then a second finger moves the mouse to the far bottom of the screen which usually scrolls content down but not with WPIDE. This can probably be replicated with a mouse by click-hold and then moving the mouse to the bottom of the screen but I don’t have a mouse to hand to try.
    3. Fair enough, I look forward to using that feature. I saw that it was removed on deactivation.
    4. I’m not sure I understand your reasoning with regards to ajax.php as I’m not sure why multiple use of admin-ajax.php by third party code would have any kind of performance issue (other than CPU usage which would surely affect all code execution on the server). Personally I’ve never seen a situation where the default admin-ajax.php wasn’t sufficient and adding another one (even an exact copy) seem superfluous to me.
    5. In your video you are using the tab key to increase tabs. That’s working. I’m talking about navigating existing tabs using the arrow keys. Before a right or left arrow key press would advance or go back a whole tab space; now it only goes forward or backward a single space.
    6. Great

    Thanks.

    Oliver

    Thread Starter Oliver Campion

    (@domainsupport)

    One more observation … when /wp-content/plugins/wpide/ajax.php is blocked (it’s common to block direct access to php in /wp-content/), WPIDE goes into an infinite loop trying to reload over and over.

    Oh, and in the file explorer, files are in alphabetical order but directories are not.

    Oliver

    Thread Starter Oliver Campion

    (@domainsupport)

    Three more things …

    1. With (5) above, deleting tabbed indents also often takes longer. Sometimes it works but more often the delete key will just delete a space rather than the whole tab
    2. The interface now automatically adds closing brackets and html tags. Can this be turned off in settings?
    3. Before, if you selected a bit of text and then pressed “Apple-F” to find something, it would automatically pre-populate the search box with the selected text. This no longer happens.

    Thank you,

    Oliver

    Thread Starter Oliver Campion

    (@domainsupport)

    Three more …

    1. When editing a file, you used to be able to right click on a directory to rename it or turn it into a .zip archive. This is no longer possible.
    2. In the File Manager you can click the three dots on a directory to make a zip file but it used to take the name of the directory and add “.zip”. Now it just calls it “archive.zip” which you then have to manually rename.
    3. When downloading the zip file, it used to stay on the same page. Now a new blank tab is opened (Safari).

    Thank you,

    Oliver

    Plugin Author XplodedThemes

    (@xplodedthemes)

    Hi Olivier,

    Thank you for these detailed observations 🙂

    Happy to say, we just released a new update that fixes them all! (v3.4)

    From the first comment:

    1. Added a scroll speed option within the settings (New File Editor setting tab), the previous version was set to 1, now the default is 3, and you can adjust as you wish.

    2. Fixed

    4. Fair enough, we switched back to the native admin-ajax.php

    5. Got it now, we added a new setting called “Use Soft Tabs” that use spaces instead of the tab character, it was enabled by default, and it shouldn’t.

    From the second comment:

    1. Fixed with the Soft Tabs disabled option

    2. Added an option to disable Editor Behaviors

    3. Added

    From the third comment:

    1. Added item menu on hover, you will be able to download, zip, unzip and rename from the file tree.

    2. Done. The default archive name, which can be changed within the settings, will only be applied when batch zipping or downloading. For a single item, the item name will be used by default instead

    3. Fixed

    Also added a couple more options, such as:
    – Editor font size,
    – Tab size
    – Highlight Active Line

    Note: Make sure to flush any caching plugins or CDN after updating.

    Let me know if all this works fine for you.

    Cheers

    Thread Starter Oliver Campion

    (@domainsupport)

    This is fantastic news, thank you.

    I will fully test and get back to you.

    Brilliant support.

    Oliver

    Thread Starter Oliver Campion

    (@domainsupport)

    OK guys, I’ve had a play and here are my comments …

    1. The interface remembers where you were last time it was open but if that folder has been subsequently deleted, it doesn’t take you to the next folder up (or even that the folder doesn’t exist), instead it says “Folder Empty”.
    2. PHP – Many ISPs don’t yet allow sites to be upgraded to PHP v7.4 (crazy!); Media Temple, for example, only go as high as v7.3.11. This obviously isn’t ideal but is there any way you can make WPIDE v2.6 backwards compatible with PHP v7.3?
    3. Tab traversal is better, it now works when a tab is being traversed but with some code the tab is made up of a bunch of spaces and WPIDE used to treat these as tabs and would traverse them as such but these are still traversed space by space with the arrow keys or the delete key
    4. Click Drag Scrolling – This still doesn’t work when scrolling down. To replicate, click and hold text in the middle of the screen then move the arrow as far down to the bottom of the screen as you can. Usually the editor would scroll down but it doesn’t.

    Many thanks,

    Oliver

    Plugin Author XplodedThemes

    (@xplodedthemes)

    Hi Oliver,

    Thanks for the comments!

    1. I will pass along this to the dev team
    2. Unfortunately, the current code base will not support anything below PHP 7.4. Also, WordPress.org recommends at least PHP 7.4, so hosting providers will have to start updating pretty soon. As for v2.6, I am not sure if it supports 7.3, probably not fully. However, we do not support it. We prefer going forward instead of patching old versions.
    3. I am not able to replicate that from my end, are you sure the soft tabs option is disabled ?
    4. Same with this, I tried with a mouse as well as with my MacBook Pro trackpad, they both work when selecting text and scrolling down at the same time.

    If you can provide screenshots or screencasts for #3, #4 would be great!

    Thank you

    Thread Starter Oliver Campion

    (@domainsupport)

    Understood. I’ve just rechecked and v2.6 is still as it was which worked perfectly with PHP v7.3 so that’s great. The confusion has come because the new version is specifying PHP v7.4 only. Yes, you’d think that ISP’s would upgrade to PHP v7.4 especially seeing as WordPress now requires but sadly that is not the case with some of our clients’ ISPs.

    If I turn on soft tabs or if I turn off soft tabs, existing indents that are made up of spaces are still traversed one space at a time with arrow keys or with the backspace key. The issue I’m having is with pre-existing tabs made up of spaces.

    Turning off soft tabs just means that new indents are made of the tab character (which are traversed correctly).

    Here’s a video showing scrolling working fine going up, but not going down https://webd.uk/no-scroll-down.mp4

    Oliver

    Plugin Author XplodedThemes

    (@xplodedthemes)

    Hi Oliver,

    For the soft tabs, I think this is normal, I mean if the existing code indentation was made using spaces, the editor will not change that.

    The scrolling issue, man it’s driving me crazy :/ I am using the MacBook Pro trackpad as well, and it’s working fine. I can select a text and drag it down while scrolling.

    I am wondering if it’s related to system settings ? Maybe this:
    https://d.pr/i/2YJ5rc

    But again, you said that it worked with the previous version, so back to square one 😀

    Thread Starter Oliver Campion

    (@domainsupport)

    Hi,

    Thanks for your continued help!

    With regards to the tabs, if you roll back to v2.6 you’ll see that multiple spaces (I think it’s four) are jumped / deleted when you press arrow keys / backspace; so they are treated like a tab.

    Also on this version, the behaviour is for the tab key to inject a bunch of spaces. You can see that this is what’s happening because you can click to position the cursor in the middle of this “virtual tab” and, if you do, the use of arrow keys / backspace reverts to the normal space by space behaviour until the beginning or end of a “virtual tab” is then reached.

    This could well not be how it’s meant to be but it’s how the plug-in used to behave and was super useful for traversing tabbed code. It may not seem like much but it really speeds things up when having to tidy messy code.

    I’m going to get back to you with the scrolling. I suspect it’s something to do with the position of the input area so close to the bottom of the screen because it’s almost like the browser doesn’t register that the pointer has reached beyond the text input area.

    Oliver

    Thread Starter Oliver Campion

    (@domainsupport)

    OK … so I’ve worked out why you can’t replicate the scrolling issue 🙂

    My browser window fills the entire screen. And actually you can replicate the problem easily (Safari) by entering full screen mode (fn-F).

    This CSS resolves the issue but you will probably find a better location for the padding-bottom or you might prefer to use margin-bottom or the height property.

    .ace_editor {
    padding-bottom: 1em;
    }

    So only the behaviour of pre-existing spaces as tabs is now left and I really hope you can revert to the way v2.6 dealt with this!

    Thanks,

    Oliver

    Thread Starter Oliver Campion

    (@domainsupport)

    I’m going to mark this one as resolved and start a new thread because I’ve found a lot more issues.

    Oliver

Viewing 14 replies - 1 through 14 (of 14 total)
  • You must be logged in to reply to this topic.