Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

Conversation

@redmunds
Copy link
Contributor

This is for #6231

Always select the first value hint. The list of values is usually a relatively short, finite list, so if user happens to want first item in list, this saves them a keystroke.

@ghost ghost assigned RaymondLim Dec 18, 2013
@redmunds
Copy link
Contributor Author

@RaymondLim This is ready for review (I was thinking of a different issue).

@redmunds
Copy link
Contributor Author

@RaymondLim Oops, I pushed some changes meant for #6239 (which is the other issue I was working on) to this branch. You're signed up to review that one also.

Is this ok, or should I keep redo these branches to keep them separate?

@RaymondLim
Copy link
Contributor

It's your call. I'm ok that unrelated unit tests being in this pull request.

@RaymondLim
Copy link
Contributor

@redmunds Actually, you're mixing fix for url code hinting issue with fix for css code hinting issue, not just the unit tests for url code hinting in this pull request. So could you please separate it into two pull requests?

@redmunds
Copy link
Contributor Author

I split this pull request into #6270 and #6271. Closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants