? ? Success

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
4 Sep 2016

Summary of Changes

The pull request add ability to drag&drop toolbar editing to help to configure TinyMCE editor.

How it works

screen 2016-09-11 13 50 00 1374x694

  1. Available buttons, and menu (Source panel). Which can be used for build your own toolbar.
  2. Your toolbar (Target panel). Drop here an available buttons.
  3. The tabs of Sets. By default it is 3 set. but it can be unlimited. Number of the sets can be changed in advanced tab:
    screen 2016-09-11 13 50 17 573x202
  4. The Builder have 3 preset: Simple, Medium, Advanced, which can be used for quick setup. The Clear button will clear the canvas.
  5. Access level for a current Set.
  6. The form with additional options for a current Set.
  7. Drop zone for Menubar.
  8. Drop zone for Toolbar1.
  9. Drop zone for Toolbar2.

How to add button to my toolbar?
Drag the button from Source panel (1) to Target panel (2), and drop it there.

How to remove the button?
Drag&drop unneeded button out of the Target panel (2).

I want more Sets!!!
Set the amount of sets in the advanced tab. A new sets will appears after save.

Testing Instructions

it does not ready for final test

First of all make sure if the drag&drop editing works.
Then need to check that all existing features still works, as here is huge code changes.

Documentation Changes Required

Update info about how to configure TinyMCE plugin (if there any).

B. C.

It can break half of user settings for TinyMCE plugin. So need to test each case: new installation, update.

After upgrade or on clear installation the editor will use next default presets:

  • Advanced => for Special access level
  • Medium => for Registered
  • Simple => for everyone else

Help needed

I made 3 presets: Simple, Medium, Advanced.

Would be cool if someone will improve them, and make it more real for what user need.
Also would be cool if they will be similar to "old toolbar", so we will keep B/C a little bit 😉
It is easy to edit, just add needed buttons in: https://github.com/Fedik/joomla-cms/blob/0b4d803c35c3995934b017f74496d0c3c26f8bb0/plugins/editors/tinymce/tinymce.php#L1064-L1106

P.S.
I found that make editing depend from "view level" is not as good as I thought on start 😄
So I have an idea to change it a bit. And make a "Presets" so user can set access level for each "Preset". Also it will allow to use separated "Preset" for each user It is already implemented 😉

I would like to hear the feedback 😉

The pull depending from #11157. But can be tested stand alone.

avatar Fedik Fedik - open - 4 Sep 2016
avatar Fedik Fedik - change - 4 Sep 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 4 Sep 2016
Category Layout JavaScript External Library Plugins Front End
avatar joomla-cms-bot joomla-cms-bot - change - 4 Sep 2016
Labels Added: ?
avatar Fedik Fedik - change - 4 Sep 2016
The description was changed
avatar Fedik Fedik - edited - 4 Sep 2016
avatar Fedik Fedik - change - 4 Sep 2016
The description was changed
avatar Fedik Fedik - edited - 4 Sep 2016
avatar Fedik Fedik - change - 4 Sep 2016
The description was changed
avatar Fedik Fedik - edited - 4 Sep 2016
avatar Fedik
Fedik - comment - 4 Sep 2016

@brianteeman @infograf768 is it what you asked in #10996?

avatar brianteeman
brianteeman - comment - 4 Sep 2016

@fedik yes - that looks very interesting.

avatar Fedik Fedik - change - 4 Sep 2016
The description was changed
avatar Fedik Fedik - edited - 4 Sep 2016
avatar brianteeman
brianteeman - comment - 4 Sep 2016

Couple of quick observations.
1. You can create a toolbar wider than the allowed area - in fact some of the samples are
2. I think you need some more contrast between the drop areas. I couldnt work out why my icons weren't "dropping" until I saw that the top row is only for the dropdown menus.

avatar Fedik Fedik - change - 4 Sep 2016
The description was changed
avatar Fedik Fedik - edited - 4 Sep 2016
avatar dgt41
dgt41 - comment - 4 Sep 2016

I like this approach, user (very) friendly

make a "Presets" so user can set access level for each "Preset"

Good news!

avatar schnuti
schnuti - comment - 6 Sep 2016

👍
But I do not quite understand the approach.
I would prefer a couple of presets (3 or create more) that I can modify and assign to user groups (users?). Alternatively different setup for back- and frontend.
Hmm... I have no clou why the current advanced settings use view levels. Since J 1.5?

I do not understand the use of the three buttons simple/advanced/clear.

Bug: I do not see any separators after applying a new setting. That might be one source of the width problem.

The user groups ("view levels") should better be in a drop down and not tabs. The list can be long.

A good start!

avatar brianteeman
brianteeman - comment - 6 Sep 2016

I do not understand the use of the three buttons simple/advanced/clear.
They are the presets you requested

On 6 September 2016 at 12:56, Ove Eriksson notifications@github.com wrote:

👍
But I do not quite understand the approach.
I would prefer a couple of presets (3 or create more) that I can modify
and assign to user groups (users?). Alternatively different setup for back-
and frontend.
Hmm... I have no clou why the current advanced settings use view levels.
Since J 1.5?

I do not understand the use of the three buttons simple/advanced/clear.

Bug: I do not see any separators after applying a new setting. That might
be one source of the width problem.

The user groups ("view levels") should better be in a drop down and not
tabs. The list can be long.

A good start!


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#11926 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8VkBh0XI6C2TUTgQON2Z95OcLaOJks5qnVTvgaJpZM4J0kpq
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar Fedik
Fedik - comment - 6 Sep 2016

The user groups ("view levels") should better be in a drop down and not tabs.

for now, each tab is a separated form for selected View level

I would prefer a couple of presets (3 or create more) that I can modify and assign to user

something similar already planed for future changes, as I wrote in p.s. 😉

avatar schnuti
schnuti - comment - 6 Sep 2016

There will not often be a need of more then 2 or three different settings. If the 3 presets used currently, simple, advanced and extended ( =all? ) are loaded by default they can be edited and assigned to user groups. The B/C problem I see are the view level settings for the "extended" settings.

Use old handling until new parameters get saved?

avatar jeckodevelopment jeckodevelopment - change - 7 Sep 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 10 Sep 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 10 Sep 2016
Category Layout JavaScript External Library Plugins Front End Language & Strings Administration Layout JavaScript External Library Plugins Front End
avatar Fedik Fedik - change - 10 Sep 2016
The description was changed
avatar brianteeman
brianteeman - comment - 11 Sep 2016

I like the new changes to have sets and not usergroups in the tabs. I assume its a bug that they are ordered
Set 2, Set 1, Set 0
and not
Set 0, Set 1, Set 2


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

avatar brianteeman
brianteeman - comment - 11 Sep 2016

Doesnt it mean that the level select dropdown needs to be a multiselect? How else can you apply Set 2 to multiple groups - maybe I am missing something

avatar brianteeman brianteeman - change - 11 Sep 2016
Labels Added: ?
avatar Fedik
Fedik - comment - 11 Sep 2016

Set 2, Set 1, Set 0

It made with purpose.
Where on clear installation we get:
0 - for advanced
1 - for medium
2 and all other which added by user will use Simple preset for initial state.

So when you set "Set amount" to 5: you will get: 4 3 2 1 0.
Where 4 3 will be new Sets and they will be in "first tabs".
I thought it can be more intuitive, mean more easy to find new. Also I thought In most cases user will want to edit 1 and 2, so I put it on front.

How else can you apply Set 2 to multiple groups

hm, but with other content we have the same. You can chose only one access level for Article, Module etc. and each access level have some inheritance , should be no problem.

other thing: it can be hard to implement 😄
and as you can have unlimited Sets (by default 3), you can make a Set for each access level

avatar brianteeman
brianteeman - comment - 11 Sep 2016

I missed the "amount of sets" option on the advanced tab ;)

On 11 September 2016 at 10:56, Fedir Zinchuk notifications@github.com
wrote:

Set 2, Set 1, Set 0

It made with purpose.
Where on clear installation we get:
0 - for advanced
1 - for medium
2 and all other which added by user will use Simple preset for initial
state.

So when you set "Set amount" to 5: you will get: 4 3 2 1 0.
Where 4 3 will be new Sets and they will be in "first tabs".
I thought it can be more intuitive, mean more easy to find new.

Also I thought In most cases user will want to edit 1 and 2, so I put it
on front.

How else can you apply Set 2 to multiple groups

hm, but with other content we have the same. You can chose only one access
level for Article, Module etc. and each access level have some inheritance
, should be no problem.

other thing: it can be hard to implement 😄
and as you can have unlimited Sets (by default 3), you can make a Set for
each access level


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#11926 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8Zo-C1NC0n5o-yfguKQlM2xYwiLCks5qo9BIgaJpZM4J0kpq
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar Fedik
Fedik - comment - 11 Sep 2016

I need small help here.
I made 3 presets: Simple, Medium, Advanced.

Would be cool if someone will improve them, and make it more real for what user needs.
It is easy to edit, just add needed buttons in: https://github.com/Fedik/joomla-cms/blob/0b4d803c35c3995934b017f74496d0c3c26f8bb0/plugins/editors/tinymce/tinymce.php#L1064-L1106

avatar brianteeman
brianteeman - comment - 11 Sep 2016

For B/C don't you think the presets should match the current sets

On 11 September 2016 at 11:03, Fedir Zinchuk notifications@github.com
wrote:

I need small help here.
I made 3 presets: Simple, Medium, Advanced.

Would be cool if someone will improve them, and make it more real for what
user needs.
It is easy to edit, just add needed buttons in: https://github.com/Fedik/
joomla-cms/blob/0b4d803c35c3995934b017f74496d0c3c26f8bb0/plugins/editors/
tinymce/tinymce.php#L1064-L1106


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#11926 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8TEaeH41DlBpqrPlqqxP4aOdnla6ks5qo9HpgaJpZM4J0kpq
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar Fedik
Fedik - comment - 11 Sep 2016

yes, this would be cool. But there no simple way to find out what was in old 😄
that why I asked for help 😉

Advanced preset I tried to make similar to what we had with Advanced mode (when all plugin is ON),
But the Medium preset is totally random stuff for now.
And Simple preset, also tried to copy, but it looks tooooo simple 😄

And on clear installation/after upgrade will be:
Advanced => for Special access level
Medium => for Registered
Simple => for everyone else

avatar Fedik Fedik - change - 11 Sep 2016
The description was changed
avatar Fedik
Fedik - comment - 11 Sep 2016

I have update the pull description, with more detailed info about how it work.
I think it is ready for testing, to find out what can be improved and how it will work in real life 😉

And would be cool to see this feature in 3.7 :neckbeard:

avatar Fedik Fedik - change - 11 Sep 2016
The description was changed
avatar Fedik
Fedik - comment - 17 Sep 2016

I have updated the presets, so they now look more close to what we have now.

Simple
screen 2016-09-17 16 23 33 485x40

Medium
screen 2016-09-17 16 23 54 1001x80

Advanced
screen 2016-09-17 16 24 08 1135x108

avatar brianteeman
brianteeman - comment - 17 Sep 2016

Great. Thanks.

avatar Fedik Fedik - change - 17 Sep 2016
The description was changed
avatar brianteeman
brianteeman - comment - 6 Oct 2016

As you can see from the screenshot I dont get the exact same layout as you with the Advanced preset

screen shot 2016-10-06 at 14 41 25

The same thing happens with the Medium preset. There is a new line after the fourth icon and the whole toolbar is wider than the available area

Tested with 3.7


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

avatar brianteeman brianteeman - test_item - 6 Oct 2016 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 6 Oct 2016

I have tested this item 🔴 unsuccessfully on 58eb3bd


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

avatar Fedik
Fedik - comment - 8 Oct 2016

please, can someone change destination branch from staging => 3.7.x?
it will be more easy to test and fix possible conflicts, also #11157 already merged there

avatar zero-24
zero-24 - comment - 8 Oct 2016

Done. You can also do it your self via the edit button where you edit the PR title ;)

avatar Fedik
Fedik - comment - 8 Oct 2016

@zero-24 thanks!

avatar joomla-cms-bot joomla-cms-bot - change - 8 Oct 2016
The description was changed
Labels Added: ?
avatar Fedik
Fedik - comment - 8 Oct 2016

@brianteeman fixed

avatar dgt41
dgt41 - comment - 8 Oct 2016

@Fedik this will be compatible with #11367 ??
Also any help on that one would be appreciated 😉

avatar brianteeman
brianteeman - comment - 8 Oct 2016

@Fedik thanks - will test soon. As you can tell I am super excited about this PR

avatar Fedik
Fedik - comment - 8 Oct 2016

@dgt41 will be a loot conflicts for sure 😄
but in final all should be compatible, after some modification,

and to be honest I not very understand what there is going on 😄

avatar brianteeman
brianteeman - comment - 8 Oct 2016

@fedik - thanks I can confirm that issue is fixed


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

avatar brianteeman brianteeman - test_item - 8 Oct 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 8 Oct 2016

I have tested this item successfully on 58eb3bd


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

avatar Fedik
Fedik - comment - 9 Oct 2016

test fail on unrelated test, something with Highlight testing

avatar brianteeman brianteeman - change - 29 Oct 2016
Labels Removed: ?
avatar brianteeman
brianteeman - comment - 16 Nov 2016

@fedik can you fix the merge conflicts please

@all this is an absolute killer feature PLEASE can people test it. I have been using it myself developing a site and it rocks. At least one less extension to install

avatar zero-24
zero-24 - comment - 17 Nov 2016

@fedik the drone tests should pass if we sync to staging :)

avatar Fedik Fedik - change - 17 Nov 2016
Labels Removed: ?
avatar Fedik
Fedik - comment - 17 Nov 2016

fixed

avatar brianteeman
brianteeman - comment - 17 Nov 2016

Great. Thanks. I will try to get more testers for this as I really want to
see this in core.

avatar infograf768
infograf768 - comment - 17 Nov 2016

One question:
what happens when a user group has multiple viewing access?

Example:
screen shot 2016-11-17 at 10 39 20

Will managers get the Special access setting or one of the others (Registered, Customer Access Level)?

avatar brianteeman
brianteeman - comment - 17 Nov 2016

Alternatively and If you are using a flat acl because you are doing role based acl you can also just create new editor groups. Basic editor, advanced editor etc

avatar infograf768
infograf768 - comment - 17 Nov 2016

it will take higher, or last in the order

Does that mean that in the case above, Managers would get the Special access setting?

avatar Fedik
Fedik - comment - 17 Nov 2016

it will take the last one from the array of the user view levels, that come from $user->getAuthorisedViewLevels()
so it will depend from the ordering, for your example

avatar brianteeman brianteeman - change - 4 Dec 2016
The description was changed
Labels
Easy No Yes
avatar brianteeman brianteeman - change - 4 Dec 2016
Labels Removed: ? ? ? ?
avatar brianteeman brianteeman - edited - 4 Dec 2016
avatar brianteeman brianteeman - change - 4 Dec 2016
Title
[RFC] TinyMCE drag&drop toolbar editing
TinyMCE drag&drop toolbar editing
avatar brianteeman brianteeman - edited - 4 Dec 2016
avatar brianteeman brianteeman - change - 4 Dec 2016
Title
[RFC] TinyMCE drag&drop toolbar editing
TinyMCE drag&drop toolbar editing
avatar dgt41
dgt41 - comment - 6 Dec 2016

I have tested this item successfully on 58eb3bd


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

avatar dgt41 dgt41 - test_item - 6 Dec 2016 - Tested successfully
avatar dgt41 dgt41 - change - 6 Dec 2016
Status Pending Ready to Commit
avatar dgt41
dgt41 - comment - 6 Dec 2016

Thank you @Fedik
RTC

avatar jeckodevelopment jeckodevelopment - change - 6 Dec 2016
Milestone Added:
avatar rdeutz rdeutz - change - 6 Dec 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-12-06 21:48:15
Closed_By rdeutz
avatar rdeutz rdeutz - close - 6 Dec 2016
avatar rdeutz rdeutz - merge - 6 Dec 2016
avatar rdeutz rdeutz - reference | cd33086 - 6 Dec 16
avatar rdeutz rdeutz - merge - 6 Dec 2016
avatar rdeutz rdeutz - close - 6 Dec 2016
avatar Fedik Fedik - head_ref_deleted - 7 Dec 2016
avatar infograf768
infograf768 - comment - 23 Dec 2016

@Fedik
REGRESSION: Looks like this broke automatic language detection
See #13343
also see
#13344

Add a Comment

Login with GitHub to post a comment