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] Fix URL hints not showing #4979

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

[CLOSED] Fix URL hints not showing #4979

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

Comments

@core-ai-bot
Copy link
Member

Issue by WebsiteDeveloper
Friday Oct 04, 2013 at 13:57 GMT
Originally opened as adobe/brackets#5422


#4949

@redmunds because the issue is assigned to you.


WebsiteDeveloper included the following code: https://github.com/adobe/brackets/pull/5422/commits

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Thursday Oct 10, 2013 at 16:31 GMT


This looks good. Since that if statement was removed, this code:

        var tagInfo,
            query,
            tokenType;

        this.editor = editor;

        tagInfo = HTMLUtils.getTagInfo(editor, editor.getCursorPos());
        query = null;
        tokenType = tagInfo.position.tokenType;

Should be cleaned up to:

        var tagInfo   = HTMLUtils.getTagInfo(editor, editor.getCursorPos()),
            query     = null,
            tokenType = tagInfo.position.tokenType;

        this.editor = editor;

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Tuesday Oct 15, 2013 at 22:19 GMT


@WebsiteDeveloper Reminder: just a small code cleanup change remaining.

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Thursday Oct 17, 2013 at 02:52 GMT


Merging. No need to hold this up for a little code cleanup.

@core-ai-bot
Copy link
Member Author

Comment by WebsiteDeveloper
Friday Oct 18, 2013 at 11:17 GMT


@redmunds sry i was away a few days, but i'm glad it is finally merged.

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