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

[GUI] New <depth> tag for stereoscopic effect for controls. #7855

Merged
merged 8 commits into from Aug 31, 2015

Conversation

afedchin
Copy link
Member

This added a new tag for control/group/window and defines stereoscopic effect for them. The possible values of tag are [-1.0, 1.0] which bring control "to back" and "to front".

ping @da-anda @mkortstiege @MartijnKaijser

also please someone review the description and name of setting.

@afedchin afedchin added Type: Feature non-breaking change which adds functionality API change RFC PR submitted for gathering feedback v16 Jarvis labels Aug 19, 2015
@mkortstiege
Copy link
Member

Thanks, looks simple and clean. @da-anda you're going to adjust Confluence?

@da-anda
Copy link
Member

da-anda commented Aug 19, 2015

yes, will give it a try, even though I have no clue of our skinning engine. Will at least hack around in Confluence to test the effect :)

@xhaggi
Copy link
Member

xhaggi commented Aug 19, 2015

@afedchin as we have something similar for subtitles and we name it stereoscopic depth e.g. CSettings::SETTING_SUBTITLES_STEREOSCOPICDEPTH, OVERLAY::GetStereoscopicDepthetc. should we change it from strength to depth to be consistent in naming?

@da-anda
Copy link
Member

da-anda commented Aug 19, 2015

@xhaggi I think you can see the "depths" value as the location within the available depths range, while "strength" defines the size of the range. So not sure if they should use the same naming.

@xhaggi
Copy link
Member

xhaggi commented Aug 19, 2015

makes sense :)

#empty strings from id 36550 to 36599
#: system/settings/settings.xml
msgctxt "#36550"
msgid "GUI Stereo Effect Strength"

This comment was marked as spam.

This comment was marked as spam.

@da-anda
Copy link
Member

da-anda commented Aug 20, 2015

works great - thanks. Adjustments to Confluence will take a while and can/should be done independent of this PR. So good to go from my end, once jjd-uk checked the labels

@@ -17115,7 +17115,18 @@ msgctxt "#36549"
msgid "Use iOS8 compatible AirPlay support. If you have trouble with older iOS devices detecting this application as a valid target try switching this off. This option requires a restart to take effect!"
msgstr ""

#empty strings from id 36550 to 36599
#: system/settings/settings.xml

This comment was marked as spam.

@afedchin
Copy link
Member Author

rebased and squashed.
jenkins build this please

@afedchin
Copy link
Member Author

jenkins is busy today. if there isn't objection and build will success, @MartijnKaijser please merge this in next window

@MartijnKaijser MartijnKaijser added this to the Jarvis 16.0-alpha3 milestone Aug 28, 2015
@da-anda
Copy link
Member

da-anda commented Aug 28, 2015

@jjd-uk reads good, but the puctuation seems odd. I'd go with:
Defines the strength of the stereoscopic 3D effect in the GUI. This is done by controlling the depth of perception within the GUI, so the higher the value, the more elements will pop out of the screen. [Zero] Disables the stereoscopic 3D effect of the GUI.[CR]For a good visual experience, the value should be higher for small screens and lower for large screens. Note: this is not supported by all skins.

@jjd-uk
Copy link
Member

jjd-uk commented Aug 28, 2015

That's fine with me

@afedchin
Copy link
Member Author

updated. thanks @jjd-uk @da-anda

@afedchin
Copy link
Member Author

jenkins build this please

MartijnKaijser added a commit that referenced this pull request Aug 31, 2015
[GUI] New <depth> tag for stereoscopic effect for controls.
@MartijnKaijser MartijnKaijser merged commit 108c034 into xbmc:master Aug 31, 2015
@afedchin afedchin deleted the stereo_controls branch February 17, 2016 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RFC PR submitted for gathering feedback Type: Feature non-breaking change which adds functionality v16 Jarvis
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants