(#337) Sélecteur de couleur de l'éditeur: plusieurs répétition des mêmes couleurs dans la palette proposée
4 participants
Forum gratuit : Le forum des forums actifs :: Entraide & Support... :: Besoin d'un Dévactif... :: Archives: les corrections apportées
Page 1 sur 1 • Partagez
(#337) Sélecteur de couleur de l'éditeur: plusieurs répétition des mêmes couleurs dans la palette proposée
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 :
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):
et le CSS qui va avec :
ainsi que le rendu :
est-il possible de l'adapter au nouvel éditeur ? si oui , comment ?
Merci .
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 .
Dernière édition par Scoubifitz le Lun 24 Fév 2014 - 14:48, édité 2 fois
Re: (#337) Sélecteur de couleur de l'éditeur: plusieurs répétition des mêmes couleurs dans la palette proposée
up ...
Scoubifitz a écrit:
... 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 .
Re: (#337) Sélecteur de couleur de l'éditeur: plusieurs répétition des mêmes couleurs dans la palette proposée
Bonsoir,Scoubifitz a écrit:... je ne sais toujours pas si le bug a été pris en compte ..
ma question : le bug sera-t-il corrigé ?
Apparemment , le souci est à l'ordre du jour , wait and see ?
petit UP pour ce soucis ...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 .
Re: (#337) Sélecteur de couleur de l'éditeur: plusieurs répétition des mêmes couleurs dans la palette proposée
Scoubifitz a écrit: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 ...
Re: (#337) Sélecteur de couleur de l'éditeur: plusieurs répétition des mêmes couleurs dans la palette proposée
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 :
Merci de transmette à un DEVactif pour analyse .
Ceci résoudrait par la même occasion mon autre souci de script .
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:
- // 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
Merci de transmette à un DEVactif pour analyse .
Ceci résoudrait par la même occasion mon autre souci de script .
Re: (#337) Sélecteur de couleur de l'éditeur: plusieurs répétition des mêmes couleurs dans la palette proposée
petit up ...
édit le 13.02.2014 :
édit le 13.02.2014 :
Scoubifitz a écrit: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
Dernière édition par Scoubifitz le Jeu 13 Fév 2014 - 10:58, édité 1 fois (Raison : complément d'info suite à la conversation avec The Godfather)
Re: (#337) Sélecteur de couleur de l'éditeur: plusieurs répétition des mêmes couleurs dans la palette proposée
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é
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é
Dernière édition par The Godfather le Mar 4 Fév 2014 - 9:43, édité 1 fois (Raison : Réouverture suite aux explications/demande du membre Scoubifitz ;-))
The Godfather- Adminactif
-
Messages : 18294
Inscrit(e) le : 02/05/2004
Re: (#337) Sélecteur de couleur de l'éditeur: plusieurs répétition des mêmes couleurs dans la palette proposée
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
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
Sujets similaires
» Ajouter nouvelles couleurs dans la palette simple de couleurs
» ajouter des couleurs dans la palette simplifiée
» J'ai voulu rajouter des couleurs dans la palette simplifiée
» Sélecteur de couleur message rapide :suppression et modification de certaines couleurs
» - Sélecteur de couleurs dans les posts
» ajouter des couleurs dans la palette simplifiée
» J'ai voulu rajouter des couleurs dans la palette simplifiée
» Sélecteur de couleur message rapide :suppression et modification de certaines couleurs
» - Sélecteur de couleurs dans les posts
Forum gratuit : Le forum des forums actifs :: Entraide & Support... :: Besoin d'un Dévactif... :: Archives: les corrections apportées
Page 1 sur 1
Permission de ce forum:
Vous ne pouvez pas répondre aux sujets dans ce forum