nicomilville
Messages postés3472Date d'inscriptionlundi 16 juillet 2007StatutMembreDernière intervention28 février 2014
-
30 avril 2008 à 10:49
nicomilville
Messages postés3472Date d'inscriptionlundi 16 juillet 2007StatutMembreDernière intervention28 février 2014
-
30 avril 2008 à 15:39
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
nicomilville
Messages postés3472Date d'inscriptionlundi 16 juillet 2007StatutMembreDernière intervention28 février 201436 30 avril 2008 à 15:39
J'ai vu que tu avais fait des modifications, c'est mieux mais il reste encore un une chose :
- es ce que tu vois une diférence entre les deux premier carré foncé ?
PS : J'attend les avis des autres pour voir si ils vont dans mon sens ou pas, pour l'instant tu a 8.5 !
ralecul
Messages postés111Date d'inscriptiondimanche 23 mars 2003StatutMembreDernière intervention 1 août 2008 30 avril 2008 à 15:14
Salut,
Ton code est intéressant mais je te propose quelques améliorations :
- Possibilité de l'instancier plusieurs fois.
- Passage d'options "à la prototype".
- Ajout de callback onUpdate et onChange.
- Amélioration de la création de la table
- Correction de bug mineur : table.style.border, cell.style.height = 8+"px", Picker
function buildHtml()
{
var table = document.createElement("table");
table.setAttribute("cellspacing", 1);
table.style.border = "1px solid black";
table.style.margin = "0";
table.style.padding = "0";
for (var red=0; red
nicomilville
Messages postés3472Date d'inscriptionlundi 16 juillet 2007StatutMembreDernière intervention28 février 201436 30 avril 2008 à 10:49
Salut,
Bon code !!!
Pas de note pour l'instant... j'attend de voir si tu va faire des modifications...
Maintenant les remarques un peu moins gentilles :
Je ne vois pas ou est le div qui est censé afficher la couleur qui est survolé !!!
Je pense également qu'il est peut être un peu trop étendu peu être que tu pourrai le faire de la forme d'un carré et suppimer quelque colone par exemple les couleur foncé on dirai qu'elles sont en trois exemplaires chacune !!!
30 avril 2008 à 15:39
- es ce que tu vois une diférence entre les deux premier carré foncé ?
PS : J'attend les avis des autres pour voir si ils vont dans mon sens ou pas, pour l'instant tu a 8.5 !
30 avril 2008 à 15:14
Ton code est intéressant mais je te propose quelques améliorations :
- Possibilité de l'instancier plusieurs fois.
- Passage d'options "à la prototype".
- Ajout de callback onUpdate et onChange.
- Amélioration de la création de la table
- Correction de bug mineur : table.style.border, cell.style.height = 8+"px", Picker
Voici le code correspondant :
{Fichier : index.html}
div.selector {
float: left;
width: 50px;
height: 50px;
margin: 2px;
border: 1px solid black;
}
function $(id) {
return document.getElementById(id);
}
new ayxColorPicker("container", {
nbStep : 30,
autoHide : true,
onUpdate : function(colorPicker, red, green, blue) {
$('update').style.backgroundColor = "rgb("+red +","+green +","+blue +")";
},
onChange : function(colorPicker, red, green, blue) {
$('change').style.backgroundColor = "rgb("+red +","+green +","+blue +")";
}
});
new ayxColorPicker("container2", {
autoHide : false,
onUpdate : function(colorPicker, red, green, blue) {
$('update2').style.backgroundColor = "rgb("+red +","+green +","+blue +")";
},
onChange : function(colorPicker, red, green, blue) {
$('change2').style.backgroundColor = "rgb("+red +","+green +","+blue +")";
}
});
{Fichier : ayxColorPicker.js}
function ayxColorPicker(div, options)
{
var _div (typeof(div) "string") ? $(div) : div;
var _options = options || {};
var _autoHide = _options.autoHide || false;
var _nbStep = _options.nbStep || 50;
var _onChange = _options.onChange || false;
var _onUpdate = _options.onUpdate || false;
var _that = this;
this.hide = function()
{
_div.style.display = 'none';
};
this.show = function()
{
_div.style.display = 'block';
};
function buildHtml()
{
var table = document.createElement("table");
table.setAttribute("cellspacing", 1);
table.style.border = "1px solid black";
table.style.margin = "0";
table.style.padding = "0";
for (var red=0; red
30 avril 2008 à 10:49
Bon code !!!
Pas de note pour l'instant... j'attend de voir si tu va faire des modifications...
Maintenant les remarques un peu moins gentilles :
Je ne vois pas ou est le div qui est censé afficher la couleur qui est survolé !!!
Je pense également qu'il est peut être un peu trop étendu peu être que tu pourrai le faire de la forme d'un carré et suppimer quelque colone par exemple les couleur foncé on dirai qu'elles sont en trois exemplaires chacune !!!
a++