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

🎄 2017 Winter Holidays update #486

Merged
merged 157 commits into from
Jan 12, 2018
Merged

Conversation

Akryum
Copy link
Member

@Akryum Akryum commented Dec 31, 2017

@Akryum Akryum mentioned this pull request Jan 4, 2018
@@ -49,7 +49,7 @@ export default {
data () {
return {
filter: '',
isChrome: typeof chrome !== 'undefined' && chrome.devtools
isChrome: typeof chrome !== 'undefined' && !!chrome.devtools
Copy link
Member Author

Choose a reason for hiding this comment

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

Quick Edit boolean! 😸

}
}
```

3. You can change `'code'` with the editor you are using, see the [supported editors list](https://github.com/lahmatiy/open-in-editor#editor).
3. The editor to launch is guessed. You can also specify the editor app with the `editor` option. See the [supported editors list](https://github.com/lahmatiy/open-in-editor#editor).
Copy link
Contributor

Choose a reason for hiding this comment

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

Supported editors was added in the launch-editor package.

https://github.com/yyx990803/launch-editor#supported-editors

@yyx990803
Copy link
Member

Absolutely amazing job @Akryum, every feature works flawlessly!!!

@yyx990803 yyx990803 merged commit 24812e5 into vuejs:master Jan 12, 2018
@fixmycode
Copy link

Chrome is notifying me that this release requires new permissions, there should be a note somewhere explaining why.

@dttvinh
Copy link

dttvinh commented Jan 13, 2018

Seems like the new permission has been removed in the latest commit. Looking forward to a new version soon 👍 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.