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

Highlight matched hints #10389

Merged
merged 1 commit into from
Jan 16, 2015
Merged

Highlight matched hints #10389

merged 1 commit into from
Jan 16, 2015

Conversation

sprintr
Copy link
Contributor

@sprintr sprintr commented Jan 15, 2015

Highlights matched hints in CSS

@sprintr sprintr force-pushed the highlight-css-hints branch from 85c6129 to f09846c Compare January 15, 2015 16:08
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2014 Adobe Systems Incorporated. All rights reserved.
* Copyright (c) 2015 Adobe Systems Incorporated. All rights reserved.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We never update the copyright date on files. This is the date the file was added to code base.

Remove

Highlight matched hints

Reset copyright year

Reset copyright year
@sprintr sprintr force-pushed the highlight-css-hints branch from f09846c to 8d474ea Compare January 15, 2015 18:03
@sprintr
Copy link
Contributor Author

sprintr commented Jan 15, 2015

@redmunds Done!

@redmunds
Copy link
Contributor

Good catch. It looks like same change needs to be made to /src/extensions/default/JavaScriptCodeHints/styles/brackets-js-hints.css.

@sprintr
Copy link
Contributor Author

sprintr commented Jan 16, 2015

JavaScript hints uses more than one rules for hints, depending on priority. It will be a nice idea to have @larz0's opinion on the choice of colors.

@redmunds
Copy link
Contributor

@sprintr

JavaScript hints uses more than one rules for hints, depending on priority.

Good point. I'll just merge this fix for now.

redmunds added a commit that referenced this pull request Jan 16, 2015
@redmunds redmunds merged commit a3068fa into adobe:master Jan 16, 2015
@sprintr sprintr deleted the highlight-css-hints branch January 16, 2015 19:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants