Dark Mode

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add space in the button for clarity #88#95

Open
BlackSou wants to merge 1 commit intopubliclab:mainfrom
BlackSou:fix-simple-data-grapher
Open

Add space in the button for clarity #88#95
BlackSou wants to merge 1 commit intopubliclab:mainfrom
BlackSou:fix-simple-data-grapher

Conversation

Copy link

BlackSou commented Nov 1, 2019 *
edited
Loading

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!

Copy link

CodeSarthak commented Dec 11, 2019

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.

Copy link

tristan-f-r commented Jan 13, 2020

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

tristan-f-r 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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants