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

Add Discord invitation on the website, README.md, and CONTRIBUTING.md #538

Merged
merged 2 commits into from
Mar 22, 2021

Conversation

Panquesito7
Copy link
Member

Things added/changed:

  • Add Discord invitation on the website, README.md, and CONTRIBUTING.md.
    • I added a new SVG for the Discord logo, but the viewBox (and probably other stuff) are not correct as per the contributing guidelines. Its width and height are modified in style.css, though. Maybe the bot should ignore this SVG.

...`README.md`, and `CONTRIBUTING.md`
@@ -8,6 +8,14 @@ html {
*, *::after, *::before {
box-sizing: inherit; }

.discord-logo:before {
Copy link
Member

Choose a reason for hiding this comment

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

This one is my bad. Should be
.discord-logo::before {

Copy link
Member Author

Choose a reason for hiding this comment

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

Are you sure? Other instances in this file use :before.

Copy link
Member

Choose a reason for hiding this comment

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

Hmm, I double checked. Seems like both are fine. See second answer of this

Copy link
Member

@Thomas-Boi Thomas-Boi left a comment

Choose a reason for hiding this comment

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

Just need to change the :before CSS to ::before.

Thomas-Boi
Thomas-Boi previously approved these changes Mar 20, 2021
@Thomas-Boi
Copy link
Member

Well, since this svg is not the icons/ folder, the bot should have no problem with it. Besides that, I'm cool with this PR. I'll wait for amacado to ensure the styling of the Discord icon is good before I merge this.

Good job 👍

Copy link
Member

@amacado amacado left a comment

Choose a reason for hiding this comment

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

Regarding this discussion we should point out, that the discord server is unoffical help/support server and the project is still beeing managed via github (issues).

Copy link
Member

@Thomas-Boi Thomas-Boi left a comment

Choose a reason for hiding this comment

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

This looks good to me 👍

@Thomas-Boi Thomas-Boi merged commit c53595f into develop Mar 22, 2021
@Panquesito7 Panquesito7 deleted the panquesito7/add/discord_channel branch March 22, 2021 23:15
@amacado amacado mentioned this pull request Apr 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants