[RFC] GeSHi highlighting

Note: We are moving the topics of this forum and it will be deleted at some point

Publish your own request for comments/change or patches for the next version of phpBB. Discuss the contributions and proposals of others. Upcoming releases are 3.2/Rhea and 3.3.
User avatar
AliasM2K
Registered User
Posts: 82
Joined: Tue Mar 27, 2012 2:33 am

Re: [RFC] GeSHi highlighting

Post by AliasM2K »

Just a suggestion; there is Prettify too...

User avatar
brunoais
Registered User
Posts: 964
Joined: Fri Dec 18, 2009 3:55 pm

Re: [RFC] GeSHi highlighting

Post by brunoais »

I believe that won't be accepted because it works in the client side (js).

User avatar
callumacrae
Former Team Member
Posts: 1046
Joined: Tue Apr 27, 2010 9:37 am
Location: England
Contact:

Re: [RFC] GeSHi highlighting

Post by callumacrae »

brunoais wrote:I believe that won't be accepted because it works in the client side (js).
What's wrong with client-side highlighting?
Made by developers, for developers!
My blog

User avatar
Erik Frèrejean
Registered User
Posts: 207
Joined: Thu Oct 25, 2007 2:25 pm
Location: surfnet
Contact:

Re: [RFC] GeSHi highlighting

Post by Erik Frèrejean »

brunoais wrote:I say that this is what I'd call a great extension.
I think the overhead of this does not make it worthwhile for the core. Many forums that run phpBB are forums related to coding, right.
I still believe that that is not enough to have such feature in the core.
So, -1 for core. Great as an extension
(technical stuff under this)
This is something very easy to place. This is just something that processes text.
Having that into account, I believe we just need to insert geshi into the BBcode parser (the part before displaying the post).
A nicely placed event will make the extension making simple enugh.
+1
Available on .com
Support Toolkit developer

User avatar
brunoais
Registered User
Posts: 964
Joined: Fri Dec 18, 2009 3:55 pm

Re: [RFC] GeSHi highlighting

Post by brunoais »

callumacrae wrote:
brunoais wrote:I believe that won't be accepted because it works in the client side (js).
What's wrong with client-side highlighting?
Does not work for non js users.

wGEric
Registered User
Posts: 521
Joined: Wed Jun 11, 2003 2:07 am
Contact:

Re: [RFC] GeSHi highlighting

Post by wGEric »

brunoais wrote:Does not work for non js users.
You could argue that this feature is like AJAX, enhances the user experience but everything still works without it.
Eric

User avatar
callumacrae
Former Team Member
Posts: 1046
Joined: Tue Apr 27, 2010 9:37 am
Location: England
Contact:

Re: [RFC] GeSHi highlighting

Post by callumacrae »

I wouldn't class syntax highlighting as an essential feature, just an enhancement - it doesn't need to work with JS disabled.
Made by developers, for developers!
My blog

User avatar
AliasM2K
Registered User
Posts: 82
Joined: Tue Mar 27, 2012 2:33 am

Re: [RFC] GeSHi highlighting

Post by AliasM2K »

callumacrae wrote:I wouldn't class syntax highlighting as an essential feature, just an enhancement - it doesn't need to work with JS disabled.
Absolutely :D

User avatar
DavidIQ
Customisations Team Leader
Customisations Team Leader
Posts: 1904
Joined: Thu Mar 02, 2006 4:29 pm
Location: Earth
Contact:

Re: [RFC] GeSHi highlighting

Post by DavidIQ »

+1 for being kept as an extension.
Image

User avatar
imkingdavid
Registered User
Posts: 1050
Joined: Thu Jul 30, 2009 12:06 pm

Re: [RFC] GeSHi highlighting

Post by imkingdavid »

DavidIQ wrote:+1 for being kept as an extension.
I do custom MODs. PM for a quote!
View My: MODs | Portfolio
Please do NOT contact for support via PM or email.
Remember, the enemy's gate is down.

Post Reply