Skip to content

Make Dart constructor calls pop out in light mode. #3327

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

Merged
merged 2 commits into from
Mar 29, 2019

Conversation

jacob314
Copy link
Contributor

Constructors are already nicely visible in Darkula mode.

Having this separate color style format will be handy for future additional UI as Code related
color tweaks. Here are screenshots of what this looks like in action. The Dark theme isn't
impacted but the light theme is.

Screen Shot 2019-03-27 at 3 53 15 PM

Screen Shot 2019-03-27 at 3 53 40 PM

@jacob314
Copy link
Contributor Author

This fixes
#3225

FYI @InMatrix in case there is a different color we should use.

Long term it would be nice but not crucial if this color default is added to the Dart Plugin but it makes sense to add it to the Flutter plugin so we can get quick feedback and iterate if need be.
For example, this color is quite aggressively light.

@InMatrix
Copy link

Can we try this color: #2196f3. It's blue[500] in material design and it feels slightly more solid than the one used now.

@jacob314 jacob314 requested a review from InMatrix March 28, 2019 00:12
@jacob314 jacob314 force-pushed the code_color_scheme branch from 6ffdde7 to 1778aa2 Compare March 29, 2019 18:18
@jacob314
Copy link
Contributor Author

That new color looks really nice and does make me wonder about creating a full material design colors only color scheme for Dart/Flutter. :)
Screen Shot 2019-03-29 at 11 04 17 AM

Copy link

@InMatrix InMatrix left a comment

Choose a reason for hiding this comment

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

LGTM

@jacob314 jacob314 merged commit 38161bd into flutter:master Mar 29, 2019
alexander-doroshko pushed a commit to alexander-doroshko/flutter-intellij that referenced this pull request Jan 24, 2020
* Make Dart constructor calls pop out in light mode. Constructors are already
nicely visible in Darkula mode.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants