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

docs: add exemplary fiddle for launch in fiddle feat #19759

Merged
merged 3 commits into from Aug 15, 2019

Conversation

deermichel
Copy link
Contributor

Description of Change

Adds a simple fiddle to demonstrate the screen module.

Used to verify the Launch in Fiddle feat on electronjs.org.

ref electron/electronjs.org-old#2848

cc @MarshallOfSound @codebytere @ckerr

Checklist

  • PR description included and stakeholders cc'd
  • relevant documentation is changed or added
  • PR title follows semantic commit guidelines

Release Notes

Notes: none

@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Aug 14, 2019
Copy link
Member

@MarshallOfSound MarshallOfSound left a comment

Choose a reason for hiding this comment

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

Not going to block this PR as it is just a single example for a Proof Of Concept.

But this isn't going to scale well till we solve my above comment

docs/api/screen.md Outdated Show resolved Hide resolved
@MarshallOfSound
Copy link
Member

#19759 (comment)

@deermichel
Copy link
Contributor Author

Oki, defaulting to root. Ready to merge @MarshallOfSound :)

Copy link
Member

@ckerr ckerr left a comment

Choose a reason for hiding this comment

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

👍 on merging this as an initial version rather than the end goal.

@MarshallOfSound could you go into a little more detail about the inference you have in mind? It's not clear to me from your comments what specifically we should be doing

@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Aug 15, 2019
@MarshallOfSound MarshallOfSound merged commit e590954 into master Aug 15, 2019
@release-clerk
Copy link

release-clerk bot commented Aug 15, 2019

No Release Notes

@MarshallOfSound MarshallOfSound deleted the intern-fiddle-test branch August 15, 2019 22:37
@trop
Copy link
Contributor

trop bot commented Aug 15, 2019

I was unable to backport this PR to "6-0-x" cleanly;
you will need to perform this backport manually.

@trop
Copy link
Contributor

trop bot commented Aug 15, 2019

I have automatically backported this PR to "7-0-x", please check out #19785

deermichel pushed a commit that referenced this pull request Aug 15, 2019
* add fit-screen

* new url format

* nit
@trop
Copy link
Contributor

trop bot commented Aug 15, 2019

A maintainer has manually backported this PR to "6-0-x", please check out #19786

codebytere pushed a commit that referenced this pull request Aug 19, 2019
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