(#337) Sélecteur de couleur de l'éditeur: plusieurs répétition des mêmes couleurs dans la palette proposée

[Scoubifitz]

Bonjour,

suite à mes remarques dans ce sujet (qui a été classé comme résolu) :

https://forum.forumactif.com/t352764-nouvel-editeur-selecteur-de-couleur#3005316

... et au complément d'information (qui a été corbeillé) :

https://forum.forumactif.com/t352781-selecteur-de-couleur#3005423

... je ne sais toujours pas si le bug a été pris en compte ..

seule information dans le compte-rendu :

13 - Amélioration : Rendre le système de couleur plus simple
>> Placé dans le planning

ma question : le bug sera-t-il corrigé ?

---

Autre question , j'avais un script avant cette mise à jour , que j'aimerais remettre en place :

je vous fourni le script (aimablement mis à disposition sur le FdF anglais il y a 3 ans):

Code:

var CopyrightNotice = 'Color-picker for forumotion text editors. Copyright © 2005-2011 by Dion Designs. All Rights Reserved. Use and/or modification of this script is allowed, provided this entire copyright notice remains in the original or modified script. Distribution is not allowed without written consent from Dion Designs.';
$(function() {
    if (document.getElementById('color')) {
        var colors=new Array('00','33','66','99','CC','FF');
        var x='<div id="colorpicker"><table cellspacing="0" id="grid">';
        var y=x;
        for (i = 5; i >= 0; i--) {
            x=x+'<tr>';
            y=y+'<tr>';
            for (j = 5; j >= 0; j--) {
                for (k= 5; k >= 0; k--) {
                    var col = colors[j]+colors[i]+colors[k];
                    x=x+'<td style="background: #'+col+';"><button style="background:#'+col+';" title="#'+col+'" onmouseover="this.className=\'selectHover\';" onmouseout="this.className=\'\';" onclick="bbfontstyle(\'[color=#'+col+']\', \'[/color]\');selectWysiwyg(this, \'color\');return false;"></button></td>';
                    y=y+'<td style="background: #'+col+';"><button style="background:#'+col+';" id="text_editor_cmd_constructBBcode_select_color_#'+col+'"></button></td>';
                }
            }
            x=x+'</tr>';
            y=y+'</tr>';
        }
        document.getElementById('color').innerHTML=x+'</table></div>';
        document.getElementById('wcolor').innerHTML=y+'</table></div>';
    }
});

et le CSS qui va avec :

Code:

/* couleurs  */
#color, #wcolor {overflow: visible;}
#colorpicker {position: absolute; top: 20px; left: -225px;}
#grid {border: 1px solid #808080;}
#grid td {padding: 0px;}
#grid td button {height: 12px; width: 12px; padding: 0px; border: 1px solid #808080;}
#grid td button:hover {border: 1px solid #ffffff;}


ainsi que le rendu :



est-il possible de l'adapter au nouvel éditeur ? si oui , comment ?

Merci .

[Scoubifitz]

un petit  up ?

[Scoubifitz]

up ...

... je ne sais toujours pas si le bug a été pris en compte ..

ma question : le bug sera-t-il corrigé ?

---

Autre question , j'avais un script avant cette mise à jour , que j'aimerais remettre en place

est-il possible de l'adapter au nouvel éditeur ? si oui , comment ?

Merci .

[Scoubifitz]

... je ne sais toujours pas si le bug a été pris en compte ..

ma question : le bug sera-t-il corrigé ?

Bonsoir,
Apparemment , le souci est à l'ordre du jour , wait and see ?

Autre question , j'avais un script avant cette mise à jour , que j'aimerais remettre en place

est-il possible de l'adapter au nouvel éditeur ? si oui , comment ?

Merci .

petit UP pour ce soucis ...

[Scoubifitz]

Bonjour,

Autre question , j'avais un script avant cette mise à jour , que j'aimerais remettre en place :

je vous fourni le script (aimablement mis à disposition sur le FdF anglais il y a 3 ans):

Code:

var CopyrightNotice = 'Color-picker for forumotion text editors. Copyright 2005-2011 by Dion Designs. All Rights Reserved. Use and/or modification of this script is allowed, provided this entire copyright notice remains in the original or modified script. Distribution is not allowed without written consent from Dion Designs.';
$(function() {
    if (document.getElementById('color')) {
        var colors=new Array('00','33','66','99','CC','FF');
        var x='<div id="colorpicker"><table cellspacing="0" id="grid">';
        var y=x;
        for (i = 5; i >= 0; i--) {
            x=x+'<tr>';
            y=y+'<tr>';
            for (j = 5; j >= 0; j--) {
                for (k= 5; k >= 0; k--) {
                    var col = colors[j]+colors[i]+colors[k];
                    x=x+'<td style="background: #'+col+';"><button style="background:#'+col+';" title="#'+col+'" onmouseover="this.className=\'selectHover\';" onmouseout="this.className=\'\';" onclick="bbfontstyle(\'[color=#'+col+']\', \'[/color]\');selectWysiwyg(this, \'color\');return false;"></button></td>';
                    y=y+'<td style="background: #'+col+';"><button style="background:#'+col+';" id="text_editor_cmd_constructBBcode_select_color_#'+col+'"></button></td>';
                }
            }
            x=x+'</tr>';
            y=y+'</tr>';
        }
        document.getElementById('color').innerHTML=x+'</table></div>';
        document.getElementById('wcolor').innerHTML=y+'</table></div>';
    }
});

et le CSS qui va avec :

Code:

/* couleurs  */
#color, #wcolor {overflow: visible;}
#colorpicker {position: absolute; top: 20px; left: -225px;}
#grid {border: 1px solid #808080;}
#grid td {padding: 0px;}
#grid td button {height: 12px; width: 12px; padding: 0px; border: 1px solid #808080;}
#grid td button:hover {border: 1px solid #ffffff;}


ainsi que le rendu :



est-il possible de l'adapter au nouvel éditeur ? si oui , comment ?

Merci .

Petit up ...

[Scoubifitz]

un petit up concernant le BUG du "color picker" (palette de couleur étendue) qui n'est apparemment pas résolu ...

Rappel du bug :

ceci est la palette actuelle sur tous les forums


Le problème est que ce tableau n'est pas complet ! il manque des couleurs , notamment les couleurs vives et sombres ...

De plus , il y a des couleurs répétitives , explication en images :





Comme vous pouvez le constater , les couleurs sont ternes et se répètent , voici la palette sans les répétitions ...



Le problème se situe au niveau du script dans :https://illiweb.com/rs3/75/frm/SCEditor/src/jquery.sceditor.js

Je me suis permis de corriger ce script et de le tester :

Code:

// START_COMMAND: Colour
 color: {
 _dropDown: function(editor, caller, callback) {
 var i, x, color, colors,
 genColor     = {r: 255, g: 255, b: 255},
 content      = $("<div />"),
 colorColumns = editor.opts.colors?editor.opts.colors.split("|"):new Array(6),
 // IE is slow at string concation so use an array
 html         = [],
 cmd          = $.sceditor.command.get('color');

 if(!cmd._htmlCache)
 {
 for (i=0; i < colorColumns.length; ++i)
 {
 colors = colorColumns[i]?colorColumns[i].split(","):new Array(37);

 html.push('<div class="sceditor-color-column">');
 for (x=1; x < colors.length; ++x)
 {
 // use pre defined colour if can otherwise use the generated color
 color = colors[x] || "#" + genColor.r.toString(16) + genColor.g.toString(16) + genColor.b.toString(16);

 html.push('<a href="#" class="sceditor-color-option" style="background-color: '+color+'" data-color="'+color+'" title="'+color+'"></a>');

 // calculate the next generated color
 if(x%6===0)
 {
 genColor.g -= 51;
 genColor.b = 255;
 if(genColor.g < 51){genColor.g = "00";}
 }
 else
 genColor.b -= 51;
 if(genColor.b < 51){genColor.b = "00";}
 }
 html.push('</div>');

 // calculate the next generated color
 if(i%1===0)
 {
 genColor.r -= 51;
 genColor.g = 255;
 genColor.b = 255;
 if(genColor.r < 51){genColor.r = "00";}
 }
 else
 {
 genColor.g = 255;
 genColor.b = 255;
 }
 }

 cmd._htmlCache = html.join('');
 }

 content.append(cmd._htmlCache)
 .find('a')
 .click(function (e) {
 callback($(this).attr('data-color'));
 editor.closeDropDown(true);
 e.preventDefault();
 });

 editor.createDropDown(caller, "color-picker", content);
 },
 _menu: function(editor, caller, callback) {
 var colors = {},
html = $('<div />');
 colors[editor._('color_dark_red')] = '#660000';
 colors[editor._('color_red')] = '#FF0000';
 colors[editor._('color_orange')] = '#FF9933';
 colors[editor._('color_brown')] = '#663300';
 colors[editor._('color_yellow')] = '#FFFF00';
 colors[editor._('color_green')] = '#006600';
 colors[editor._('color_olive')] = '#666633';
 colors[editor._('color_cyan')] = '#00FFFF';
 colors[editor._('color_blue')] = '#0000FF';
 colors[editor._('color_dark_blue')] = '#000099';
 colors[editor._('color_indigo')] = '#6600FF';
 colors[editor._('color_violet')] = '#990099';
 colors[editor._('color_grey')] = '#999999';
 colors[editor._('color_white')] = '#EEEEEE';
 colors[editor._('color_black')] = '#000000';
 
for(var k in colors)
 html.append('<div><a class="sceditor-fontsize-option"><font color="' + colors[k] + '">' + k + '</font></a></div>');
 
html.find('font').click(function(e) {
 callback($(this).attr('color'));
 editor.closeDropDown(true);
 e.preventDefault();
 });
 
editor.createDropDown(caller, "color-picker", html);
 },
 exec: function (caller) {
 var editor = this;

 if(palette === 1) {
 $.sceditor.command.get('color')._dropDown(
 editor,
 caller,
 function(color) {
 editor.execCommand("forecolor", color);
 }
 );
 }
 else {
 $.sceditor.command.get('color')._menu(
 editor,
 caller,
 function(color) {
 editor.execCommand("forecolor", color);
 }
 );
 }
 },
 tooltip: "Font Color"
 },
 // END_COMMAND
 // START_COMMAND: Remove Format
 removeformat: {
 exec: "removeformat",
 tooltip: "Remove Formatting"
 },
 // END_COMMAND


détail des corrections:

Vous pouvez voir le résultat (j'ai rajouté un titre pour visualiser le code couleur au survol) :



Merci de transmette à un DEVactif pour analyse .

Ceci résoudrait par la même occasion mon autre souci de script .

[Scoubifitz]

petit up ...

édit le 13.02.2014 :

Bonsoir,

Ce message concerne le sujet qui vient d'être archivé dans "besoin d'un devactif" :

https://forum.forumactif.com/t353245-bug-et-amelioration-337-selecteur-de-couleur#3084934

---

Je n'ai jamais prétendu que le color picker ne fonctionnait pas à cause d'un bug technique ,

mais qu'il y a une énorme boulette de programmation qui multiplie les couleurs par 5 !



sur les 441 cases affichées , il n'y a que 105 couleurs disponibles ... les autres cases sont des répétitions à cause d'une boulette de codage .

Si je n'ai pas jugé nécessaire de faire une suggestion dans la rubrique appropriée , c'est parce que j'ai déjà donné une solution dans un sujet qui en parlait :

https://forum.forumactif.com/t356066-ajouter-nouvelles-couleurs-dans-la-palette-simple-de-couleurs#3033647

Si je vous contacte , C'est pour vous dire que le sujet archivé ne pourra être considéré comme résolu que
quand il y aura effectivement 105 cases pour les 105 couleurs affichées et non 441 cases ...
Ce n'est pas une question d'esthétique , mais de principe .
Je ne pense pas que Forumactif fasse dans l'amateurisme , et si cette boulette peut être corrigée , qu'elle le soit .

Merci de transmettre à The Godfather , si vous le jugez nécessaire .

En toute amitié ,
Scoubifitz

[The Godfather]

Bonjour,

Ceci n'est en aucun cas un bug technique! c'est une demande d'amélioration de l’éditeur car dans l’immédiat le color picker est fonctionnel.

Certes il ne propose pas les meilleurs couleurs mais vous pouvez toujours mettre le code couleur que vous souhaitez

Je garde néanmoins cette demande sous le coude car la demande d'amélioration me parait justifié

[Strike Freedom]

Bonjour à tous,

Je vous informe que l’amélioration a bien été prise en compte et que ce dernier vient d’être déployé

Merci à Scoubifitz pour ta contribution

[Serang]

Merci Dévactif

[Scoubifitz]

Merci !

Sujet résolu .