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

Submission - rish-16 #2

Open
1 of 2 tasks
rish-16 opened this issue May 28, 2020 · 6 comments
Open
1 of 2 tasks

Submission - rish-16 #2

rish-16 opened this issue May 28, 2020 · 6 comments
Labels
submission.ACCEPTED Accepted Submission for OpenHack submission A submission for OpenHack

Comments

@rish-16
Copy link

rish-16 commented May 28, 2020

Submission Details

Type of Submission:

  • 3 Pull requests
  • Self-initiated open-source project

Link to open-source project: https://github.com/rish-16/Angelfire (leave a ⭐️if you 💛it!)

Additional information (optional):
Angelfire is a JavaScript library that allows users to build fully-customisable context menus (the menu that shows up when you right-click something on a webpage) and drop-down menus.

You can add styles to the menus and can even add sub-menus (WIP). It takes in a config object containing the different options in the menu and the respective on-click events.

image

When building menus for any type of element, I've had to write a lot of code to create them. With Angelfire, you can quickly generate one with only a few lines of code.

More info on the README.

Comment below if you have any feedback!

@rish-16 rish-16 added the submission A submission for OpenHack label May 28, 2020
@rish-16 rish-16 changed the title Submission [WIP] - @rish-16 Submission - @rish-16 May 28, 2020
@rish-16 rish-16 changed the title Submission - @rish-16 Submission - rish-16 May 28, 2020
@chrisgzf
Copy link
Member

Hi @rish-16, I think this is very good effort for a day of work!

Before we accept this:

  • in the spirit of open-source, you'd want your project to be used by others as well, can you consider adding fallback fonts for non-macOS users?

Just some other tips:

  • most projects which use git encourage writing commit messages in imperative present tense as convention. It's not a standard but I would encourage you to do so from now on.
  • try to keep your commit messages descriptive (i.e. we should be able to easily tell what each commit changes by reading the commit message)

Once you make the change to the fonts, my team and I are ready to accept your submission. But since we are still at a very early stage of OpenHack, I'd like to offer you an (additional) challenge: write tests for your code (you could look up jest as a starting point) and achieve > 70% code coverage, and then proudly put a badge on your repository.

Have fun!

@rish-16
Copy link
Author

rish-16 commented May 29, 2020

Hey @chrisgzf thanks for the looksee!

I'll add some fonts for windows as well.

Will also look up Jest. Haven't used it before. The idea of using badges sounds interesting. Will see what all I can add to the README.

Cheers :D

@rish-16
Copy link
Author

rish-16 commented May 29, 2020

I've changed the fonts! @chrisgzf

Will move onto the Jest testing.

Also, thanks for organising OpenHack! It was super fun making this project (had it at the back of my mind but never really had the chance to implement it).

@rish-16
Copy link
Author

rish-16 commented Jun 1, 2020

@chrisgzf any more things I have left before having my submission finalised?

@chrisgzf
Copy link
Member

chrisgzf commented Jun 1, 2020

Hi @rish-16, congratulations! Your OpenHack submission is now accepted!

Please visit https://openhack.nushackers.org/congrats and fill up the form so that we can capture your details for the prizes.

We hope you had fun working on the challenge and we hope you will continue making open-source contributions, even outside of the event!

@chrisgzf chrisgzf added the submission.ACCEPTED Accepted Submission for OpenHack label Jun 1, 2020
@rish-16
Copy link
Author

rish-16 commented Jun 2, 2020

Will do, thanks Chris! Was a fun experience :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
submission.ACCEPTED Accepted Submission for OpenHack submission A submission for OpenHack
Projects
None yet
Development

No branches or pull requests

2 participants