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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add space in the button for clarity #88 #95

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

Conversation

BlackSou
Copy link

@BlackSou BlackSou commented Nov 1, 2019

Fixes #0000 (<=== Add issue number here)

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

  • PR is descriptively titled 馃搼 and links the original issue above 馃敆
  • code is in uniquely-named feature branch and has no merge conflicts 馃搧
  • screenshots/GIFs are attached 馃搸 in case of UI updation
  • ask @publiclab/reviewers or the user who published the issue for help, in a comment below

We're happy to help you get this ready -- don't be afraid to ask for help!

Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software

Thanks!

@CodeSarthak
Copy link

Hi @BlackSou , good job on opening your first PR on this repository 馃憤 .

Can you please add the issue number this PR resolved and also attach a screenshot of the button with the added space?

Happy Contributing 馃槃
Thank you.

@LeoDog896
Copy link

This pull request is outdated and should be moved to pull #96 -- essentially does the same thing with more context.

@LeoDog896 LeoDog896 mentioned this pull request Jan 13, 2020
4 tasks
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.

None yet

3 participants