-
Notifications
You must be signed in to change notification settings - Fork 27
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
Fix the projects page #97
Comments
|
@rsoika I have scheduled a call with @cesarhernandezgt and @secondsun to work on your ticket. Please feel free to join the call. The joining info can be found in MP calendar (the only session on Friday) |
Fix for some issues #98 |
Hello everyone! Reviewed styling issues:
Question now: Should I add github-syntax-light MIT css (https://github.com/primer/github-syntax-light/blob/master/lib/github-light.css) as I see code is already processed for it on API side? (.pl-c .pl-k and other classes) P.S. github-syntax-light is just few lines of css minified:
|
Thank you @Dexmaster |
Oh, Yes it fully uses API, but to add/remove/edit CSS or JS you will need to do it through WP correct. |
Do it through word press or change our CDN rules to use the EC2 app for the docs and projects pages and leave the blog posts on Wordpress. |
The changes with github-syntax-light MIT where applied today to the website. This fixes eclipse/microprofile-marketing#341 and eclipse/microprofile-marketing#648 |
Need to fix the following issues:
eclipse/microprofile-marketing#648
eclipse/microprofile-marketing#341
@cesarhernandezgt @secondsun can we meet up to discuss this in more details?
The text was updated successfully, but these errors were encountered: