? ? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
13 May 2017

Pull Request for Issue # .

This is a POC/RFC

Summary of Changes

  • Remove highlighter.(min.)js
  • Remove behavior.highlighter
  • Introduce behavior.highlight

Testing Instructions

Not yet testable

Expected result

Same highlighting as before

Actual result

Documentation Changes Required

YUP

avatar dgt41 dgt41 - open - 13 May 2017
avatar dgt41 dgt41 - change - 13 May 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 May 2017
Category Front End com_finder Repository Libraries JavaScript Plugins Unit Tests
avatar dgt41 dgt41 - change - 13 May 2017
Labels Added: ? ?
2b9956f 13 May 2017 avatar dgt41 init
avatar wilsonge
wilsonge - comment - 13 May 2017

OK I'm not against us moving to this new library. But let's have a think about working out a method of making the old JHtml method signature work with the new library, if possible.

avatar dgt41
dgt41 - comment - 14 May 2017

@wilsonge it's doable, I did this one as a POC, but I can use the old PHP API, adding just an array as the first param and doing some basic checking, you get the idea...

avatar wilsonge
wilsonge - comment - 14 May 2017

I'm happy with moving to something Not In House but I'd rather keep the PHP API b/c from 3.x and deprecate the old API for 5 if we can

avatar brianteeman
brianteeman - comment - 14 May 2017

What is the problem this is solving? Does it provide anything more than we have right now?

avatar dgt41
dgt41 - comment - 14 May 2017

What is the problem this is solving? Does it provide anything more than we have right now?

Two objectives:

  • vanilla javascript (no dependencies, also this script is future proof, e.g. ES6)
  • less code to support in-house...
avatar dgt41
dgt41 - comment - 24 May 2017

I will redo the PHP part so we can keep B/C here

avatar dgt41 dgt41 - change - 24 May 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-05-24 21:25:11
Closed_By dgt41
avatar dgt41 dgt41 - close - 24 May 2017

Add a Comment

Login with GitHub to post a comment