Use .focus() instead of .select()?

[sindresorhus]

I can't remember why we started using .select() in the first place, but @IgorMinar has a point:

because when I doubleclick on a item I typically want to change only a small part of it. select() highlights the whole text and forces me to deselect it. if I do want to select the whole thing I can just triple click on the item and then I'll get this behavior.

I think we should change it back.

[addyosmani]

+1 on changing.
On Jul 1, 2012 3:11 PM, "Sindre Sorhus" <
reply@reply.github.com>
wrote:

I can't remember why we started using .select() in the first place, but
@IgorMinar has a point:

because when I doubleclick on a item I typically want to change only a
small part of it. select() highlights the whole text and forces me to
deselect it. if I do want to select the whole thing I can just triple click
on the item and then I'll get this behavior.

I think we should change it back.

---

Reply to this email directly or view it on GitHub:
#203

[sindresorhus]

Converted all apps to use .focus, but even so, there are some that still selects the text. Need to figure out why...

The following still selects the text:

• Dojo
• Emberjs
• Spine
• YUI
• Backbone Require
• Knockout Require
• Stapes
• Stapes Require
• Epitome
• Soma.js
• Backbone Marionette

Edited by arthurvr to remove deleted apps.

[sindresorhus]

@jsoverson just pinging you incase you would be interested in tackling any of these :)

[passy]

Is the list still up to date?

[sindresorhus]

@passy partly. those apps is not fixed, but there has been included additional apps since then that might have the same issue.

[arthurvr]

Emberjs

Seems to be working correctly for me. Can you confirm that, @sindresorhus?

[arthurvr]

Ticked ember, then.

[arthurvr]

I never noticed it back then, but since when I ticked the ember app and removed the dropped apps, this can be closed :) Yay!

[sindresorhus]

🎉🎉🎉