search

InfiniteAmmoInc / Yarn

934 stars 93 forks source link

yarn - electron - lots of new features #80

Closed blurymind closed 5 years ago

blurymind commented 5 years ago

This also adds the feature of showing tags and title of active nodes: bondagejsnodedata2

As well as fixes some code that can make easier to embed in other electron software

Tags and node titles can be used to trigger events in game, such as change of avatars or animations during cutscenes

Also updating bondage.js after I made some changes: https://github.com/jhayley/bondage.js/pull/39#issuecomment-417941721 https://github.com/jhayley/bondage.js/pull/40 https://github.com/jhayley/bondage.js/pull/41

you can download a build here btw: https://github.com/blurymind/Yarn/releases/tag/v0.3.2 (outdated build)

added after build:

blurymind commented 5 years ago

@charblar and @julsam requested me to unbeautify the code, which I ran beautify on in a previous commit

-That is included in this pull request

blurymind commented 5 years ago

@charblar can you review this please? I added some new features, merged changes from master/electron (here), updated bondage.js to the latest version

blurymind commented 5 years ago

you can download a build here btw: https://github.com/blurymind/Yarn/releases/tag/v0.3.2

the linux build hasnt been tested and was cross compiled

blurymind commented 5 years ago

fixed #78

blurymind commented 5 years ago

fixed #83

blurymind commented 5 years ago

reverted fix for #78 as it is already in another pull here: #79

blurymind commented 5 years ago

added new bbcode and [[ autocompletion features + bb format preview features

blurymind commented 5 years ago

added editor preview bbcode mode, still needs a few small fixes before its ready

blurymind commented 5 years ago

added optional spell checking - exposing it to the user. Had to make a few changes to: https://github.com/swenson/ace_spell_check_js/pull/3 in order to get it to also be able to disable spell checking

blurymind commented 5 years ago

added a demo to youtube. See it here https://www.youtube.com/watch?v=Cyr8nbjDxB4

@charblar look at these sweet new features :p why you not review

blurymind commented 5 years ago

Added a color picker for bbcode fonts (using the spectrum.js)

ezgif com-optimize

blurymind commented 5 years ago

Preview mode now has the same scroll position as ace editor. It also has a row counter

blurymind commented 5 years ago

I added a bbcode toolbar, moved the counter to preserve vertical space, check it out

blurymind commented 5 years ago

I need to fix the '#' making text after greyed - my suspicion is that this is part of the ace editor theme settings somewhere. Need to also move the addlink menu to the toolbar as well as add command tagging <<>>

blurymind commented 5 years ago

fixed the annoying greying out after hashtags, added a context menu to the editor as well as autocompletion of common words and suggestions for any misspelled words

blurymind commented 5 years ago

Added night mode to the editor. Toggling it will invert all the light colors which the editor currently uses

blurymind commented 5 years ago
blurymind commented 5 years ago

replaced typo.js with nspell -as typo.js hasnt seen an update in over 3 years, it has a serious delay on some word suggestions and is much less maintained. Made pull here too https://github.com/swenson/ace_spell_check_js/pull/4

blurymind commented 5 years ago

I think I'm getting closer to a milestone on this. Will do a bit of re-factoring and testing, then do another build

blurymind commented 5 years ago

well anyways, if nobody is maintaining this repository, I might have to make my fork of this project a new project. :)

@julsam has added node resizing and fixed a bug, so I want to get his changes into my fork, but his pull is still not being reviewed either

desplesda commented 5 years ago

I’ll review this in the next 48h!

desplesda commented 5 years ago

This is really, really good.

When should we consider merging this into master?

blurymind commented 5 years ago

Thank you! I will also upload a portable electron version soon, so people won't have to build it to try it =)

On Mon, Jan 21, 2019, 9:29 AM Jon Manning <notifications@github.com wrote:

Merged #80 https://github.com/InfiniteAmmoInc/Yarn/pull/80 into electron.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/InfiniteAmmoInc/Yarn/pull/80#event-2085852104, or mute the thread https://github.com/notifications/unsubscribe-auth/AGMbVaIZM6ThmkXvla0WD-Xzu9lXylZNks5vFYiHgaJpZM4WWu-s .

desplesda commented 5 years ago

Something that we should discuss soon is setting up a CI server of some kind to automate the builds. Travis CI has worked well for Yarn Spinner.

lazerwalker commented 5 years ago

Just to chime in on the CI server front, this is something I'm dealing with right now for https://github.com/lazerwalker/tracery-dot-app.

For Windows builds, you want Appveyor or Azure Pipelines; Travis CI's Windows setup is slow and flaky. You can check out Tracery.app's Appveyor config file, although my deployment setup is a bit different from yours since I'm using electron-forge/electron-builder to manage that instead of vanilla Electron. It's set up really nicely to automatically upload a new prerelease binary to the GitHub 'releases' section of the repo every time code is pushed/merged.

I'm still figuring out how to properly generate macOS builds on CI (since you ideally need to both codesign and notarize builds to avoid users needing to jump through onerous "this app is dangerous!" security hoops), but the data I've gathered so far suggests CircleCI is going to be the simplest way to configure a CI setup that can properly codesign/notarize given proper credentials.

blurymind commented 5 years ago

I made some builds here, but they are only for windows https://github.com/blurymind/Yarn/releases/tag/0.3.5

server builds would be fantastic to have!