Sign in to follow this  
Followers 0

Bring back the 3.1 standard editor as an option

139 posts in this topic

Posted

Apologies in advance for the length of this post.

It seems to me that most people who are seriously unhappy with the switch to CKEditor - including myself - are primarily users of the non-RTE mode.

The standard mode of the 3.2 editor has so far proved to be very buggy - I personally have submitted 8 different bug reports in the last 3 weeks related to it. Most serious was a bug involving the img tag which allowed an attacker to insert any arbitrary Javascript which would run whenever the post was viewed (you won't see this bug in the tracker as it was made private - it is fixed in 3.2.3).

Bugs aside, we also have the complete removal of bbcode insertion buttons in the standard mode. You have to type every single tag manually. I haven't used a forum editor for many years where this was required.

Also, the autosave function which was touted as one of the major advantages of the new editor, does not function in standard mode. I'm not sure if this is intended or a bug - I've submitted an issue in the tracker to find out.

I know there were many complaints about the RTE mode of the 3.1 editor. However, I'm not aware of any complaints from users of the standard bbcode mode. Personally I was extremely happy with it, the buttons saved me a lot of time and I never experienced any problems with it. My members were happy too, and all knew how to use it because the concept of bbcode has been standard across forum software for many years.

In short, there's are no advantages to the 3.2 editor for standard mode users, and many disadvantages.

The primary issue given in Matt's blog back in February for changing to CKEditor was the difficulty of maintaining and developing a rich text editor with constantly evolving browser standards. So the decision does seem to have been driven by the RTE mode.

My suggestion therefore, is this. Bring back the 3.1 standard editor, without its RTE mode.

Make it possible for admins to choose between using CKEditor, with its two modes, and the 3.1 bbcode-only editor as the default option. Let members choose too in their User CPs. Those happy with CKEditor could continue to use it, while those who are upset by all its missing functionality and bugginess can switch back.

IPS wouldn't have to maintain its own proprietary rich text editor as this mode would only be available for those choosing to use CKEditor. And presumably the amount of development involved in making the standard 3.1 editor compatible with IPB 3.2 would be minimal, compared to further hacking CKEditor to support raw bbcode insertion.

I would be more than happy to sacrifice the ability to switch between standard and RTE modes in the editor if it meant I could get all the other 3.1 functionality back. It seems like all the development time needed to adapt the 3.2 editor to restore full bbcode functionality is a waste when you could probably satisfy most complainants by simply integrating the 3.1 standard editor and giving admins this option.

Please at least consider this. Not everyone wants to use a rich text editor on forums.

RLKP, PrinceOfAbyss, bosss and 8 others like this

Share this post


Link to post
Share on other sites

Posted

[quote name='insectdude' timestamp='1316873717' post='2174751']
*snip

YES.... as a coder... ive NEVER had any use for ANY RTE... all it amounts to is an annoyance... CKE triply so...
1000% agreed.... 3.1.x I never had to be afraid of codebox usage.... let the folks have their glorified garbage chomping RTE... give us back the WORKING simple bbcode editor.

PrinceOfAbyss likes this

Share this post


Link to post
Share on other sites

Posted

No thanks. Like anything, it just takes time to mature. CKE in 3.2.3 is the best one yet as far as I can tell.

Oh, and it means more debugging for making apps that use the editor - it's already a pain to get working right!

Marcher Technologies and AndyF like this

Share this post


Link to post
Share on other sites

Posted

[quote name='Lewis P' timestamp='1316874604' post='2174760']
No thanks. Like anything, it just takes time to mature. CKE in 3.2.3 is the best one yet as far as I can tell.

Oh, and it means more debugging for making apps that use the editor - it's already a pain to get working right!

...no argument sadly... it IS a flat pain to get working proper... can we win for losing???
can we get a TRUE STD editor out of the CKE(AFAIK STD!=no bbcode buttons)?

Share this post


Link to post
Share on other sites

Posted

[quote name='Marcher Technologies' timestamp='1316874754' post='2174762']
can we get a TRUE STD editor out of the CKE?

I haven't personally even looked at the CKE source, but I don't see why it couldn't be made to work correctly - even if it takes some work and lots of maintenance, anything is possible.

Share this post


Link to post
Share on other sites

Posted

[quote name='Lewis P' timestamp='1316874971' post='2174764']
I haven't personally even looked at the CKE source, but I don't see why it couldn't be made to work correctly - even if it takes some work and lots of maintenance, anything is possible.

I have... its... eesh... a mess..
CKE in IPB is run through IPB internal bbcode processing...
Which is precisely why even going through all the motions of "enabling" a CKE advanced feature wont help unless its in the IPB core.... which from what im being told, is a similar issue to the simple buttons gone poof.

Share this post


Link to post
Share on other sites

Posted

[quote name='Lewis P' timestamp='1316874604' post='2174760']
Like anything, it just takes time to mature.

It's nearly 4 months and 3 minor releases since 3.2 was launched and I've yet to see even an acknowledgement by IPS that the missing standard mode functionality will be added back (specifically the bbcode insertion buttons). They have many other things to be working on, such as all the addon apps. The 3.1 standard editor is already mature and proven. Why duplicate the effort? How long will we have to wait for it to happen?

PrinceOfAbyss and EBrown like this

Share this post


Link to post
Share on other sites

Posted

[quote name='insectdude' timestamp='1316875304' post='2174768']
It's nearly 4 months and 3 minor releases since 3.2 was launched and I've yet to see even an acknowledgement by IPS that the missing standard mode functionality will be added back (specifically the bbcode insertion buttons). They have many other things to be working on, such as all the addon apps. The 3.1 standard editor is already mature and proven. Why duplicate the effort? How long will we have to wait for it to happen?

while not on a user-by-user basis... i do happen to know if twas a 3.1.x upgrade... the code is still there... in fact most apps have had to manually switch the call for 3.2.x... just saying, one could viably swap it back(though youd have to hunt down every swap to cke and revert the code)

Share this post


Link to post
Share on other sites

Posted

[quote name='Marcher Technologies' timestamp='1316875469' post='2174769']
while not on a user-by-user basis... i do happen to know if twas a 3.1.x upgrade... the code is still there... in fact most apps have had to manually switch the call for 3.2.x... just saying, one could viably swap it back(though youd have to hunt down every swap to cke and revert the code)

That would make my site unsupported by IPS though, which is not something I'm willing to do at the moment. I'm not confident enough in my coding ability to make it happen either, sadly.

Share this post


Link to post
Share on other sites

Posted

[quote name='insectdude' timestamp='1316875645' post='2174770']
That would make my site unsupported by IPS though, which is not something I'm willing to do at the moment. I'm not confident enough in my coding ability to make it happen either, sadly.

never said I would recommend doing that... simply stating it is "possible"

Share this post


Link to post
Share on other sites

Posted

[quote name='Marcher Technologies' timestamp='1316875714' post='2174771']
never said I would recommend doing that... simply stating it is "possible"

Yep and I appreciate the info. Maybe if it becomes clear that the old functionality is not coming back I might have to consider this more seriously.

Share this post


Link to post
Share on other sites

Posted

[quote name='insectdude' timestamp='1316875853' post='2174772']
Yep and I appreciate the info. Maybe if it becomes clear that the old functionality is not coming back I might have to consider this more seriously.

This is feedback of prime... a LOT of us have NO use for an RTE... fix "simple" mode please... buttons must be enabled.... pretty certain THAT simply would be answered with glee by customers opposed to CKE.

Share this post


Link to post
Share on other sites

Posted

oh... and this:

is flat unacceptable....
Never before have i been unwilling to use a codebox.... which is saddening... as without it proper syntax highlighting vanishes.


$this->registry->getClass('output')->buildUrl('blah', 'publicWithApp')

Share this post


Link to post
Share on other sites

Posted


oh... and this:

$this->registry->getClass('output')->buildUrl('blah', 'publicWithApp')

is flat unacceptable....
Never before have i been unwilling to use a codebox.... which is saddening... as without it proper syntax highlighting vanishes.

as a note... i did NOT doctor that
'

'

Share this post


Link to post
Share on other sites

Posted

I think that issue with using the ' character in the code box is probably a result of the fix to the img tag JS injection bug I mentioned.

Share this post


Link to post
Share on other sites

Posted

[quote name='insectdude' timestamp='1316877065' post='2174783']
I think that issue with using the ' character in the code box is probably a result of the fix to the img tag JS injection bug I mentioned.

so 1 step forward... 2 back with every "Fix".... Awesome.

Share this post


Link to post
Share on other sites

Posted

Maybe this is JUST ME, but i feel the horse is being placed in FRONT of the cart here...
STD integration should be a higher priority than RTE....
AS once the STD is sorted... the RTE NATURALLY follows suit.

Share this post


Link to post
Share on other sites

Posted

On a similar theme I don't think ease of maintenance or development should be placed ahead of functionality or ease of use.

There's no point having an editor that's easy to maintain and integrate if it doesn't have the features that users want. That's a common mistake in software development. Users should come first.

Share this post


Link to post
Share on other sites

Posted

[quote name='Marcher Technologies' timestamp='1316877720' post='2174785']
Maybe this is JUST ME, but i feel the horse is being placed in FRONT of the cart here...
STD integration should be a higher priority than RTE....
AS once the STD is sorted... the RTE NATURALLY follows suit.

Share this post


Link to post
Share on other sites

Posted

[quote name='Aisha' timestamp='1316881265' post='2174799']
Pardon the tangent but...isn't that where you'd want a horse to be? O.o

totally meant reverse... cart in front of horse.

Share this post


Link to post
Share on other sites

Posted

The CKEditor seems like a step backwards in some respects and a step forwards in others, but overall I prefer it to the previous editor despite there being some annoying bugs and issues. I recommend that IPS modify the CKEditor until all outstanding bugs and major issues are resolved.

cmh and NewRockRabbit like this

Share this post


Link to post
Share on other sites

Posted

[quote name='+++' timestamp='1316889041' post='2174849']
The CKEditor seems like a step backwards in some respects and a step forwards in others, but overall I prefer it to the previous editor despite there being some annoying bugs and issues. I recommend that IPS modify the CKEditor until all outstanding bugs and major issues are resolved.

now... that is sad... third-party editor is causing more work... not less.... hindsight 20/20 and all that.

Share this post


Link to post
Share on other sites

Posted

[quote name='Marcher Technologies' timestamp='1316889313' post='2174851']
now... that is sad... third-party editor is causing more work... not less.... hindsight 20/20 and all that.


On the other hand, if IPS can modify the previous editor to include the benefits of the CKEditor without its current bugs and issues, I would be content with that as well.

Wasn't the previous editor also a third-party editor btw? I thought that was mentioned somewhere.

cmh likes this

Share this post


Link to post
Share on other sites

Posted

[quote name='+++' timestamp='1316889421' post='2174852']

On the other hand, if IPS can modify the previous editor to include the benefits of the CKEditor without its current bugs and issues, I would be content with that as well.

Wasn't the previous editor also a third-party editor btw? I thought that was mentioned somewhere.

No... was home-brewed IPS.

Share this post


Link to post
Share on other sites

Posted

[quote name='Marcher Technologies' timestamp='1316876475' post='2174780']
oh... and this:
$this->registry->getClass('output')->buildUrl('blah', 'publicWithApp')
is flat unacceptable....
Never before have i been unwilling to use a codebox.... which is saddening... as without it proper syntax highlighting vanishes.


That's a bug I introduced late Friday when I hot fixed an issue. Keep in mind that we use this forum to test various revisions as we make fixes. What you see here doesn't mean you'll see it in a production release. Test this bug on your own forum and you'll see it doesn't happen.




			
		
Alex K. and AndyF like this

Share this post


Link to post
Share on other sites
This topic is now closed to further replies.
Sign in to follow this  
Followers 0

  • Who's Browsing   0 members

    No registered users viewing this page.