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 new page that links to Discord Server #437

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lilliangonick
Copy link
Contributor

Changes

Added a new page on the ButtonBar. The "Discord" button links directly to the Discord permalink.

Type of change

New feature

Related Issues

close #383

@nathanzzhang
Copy link
Contributor

Nice change! However, I think it looks a bit out of place on the nav bar tbh. I think it might need some more discussion. Regardless, good job!

@katelynsyu
Copy link
Collaborator

katelynsyu commented Feb 26, 2024

Nice change! However, I think it looks a bit out of place on the nav bar tbh. I think it might need some more discussion. Regardless, good job!

@nathanzzhang @jakobreinwald @maggieelli @jamesmwu @EinarBalan
Do you think it would be better if we changed the the text that appears in the nav bar that links to the discord to "Contact (Us)" or something like that?

I still agree with the original issue that raised that we should have a link to our discord available to people who are interested without them having to click around to find it. Other issues that arise are that we might get more bots/spammers though.

@jamesmwu
Copy link
Contributor

I think that having the text on the nav bar as "Discord" is fine-- I feel like Contact Us implies email usually. And while it's not typical, I think having the link directly on the nav bar is fine since it's our club's main way of communication and up to date information.

@maggieelli
Copy link
Member

I kind of agree with Katelyn that "Discord" is a bit odd to have on our nav bar since it feels out of place compared to the other pages, but I get why we want to have the Discord link more accessible. Personally I think at least changing the text to "Join Us" or something would be much better, but also I think it still is a bit odd to have it on the nav bar since it's not directing to a page, but rather a link.

Do we think the glowy gradient link on our home page under our banner that says "Want to learn more about our internship program..." is necessary to have right now? If not, we could replace that button to something like "Join our Discord to get up-to-date information about our events!" or something

@nathanzzhang
Copy link
Contributor

I actually like the idea of replacing the glowy gradient with a link to our discord with some relevant text instead of adding it to the navbar. Although with that change, the button about half a page down "Join the ACM Hack Discord" is a bit irrelevant LOL

@katelynsyu
Copy link
Collaborator

Guys lets just nix the discord link in the nav bar entirely and just create a page hack.uclaacm.com/discord that links to the discord so we can tell people to go there (it's an easier link to remember and type)

@jamesmwu jamesmwu self-requested a review May 19, 2024 23:32
Copy link
Contributor

@jamesmwu jamesmwu left a comment

Choose a reason for hiding this comment

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

Could you remove the current link as it is and instead change the "Apply" button to say "Discord" and link to the Discord?

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.

Add /discord page
5 participants