Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CLOSED] CSS code hints improvements #5686

Open
core-ai-bot opened this issue Aug 30, 2021 · 3 comments
Open

[CLOSED] CSS code hints improvements #5686

core-ai-bot opened this issue Aug 30, 2021 · 3 comments

Comments

@core-ai-bot
Copy link
Member

Issue by YuAo
Friday Dec 13, 2013 at 17:38 GMT
Originally opened as adobe/brackets#6242


Select the initial hint on backspace key press

Problem: When you input "col", the code hint plugin will present hints like "color","column-count",etc, with the initial item "color" selected. But, after you press backspace key once, nothing is selected on the hints list.

This patch set the selectInitial = true on implicitChar === null, making the first hint selected on backspace key press.

Clear hints when nothing's left

Problem: When you input "co", then press backspace twice to delete the CSS property, the code hints list remains open with hints like "align-content","align-items",etc.

This patch fixed the problem by returning null for the hints list when this.info.offset === 0.


YuAo included the following code: https://github.com/adobe/brackets/pull/6242/commits

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Thursday Dec 19, 2013 at 16:22 GMT


@YuAo My pull request adobe/brackets#6270 that was merged last night conflicts with your code. Sorry about that. You'll have to merge with the latest master before I can review your latest changes. Let me know if you have any questions about the code change.

@core-ai-bot
Copy link
Member Author

Comment by YuAo
Friday Dec 20, 2013 at 02:02 GMT


@redmunds That's all right. All fixed now.

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Friday Dec 20, 2013 at 17:23 GMT


Thanks. Nice fix. Merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant