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 more fiddles for launch in fiddle
feature
#19849
Conversation
Co-Authored-By: Mark Lee <malept@users.noreply.github.com>
Co-Authored-By: Samuel Attard <samuel.r.attard@gmail.com>
Co-Authored-By: Samuel Attard <samuel.r.attard@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love this; thank you for the PR!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for tackling this :).
Reviewing on iPad sucks😅 – but what i can say rn is that you‘re missing the end backticks of fiddle='url'
(instead of fiddle='url
).
Co-Authored-By: Charles Kerr <ckerr@github.com>
Co-Authored-By: Charles Kerr <ckerr@github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I'm fine with merging this once @deermichel is happy
@deermichel Can you take another look of this PR? |
Yes, I'll do that tmw 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Found some last changes.
Additionally on my German keyboard layout, the global shortcuts don't really work like expected: Ctrl doesn't work at all and Y & Z are swapped - but I think that's an Electron problem 😬
}); | ||
|
||
// and load the index.html of the app. | ||
mainWindow.loadFile('index.html'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm there is no index.html
in this fiddle - we can remove this line or add a simple html
y: externalDisplay.bounds.y + 50 | ||
}) | ||
win.loadURL('https://electronjs.org') | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how about an else case with a console.log("no ext display or so")?
// Adding a global shortcut to electron | ||
// | ||
// For more info, see: | ||
// https://electronjs.org/docs/api/screen |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should point to the global shortcut docs :)
@cvaldez98 Can you address the review above? |
I'm closing this PR since there has been no activity for a long time, please let me know if you still want to work on this and I'll reopen this PR. |
Description of Change
Add in more fiddles to demonstrate the
screen
andaccelerator
module.This is adding more code examples that can have the
Launch in Fiddle
feat on electronjs.org .The code examples are slightly modified to help the user out, but I'm open to changes of course.
Follow up from #19759
Notes: none
cc @MarshallOfSound @codebytere @ckerr @deermichel