-
Notifications
You must be signed in to change notification settings - Fork 256
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
Adding Info Page for Community Channels #616
base: master
Are you sure you want to change the base?
Conversation
@quozl this time I add all the files in a single commit as they all are added to to create a single page only so is it the correct way or any changes required |
looks very nice! |
Thanks @retrogtx |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- our guide to making commit messages has not been followed,
- the screenshots do not scale,
- the screenshots do not follow browser dark and light mode preferences.
I'd be happy not to have screenshots.
</div> | ||
</div> | ||
</div> | ||
</section> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing newline at end of file.
.join-mat:active { | ||
top:0.1em; | ||
color: #fff !important; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do not change css/airspace.css, as this will hinder our merge of upstream change. Change css/custom.css instead, or another css file.
counter.innerText = `${target}+`; | ||
} | ||
|
||
counter.innerText = target.toLocaleString() + '+'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have included changes relating to the counters pull request. Are you trying to sneak these past? 😁
My idea for screenshots were to help a newbie understand what to see when navigating creating an account for the first time. I don't think you should use those screenshots. This is more of what I was thinking of: In fact, that one screenshot is probably sufficient. We may want to also find an "official" how-to for registering to Matrix for the first time that we can link to, so that we know we're sending people to something that's probably always up-to-date, and that we don't have to maintain ourselves. |
@pikurasa okay so i ll be deleting the images and also trying to add more text information about the make account creation and also try to search some matrix link to their guide for creating an account for matrix and again similar for discord too. |
Yes. If you have any questions, please let me know. |
Yeah @pikurasa I ll be making this soon as in my device I m getting many dependencies issue with the www repo I m trying to reset it @haroon10725 just wait for some as I did many changes if It not possible by you can surely take this issue but wait for sometime atleast |
@omsuneri No problem, will continue from here. |
@haroon10725 I ll inform you if it works me on 01/16 meet will discuss about this topic |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is the only sample image that we really need, but I suppose it doesn't hurt to have the others...
@@ -54,7 +54,7 @@ <h4 class="footerHeaderStyle align-center-md"><b>Development</b></h4> | |||
<li class="footerListStyle"><a href="{{ site.baseurl }}/develop">Read about development</a></li> | |||
<li class="footerListStyle"><a href="https://wiki.sugarlabs.org/go/Mailing_Lists">Join mailing list</a></li> | |||
<li class="footerListStyle"><a href="https://help.sugarlabs.org/en/">Documentation</a></li> | |||
<li class="footerListStyle"><a href="https://matrix.to/#/#sugar:matrix.org">Chat with us on Matrix</a></li> | |||
<li class="footerListStyle"><a href="{{ site.baseurl }}/chat-with-us-on-matrix">Join the Community</a></li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think "Chat with the community" or "Chat with us" would be better.
<div class="row justify-content-center"> | ||
<div class="col-md-8 col-md-push-2" > | ||
<div class="section-title text-center"> | ||
<h2>SUGARLABS ON MATRIX ELEMENTS</h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Sugar Labs" should be two words here and elsewhere on this page.
<div class="row justify-content-center"> | ||
<div class="col-md-8 col-md-push-2" > | ||
<div class="section-title text-center"> | ||
<h2>SUGARLABS ON DISCORD</h2> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be "Sugarizer on Discord", because Sugarizer is the only SL project on Discord.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will be doing it soon as my www local setup breaks some how and just figuring out the exact reason. For that but unable to find a solution till now but soon I ll do it
Creating Informative Page for the Community Channel
New page in website provides more understanding to the community channel for chat and discussion