? ? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
10 Mar 2017

Pull Request for improving the UX and making editing a bit easier.

Summary of Changes

TinyMCE out of the box comes with some very basic styling. Eg the Bold button is just a <strong> around the selected text. But the editor gives the ability for further customisations e.g.: attach a class or inline styles etc...

So this PR brings that functionality to Joomla.
It covers both styles and formats (styles are the ones that correspond to the buttons e.g. bold, align left, etc and formats are the ones that will be in the formats dropdown)

Testing Instructions

Grab the files from https://gist.github.com/dgt41/be9e16814371f43ce99e8dc74ef54e99
and place them in administrator/templates/isis/css directory
Edit tinyMCE's configuration and type the relative paths:
screen shot 2017-03-10 at 13 09 07
Edit an article and observe the code when selecting a format or applying a style (e.g. bold, align right)
screen shot 2017-03-10 at 12 42 49

Expected result

Styles (standard buttons):

screen shot 2017-03-10 at 13 14 35

Formats (Formats dropdown

screen shot 2017-03-10 at 13 14 03
):

Documentation Changes Required

Yup!

Inspired by @coolcat-creations post on @brianteeman 's site...

f4cf8dd 10 Mar 2017 avatar dgt41 init
avatar dgt41 dgt41 - open - 10 Mar 2017
avatar dgt41 dgt41 - change - 10 Mar 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Mar 2017
Category Administration Language & Strings Front End Plugins
avatar brianteeman
brianteeman - comment - 10 Mar 2017

From your screenshot you show the file being a json file

The language strings make no mention of this at all

avatar dgt41
dgt41 - comment - 10 Mar 2017

@brianteeman you can improve it as you like...

avatar brianteeman
brianteeman - comment - 10 Mar 2017

@dgt41 but I dont know what it should be

avatar dgt41
dgt41 - comment - 10 Mar 2017

The filename could be anything, same goes for the path, the only requirement is to be json type.

avatar brianteeman
brianteeman - comment - 10 Mar 2017

the only requirement is to be json type

So that needs to be mentioned ;)

avatar infograf768
infograf768 - comment - 10 Mar 2017

maybe with ahint in the field?


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

avatar dgt41 dgt41 - change - 24 May 2017
Labels Added: ? ?
avatar dgt41 dgt41 - change - 22 Jul 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-07-22 20:08:35
Closed_By dgt41
avatar dgt41 dgt41 - close - 22 Jul 2017

Add a Comment

Login with GitHub to post a comment