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

added github dark theme #5154

Merged
merged 3 commits into from
May 12, 2023
Merged

added github dark theme #5154

merged 3 commits into from
May 12, 2023

Conversation

bhagyajkumar
Copy link
Contributor

Issue #, if available:

Github dark theme was added.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Comment on lines 127 to 142
.ace-github .ace_constant.ace_buildin {
color: #0086B3;
}

.ace-github .ace_support.ace_function {
color: #0086B3;
}

.ace-github .ace_comment {
color: #90bbe1;
font-style: italic;
}

.ace-github .ace_variable.ace_language {
color: #ffffff;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

These lines here are for ace-github and not ace-github-dark and they seem to be duplicates from above, I think you forgot to remove these?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for your feedback and for pointing out the duplicated lines in the code. You are correct that these lines were intended for ace-github and not ace-github-dark and I apologize for the oversight. I will remove these lines as soon as possible to ensure that the code is clean and efficient.

@codecov
Copy link

codecov bot commented May 10, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (c6ec82c) 86.68% compared to head (3ba4163) 86.68%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5154   +/-   ##
=======================================
  Coverage   86.68%   86.68%           
=======================================
  Files         560      560           
  Lines       44199    44199           
  Branches     6850     6850           
=======================================
  Hits        38312    38312           
  Misses       5887     5887           
Flag Coverage Δ
unittests 86.68% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@akoreman akoreman merged commit 3f29428 into ajaxorg:master May 12, 2023
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

Successfully merging this pull request may close these issues.

2 participants