? ? ? Failure

User tests: Successful: Unsuccessful:

avatar okonomiyaki3000
okonomiyaki3000
10 Jul 2017

Pull Request for Issue # .

Summary of Changes

Added a CodeMirror editor to the 'Appearance Options' tab of the CodeMirror plugin config page to take the guesswork out of configuring CodeMirror's appearance.

Testing Instructions

Open the CodeMirror config page and click on th 'Appearance Options' tab. Change the theme, font, etc. Save changes.

Expected result

The new appearance settings will be shown in the embedded editor field.

Actual result

Should be as expected.

Documentation Changes Required

No but there are a couple of new strings in the language file.

avatar okonomiyaki3000 okonomiyaki3000 - open - 10 Jul 2017
avatar okonomiyaki3000 okonomiyaki3000 - change - 10 Jul 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Jul 2017
Category Administration Language & Strings Front End Plugins
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 10 Jul 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 10 Jul 2017

I have tested this item successfully on f2fac83


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17042.

avatar brianteeman
brianteeman - comment - 10 Jul 2017

There must be a better way of doing this without adding inline jquery javascript and it really needs to be something that is translatable and not hard coded in english

avatar okonomiyaki3000
okonomiyaki3000 - comment - 10 Jul 2017

@brianteeman ? I think you've missed something. The "inline jquery" is just sample js code. It doesn't execute, it just appears in the editor so you see what it will look like. The only hard coded English is "Hello World" which I think is probably acceptable but, if not, I can change it.

avatar brianteeman
brianteeman - comment - 10 Jul 2017

Serves me right for replying before coffee

I understand what you are doing now - not sure of the benefit of having it - when I looked at the options page it confused me more than it helped me

Maybe it needs a different label than "sample text"

Layout preview?
Configuration preview?

avatar okonomiyaki3000
okonomiyaki3000 - comment - 10 Jul 2017

Maybe just 'Preview'?

avatar okonomiyaki3000 okonomiyaki3000 - change - 12 Jul 2017
Labels Added: ? ?
avatar okonomiyaki3000
okonomiyaki3000 - comment - 18 Jul 2017

Here's that pr-merge error again. What is this?

avatar okonomiyaki3000
okonomiyaki3000 - comment - 3 Aug 2017

Don't believe that pr-merge error. There is no problem with this PR.

avatar Quy
Quy - comment - 6 Dec 2017

It would be nice to see the preview when Theme is changed without having to click Save button each time.

avatar brianteeman
brianteeman - comment - 8 May 2018

I have tested this item successfully on 2d6a9d5


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17042.

avatar brianteeman brianteeman - test_item - 8 May 2018 - Tested successfully
avatar carlitorweb
carlitorweb - comment - 8 May 2018

I have tested this item successfully on 2d6a9d5


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17042.

avatar carlitorweb carlitorweb - test_item - 8 May 2018 - Tested successfully
avatar Quy Quy - change - 8 May 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 8 May 2018

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17042.

avatar mbabker mbabker - change - 10 May 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-05-10 00:59:35
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 10 May 2018
avatar mbabker mbabker - merge - 10 May 2018

Add a Comment

Login with GitHub to post a comment