FakeSelect:select

[SouroucheB]

Hello,

Je pense qu'il serait préférable de changer ceci dans le fichier FakeSelect:

this.cover.html(this.el.children(':selected').text());

en :

this.cover.html(this.el.find('option:selected').text());

de façon à pouvoir inclure un "optgroup" dans le select ;)

[SouroucheB]

Et deuxième petite chose :

Sur les selects, en partant du principe qu'on a besoin de cliquer pour apporter le focus à l'input, ne serait-il pas mieux d'écrire cela pour l'attribution de la classe "has-focus" ?

this.el.on('click', function() {
self.wrapper.toggleClass('has-focus');
}).on('blur', function() {
self.wrapper.toggleClass('has-focus');
});

plutôt que ça :

this.el.on('focus', function() {
self.wrapper.addClass('has-focus');
}).on('blur', function() {
self.wrapper.removeClass('has-focus');
});

Parce que le problème que j'ai rencontré avec ton code de base, c'est que lorsqu'on clique à nouveau sur le select pour fermer les options, le wrapper ne perd pas le focus pour autant, ce que je trouve personnellement pas user friendly.

[SouroucheB]

Petite correction :

this.el.on('click', function() {
self.wrapper.toggleClass('has-focus');
}).on('blur', function() {
self.wrapper.removeClass('has-focus');
});

[Darklg]

Merci pour tes retours, je regarde ça :)