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

Tweak third person eye location #1777

Closed
wants to merge 1 commit into from
Closed

Tweak third person eye location #1777

wants to merge 1 commit into from

Conversation

Zeno-
Copy link
Contributor

@Zeno- Zeno- commented Oct 27, 2014

No description provided.

@Zeno- Zeno- changed the title https://github.com/minetest/minetest/issues/1774 Tweak thrid person eye location Oct 27, 2014
@Zeno- Zeno- changed the title Tweak thrid person eye location Tweak third person eye location Oct 27, 2014
@donat-b
Copy link

donat-b commented Oct 27, 2014

👍 Way better now.

@Megaf
Copy link
Contributor

Megaf commented Oct 27, 2014

Screenshot?

@donat-b
Copy link

donat-b commented Oct 27, 2014

img

@Zeno-
Copy link
Contributor Author

Zeno- commented Oct 27, 2014

hmm

Edit: Oops, donat-b beat me to it

@Megaf
Copy link
Contributor

Megaf commented Oct 27, 2014

It looks much better, now it' s usable.
I like it.

@SmallJoker
Copy link
Member

Looks good.
But, why is it left-sided?

@Zeno-
Copy link
Contributor Author

Zeno- commented Oct 28, 2014

@SmallJoker because the wield item is at the right

@Calinou
Copy link
Member

Calinou commented Oct 28, 2014

Shoulder camera can be a problem sometimes, eg. in tight spaces.

Also, generally, the camera would be at the right. It works like that in most third person shooters.

@Zeno-
Copy link
Contributor Author

Zeno- commented Oct 29, 2014

@Calinou I didn't see your PR before I did this (sorry)

Ok I'll move it to the right if that's the what is more common. I did notice the issue in tight spaces but the camera immediately behind seemed to have the same (similar) issue so I didn't dwell on it for too long.

#1774

Also used modified version of #1772

Update: Moved to be over the right shoulder
@hoodedice
Copy link

Hmm, I think it is better to put this as an option... maybe a fourth F7 view-mode?

@BlockMen
Copy link
Contributor

BlockMen commented Nov 8, 2014

I prefer the current style (wow, suprise).

We already have the option to use different camera position and i have seen no server yet using it.
Now the interesting question is: Do ppl not know about it, is it to hard for most to change or is it not wanted?

@hoodedice
Copy link

"Now the interesting question is: Do ppl not know about it, is it to hard for most to change or is it not wanted?"

I actually did mess around with it, and did put the results on IRC, but people weren't so interested in it. Surprising, considering the fact that it is just one line of code to add/edit.

@Zeno-
Copy link
Contributor Author

Zeno- commented Nov 9, 2014

@BlockMen what would happen if the player wanted to play on a server but still change the eye position?

Perhaps a better solution than this current one of mine would be to add the eye position option as an option for the client (in minetest.conf) rather than it having to be modified by a server-side mod using Lua and set_eye_offset. If that solution is preferable (and I think that it probably would be) then I am happy to close this PR.

@Zeno-
Copy link
Contributor Author

Zeno- commented Nov 9, 2014

Closed/revoked for now

@Zeno- Zeno- closed this Nov 9, 2014
@BlockMen
Copy link
Contributor

BlockMen commented Nov 9, 2014

@Zeno- Then make sure it only applies when its nod set/modified by mods already. Else you would break that feature.

@Zeno-
Copy link
Contributor Author

Zeno- commented Nov 9, 2014

@BlockMen for sure. It needs more thought and input which is why I closed before waiting for your reply

Edit: In channel I said you were right and perhaps some kind of hybrid approach would be better

@Zeno- Zeno- deleted the tweak_3rdPerson_view branch December 6, 2014 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants