JavaScriptUtilities icon indicating copy to clipboard operation
JavaScriptUtilities copied to clipboard

FakeSelect:select

Open SouroucheB opened this issue 11 years ago • 3 comments

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 avatar Sep 16 '14 09:09 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 avatar Sep 16 '14 10:09 SouroucheB

Petite correction :

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

SouroucheB avatar Sep 16 '14 11:09 SouroucheB

Merci pour tes retours, je regarde ça :)

Darklg avatar Sep 20 '14 15:09 Darklg