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

Teleport! #1287

Merged
merged 8 commits into from Apr 1, 2017

Conversation

Projects
None yet
4 participants
@only-a-ptr
Copy link
Member

commented Feb 28, 2017

When in character mode, press F2 (inputmap: COMMON_TELEPORT_TOGGLE) to open minimap with telepoints.
screenshot_2017-02-28_01-28-11_1

Telepoints are defined in .terrn2 file like this (Updated 2017-03-01):

[Teleport]
; numbered names must be consecutive: Telepoint1, Telepoint2...

Telepoint1/Name=Left side flag
Telepoint1/Position=177, 1.5, 210

Telepoint2/Name=Right side flag
Telepoint2/Position=180, 1.5, 192

Telepoint3/Name=Behind the garage
Telepoint3/Position= 221.3, 1.3, 152.5

Fixes #761

@only-a-ptr only-a-ptr force-pushed the only-a-ptr:teleport branch from 2180e74 to 1a87719 Feb 28, 2017

@only-a-ptr

This comment has been minimized.

Copy link
Member Author

commented Feb 28, 2017

Added a character icon to the minimap:

screenshot_2017-02-28_21-19-52_1

@ghost

This comment has been minimized.

Copy link

commented Mar 1, 2017

👍

@only-a-ptr only-a-ptr added this to the 0.4.8 milestone Mar 1, 2017

@only-a-ptr only-a-ptr force-pushed the only-a-ptr:teleport branch 2 times, most recently from 0b17799 to 7bedc42 Mar 1, 2017

@only-a-ptr

This comment has been minimized.

Copy link
Member Author

commented Mar 1, 2017

Final feat: Alt-mode = teleport anywhere

With Teleport window open, hold Alt key: telepoints will disappear and you'll have a round lime cursor on the minimap. Click anywhere to teleport.

2017-03-01--teleport-altmode-demo

@tritonas00

This comment has been minimized.

Copy link
Collaborator

commented Mar 2, 2017

Great!!

@only-a-ptr only-a-ptr force-pushed the only-a-ptr:teleport branch from 7bedc42 to 919f08f Mar 2, 2017

@only-a-ptr

This comment has been minimized.

Copy link
Member Author

commented Mar 2, 2017

@tritonas00 I incorporated the build fix into original commit and deleted your reports.
Thanks for testing!

@tritonas00

This comment has been minimized.

Copy link
Collaborator

commented Mar 2, 2017

hmm i can't make it work. I press f2, the mini map opens then i press and hold alt i see the round lime cursor and i click anywhere in the map but nothing happens 😊

EDIT, it works when i first click anywhere in the map (keep the button pressed), then press alt and release the mouse button... Is this normal?

@only-a-ptr

This comment has been minimized.

Copy link
Member Author

commented Mar 3, 2017

@tritonas00 Well, it's a bit weird behavior but it makes sense. I'm relying on 'mouse click' event which apparently doesn't measure time interval between press/release and arrives when the button is released.
I'll just leave it like that #lazy

@vido89

This comment has been minimized.

Copy link

commented Mar 3, 2017

@only-a-ptr only-a-ptr force-pushed the only-a-ptr:teleport branch 3 times, most recently from c3beff6 to c66c671 Mar 28, 2017

only-a-ptr added some commits Feb 24, 2017

Teleport
When in character mode, press [F2] to open UI and click on minimap.

DETAILS:
* Added teleport to terrn2: section [Teleport]
* Added Teleport GUI window

NOTE: The minimap texture is not yet displayed on the TeleportGUI - only the clickable icons.
Teleport: Alt-mode = teleport anywhere
With Teleport window open, hold Alt key: telepoints will disappear and you'll have a round lime cursor on the minimap. Click to teleport.
📐 Tidy-up of `TerrainGeometryManager`
No change to logic, only renaming/re-formatting/removal of dead code.

CLEANUPS:
* removed dead function `Ogre::Vector3 getTerrainPosition()` and subsequently unused member `Ogre::Vector3 terrainPos`
* member variable `int terrainLayers` (number of layers) removed - It was only used in a single function -> transformed to local var.
* Members `int pageMinX, pageMaxX, pageMinZ, pageMaxZ` removed; only used in single function -> transformed to locals.
* Removed dead functon `Ogre::String getCompositeMaterialName()`
* Purged unused includes

@only-a-ptr only-a-ptr force-pushed the only-a-ptr:teleport branch 3 times, most recently from 07102ed to 2d8546d Mar 29, 2017

@codacy-bot

This comment has been minimized.

Copy link

commented Apr 1, 2017

Codacy Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 16
- Added 4
           

See the complete overview on Codacy

@only-a-ptr only-a-ptr merged commit 3b0ea50 into RigsOfRods:master Apr 1, 2017

1 check passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details

@only-a-ptr only-a-ptr deleted the only-a-ptr:teleport branch Apr 1, 2017

@tritonas00 tritonas00 referenced this pull request Apr 10, 2017

Merged

Terrn2: 2 new options #1314

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.