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

version 0.2.0 #35

Merged
merged 14 commits into from Apr 11, 2016
Merged

version 0.2.0 #35

merged 14 commits into from Apr 11, 2016

Conversation

ghost
Copy link

@ghost ghost commented Apr 11, 2016

I should have created a pull request for each feature, but a bit short on time, sorry. You can always revert parts you don't want.

Changelog:

  • Merged "remove rightclick context menu" from firestack's fork
  • Added navigation: WASD and arrow keys move the canvas in all directions and zoom now works as on Google Maps (zooming in/out according to the cursor position). I personally can't use Yarn without this feature
  • Fix encoding. Replaced readAsBinaryString with readAsText, because JSON files with "non-standard" letters (öäüß) don't work at the moment. I checked only the "Open..." menu with a JSON file and haven't checked XML and Twee. I assume that this change is safe, but can be tested better.
  • Main menu doesn't use a shared container anymore. The gap between the search pane and the buttons drives me crazy, be cause it looks like zoom and select should work there, but they don't
  • Fixed the </bosy> typo (was it a typo?)

The version is bumped to 0.2.0 because of the readAsText change.

Firestack and others added 14 commits February 22, 2016 14:17
Zoom and select don't work in the gap between the search pane and
control buttons, because there is a "control" element there.
This is a bit frustrating from the UX point of view.
This allows us to cache DOM elements during the app initialization process
Move canvas with ASDW or arrow keys
Zoom works as on Google Maps
for charachters like äöüß
@InfiniteAmmoInc InfiniteAmmoInc merged commit 2636925 into InfiniteAmmoInc:master Apr 11, 2016
@InfiniteAmmoInc
Copy link
Owner

Awesome! :)

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.

1 participant