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 Spectacle #29

Merged
merged 9 commits into from
Jan 8, 2020
Merged

Add Spectacle #29

merged 9 commits into from
Jan 8, 2020

Conversation

miguelmartins17
Copy link
Contributor

Adding some info about Spectacle

Adding some info about Spectacle
readme.md Outdated
If you want to be super fast and efficient "You need to learn how to use Spectacle".
Once you learn some of the shortcuts you will just need to use your "Muscle Memory" to use them.

# Spectacle Installation
Copy link
Member

Choose a reason for hiding this comment

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

When capturing a screenshot of a whole screen, try to include only the tab you have in focus rather than having a lot of "noise" ... in the case of the installation you are showing your musical preferences which is not relevant to the instructions.
https://github.com/dwyl/dev-setup/blob/a08e7e2ce3ddcb5cfb47c67a52ffd5e7b3e29f0b/readme.md#spectacle-installation
image
Consider re-taking the screenshot with just the browser tab that is relevant. 💭

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Already changing the Screenshot. 👋

Copy link
Member

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

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

@miguelmartins17 this is looking great.
The only thing I would add is a mention that Spectacle is for Mac.
And I would take a more focussed screenshot avoiding any noise from other tabs.
Thanks! ✨

@miguelmartins17
Copy link
Contributor Author

Im going to add that mention right now and add to an alternative app for windows too.
Changing the Screenshot too. 👌

readme.md Outdated
# Spectacle Installation

First of all access the website :https://www.spectacleapp.com/

![Spectacle](https://i.postimg.cc/28frXpZL/9688-AE1-C-A633-427-D-91-EE-56-CABA0-DF2-D6-1-105-c.jpg)
![Spectacle](https://cdn1.imggmi.com/uploads/2020/1/7/00355c30da035c3f844a03854784892c-full.png)
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@nelsonic Sorry I didn't see that the image was like that

Copy link
Member

Choose a reason for hiding this comment

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

@miguelmartins17 remember that you can drag-and-drop images (or paste them) into any GitHub Comment and not have to use another image service.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@nelsonic I didn't know that I could drag images. So I was using an upload program.

@miguelmartins17
Copy link
Contributor Author

@nelsonic I was trying to add the Image in the Spectacle file by simply drag in to the file but don't work.

Adding the Image of the Website.
@nelsonic
Copy link
Member

nelsonic commented Jan 8, 2020

Drag-and-drop does not work in the Atom Editor. (sadly...)
But dragging into a GitHub issue Comment and then copying the URL works.
That's how we do it for all our projects.

readme.md Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
readme.md Outdated Show resolved Hide resolved
Copy link
Member

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

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

@miguelmartins17 please go through the suggestions and use the "Add suggestion to batch" button:
add-suggestion-to-batch

Then commit the changes.
Thanks! 👍

miguelmartins17 and others added 5 commits January 8, 2020 11:51
Fixing

Co-Authored-By: Nelson <nelson+github@dwyl.io>
Fixing Titles

Co-Authored-By: Nelson <nelson+github@dwyl.io>
Fixing titles

Co-Authored-By: Nelson <nelson+github@dwyl.io>
Fixing titles

Co-Authored-By: Nelson <nelson+github@dwyl.io>
Fixing titles

Co-Authored-By: Nelson <nelson+github@dwyl.io>
Copy link
Member

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

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

@miguelmartins17 looks great. thanks for updating the headings. 👍

@nelsonic nelsonic merged commit 73d9724 into dwyl:master Jan 8, 2020
@miguelmartins17
Copy link
Contributor Author

miguelmartins17 commented Jan 8, 2020

@nelsonic No Problem and Sorry for that mistake 👋

@nelsonic
Copy link
Member

nelsonic commented Jan 8, 2020

No "mistakes". Only learning opportunities. 😉

@miguelmartins17
Copy link
Contributor Author

Sure but still Thanks! 😄

iteles added a commit that referenced this pull request Apr 15, 2020
This was referenced Apr 15, 2020
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.

2 participants