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

Suggestion - Improve Dart Colours #8

Open
aedwardg opened this issue Nov 14, 2020 · 7 comments
Open

Suggestion - Improve Dart Colours #8

aedwardg opened this issue Nov 14, 2020 · 7 comments
Assignees
Labels
bug 🐛 Something isn't working enhancement ✨ New feature or request

Comments

@aedwardg
Copy link
Contributor

Hi @trallard,

What do you think about the current Dart colors?

image

To me they don't seem to align as well with the theme as other languages. There is a lot of yellow. It looks like all the class names are yellow and strings are all showing up as the periwinkle color: #7998F2. There are a few other inconsistencies, but those are the big ones that stand out right away.

If you think this is something worth fixing, I'm willing to help out.

@trallard
Copy link
Owner

hey @aedwardg thanks for raising this! I personally do not use Dart but I do agree this does not align with most of the theme approach.
I have a couple of thoughts here:

  • too much yellow - will perhaps need to balance out with the soft pink
  • strings -> will need to match with the overall light turquoise for the theme
  • classes are currently yellow -> should be periwinkle

Since I am not a Dart user, would you be happy to work together in this? I certainly would appreciate having the input from someone who does use Dart and make sure the theme also has a consistent look and feel

@trallard trallard added enhancement ✨ New feature or request bug 🐛 Something isn't working labels Nov 15, 2020
@aedwardg
Copy link
Contributor Author

Sounds good. I'm definitely willing to collaborate on this. I was surprised to see that the coloring was so different from JavaScript, since many aspects of Dart seem to be a hybrid of Java and JavaScript.

@trallard
Copy link
Owner

trallard commented Nov 15, 2020

interesting 🤔 for JavaScript I used Night Owl as a strong reference will also check how it compares for Dart.

I had a similar issue when working with Arm templates - they're basically JSON but the highlighting was so different so I had to do some scope digging

To get started with the PR - would you be able to add a sample Dart script in the /tests directory? I can then start doing some comparisons and we can then move onto scope matching/updating

@aedwardg
Copy link
Contributor Author

Sure, no problem.

Anything specific I should include to cover the bases?

@aedwardg
Copy link
Contributor Author

@trallard I went ahead and PRed some samples for both regular Dart and Dart as it is used with Flutter, since the two differ somewhat.

Just for fun, the Flutter sample is actually the code for a Pitaya Smoothie demo app that I deployed to the web 😂:
https://aedwardg.github.io/pitaya-smoothie-demo/#/

@trallard
Copy link
Owner

trallard commented Dec 2, 2020

Thanks @aedwardg I loooove the demo app ❤️

I have been struggling to put some time onto this but I am going to be on leave soon and will make sure to spend some time here. Thanks for your effort and patience ✨

@aedwardg
Copy link
Contributor Author

aedwardg commented Dec 2, 2020

@trallard not a problem, I know how that goes!
This isn't the most pressing issue, so take however much time you need.

p.s., glad you enjoyed the demo app 🙂 it made writing up a code sample a little more fun.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working enhancement ✨ New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants