Implement amplitude
to Input.vibrate_handheld
Co-authored-by: A Thousand Ships <96648715+AThousandShips@users.noreply.github.com> Co-authored-by: m4gr3d <m4gr3d@users.noreply.github.com>
This commit is contained in:
parent
4e9543d849
commit
789c6ebdfd
17 changed files with 132 additions and 35 deletions
41
core/input/input.compat.inc
Normal file
41
core/input/input.compat.inc
Normal file
|
@ -0,0 +1,41 @@
|
|||
/**************************************************************************/
|
||||
/* input.compat.inc */
|
||||
/**************************************************************************/
|
||||
/* This file is part of: */
|
||||
/* GODOT ENGINE */
|
||||
/* https://godotengine.org */
|
||||
/**************************************************************************/
|
||||
/* Copyright (c) 2014-present Godot Engine contributors (see AUTHORS.md). */
|
||||
/* Copyright (c) 2007-2014 Juan Linietsky, Ariel Manzur. */
|
||||
/* */
|
||||
/* Permission is hereby granted, free of charge, to any person obtaining */
|
||||
/* a copy of this software and associated documentation files (the */
|
||||
/* "Software"), to deal in the Software without restriction, including */
|
||||
/* without limitation the rights to use, copy, modify, merge, publish, */
|
||||
/* distribute, sublicense, and/or sell copies of the Software, and to */
|
||||
/* permit persons to whom the Software is furnished to do so, subject to */
|
||||
/* the following conditions: */
|
||||
/* */
|
||||
/* The above copyright notice and this permission notice shall be */
|
||||
/* included in all copies or substantial portions of the Software. */
|
||||
/* */
|
||||
/* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, */
|
||||
/* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF */
|
||||
/* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. */
|
||||
/* IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY */
|
||||
/* CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, */
|
||||
/* TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE */
|
||||
/* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. */
|
||||
/**************************************************************************/
|
||||
|
||||
#ifndef DISABLE_DEPRECATED
|
||||
|
||||
void Input::_vibrate_handheld_bind_compat_91143(int p_duration_ms) {
|
||||
vibrate_handheld(p_duration_ms, -1.0);
|
||||
}
|
||||
|
||||
void Input::_bind_compatibility_methods() {
|
||||
ClassDB::bind_compatibility_method(D_METHOD("vibrate_handheld", "duration_ms"), &Input::_vibrate_handheld_bind_compat_91143, DEFVAL(500));
|
||||
}
|
||||
|
||||
#endif // DISABLE_DEPRECATED
|
|
@ -29,6 +29,7 @@
|
|||
/**************************************************************************/
|
||||
|
||||
#include "input.h"
|
||||
#include "input.compat.inc"
|
||||
|
||||
#include "core/config/project_settings.h"
|
||||
#include "core/input/default_controller_mappings.h"
|
||||
|
@ -120,7 +121,7 @@ void Input::_bind_methods() {
|
|||
ClassDB::bind_method(D_METHOD("get_joy_vibration_duration", "device"), &Input::get_joy_vibration_duration);
|
||||
ClassDB::bind_method(D_METHOD("start_joy_vibration", "device", "weak_magnitude", "strong_magnitude", "duration"), &Input::start_joy_vibration, DEFVAL(0));
|
||||
ClassDB::bind_method(D_METHOD("stop_joy_vibration", "device"), &Input::stop_joy_vibration);
|
||||
ClassDB::bind_method(D_METHOD("vibrate_handheld", "duration_ms"), &Input::vibrate_handheld, DEFVAL(500));
|
||||
ClassDB::bind_method(D_METHOD("vibrate_handheld", "duration_ms", "amplitude"), &Input::vibrate_handheld, DEFVAL(500), DEFVAL(-1.0));
|
||||
ClassDB::bind_method(D_METHOD("get_gravity"), &Input::get_gravity);
|
||||
ClassDB::bind_method(D_METHOD("get_accelerometer"), &Input::get_accelerometer);
|
||||
ClassDB::bind_method(D_METHOD("get_magnetometer"), &Input::get_magnetometer);
|
||||
|
@ -803,8 +804,8 @@ void Input::stop_joy_vibration(int p_device) {
|
|||
joy_vibration[p_device] = vibration;
|
||||
}
|
||||
|
||||
void Input::vibrate_handheld(int p_duration_ms) {
|
||||
OS::get_singleton()->vibrate_handheld(p_duration_ms);
|
||||
void Input::vibrate_handheld(int p_duration_ms, float p_amplitude) {
|
||||
OS::get_singleton()->vibrate_handheld(p_duration_ms, p_amplitude);
|
||||
}
|
||||
|
||||
void Input::set_gravity(const Vector3 &p_gravity) {
|
||||
|
|
|
@ -264,6 +264,11 @@ private:
|
|||
|
||||
EventDispatchFunc event_dispatch_function = nullptr;
|
||||
|
||||
#ifndef DISABLE_DEPRECATED
|
||||
void _vibrate_handheld_bind_compat_91143(int p_duration_ms = 500);
|
||||
static void _bind_compatibility_methods();
|
||||
#endif // DISABLE_DEPRECATED
|
||||
|
||||
protected:
|
||||
static void _bind_methods();
|
||||
|
||||
|
@ -323,7 +328,7 @@ public:
|
|||
|
||||
void start_joy_vibration(int p_device, float p_weak_magnitude, float p_strong_magnitude, float p_duration = 0);
|
||||
void stop_joy_vibration(int p_device);
|
||||
void vibrate_handheld(int p_duration_ms = 500);
|
||||
void vibrate_handheld(int p_duration_ms = 500, float p_amplitude = -1.0);
|
||||
|
||||
void set_mouse_position(const Point2 &p_posf);
|
||||
|
||||
|
|
|
@ -185,7 +185,7 @@ public:
|
|||
virtual int get_process_id() const;
|
||||
virtual bool is_process_running(const ProcessID &p_pid) const = 0;
|
||||
virtual int get_process_exit_code(const ProcessID &p_pid) const = 0;
|
||||
virtual void vibrate_handheld(int p_duration_ms = 500) {}
|
||||
virtual void vibrate_handheld(int p_duration_ms = 500, float p_amplitude = -1.0) {}
|
||||
|
||||
virtual Error shell_open(const String &p_uri);
|
||||
virtual Error shell_show_in_file_manager(String p_path, bool p_open_folder = true);
|
||||
|
|
|
@ -397,11 +397,14 @@
|
|||
<method name="vibrate_handheld">
|
||||
<return type="void" />
|
||||
<param index="0" name="duration_ms" type="int" default="500" />
|
||||
<param index="1" name="amplitude" type="float" default="-1.0" />
|
||||
<description>
|
||||
[b]Note:[/b] While [code skip-lint]amplitude[/code] expects a value between 0 and 1, -1 does the default amplitude for the device.
|
||||
Vibrate the handheld device for the specified duration in milliseconds.
|
||||
[b]Note:[/b] This method is implemented on Android, iOS, and Web. It has no effect on other platforms.
|
||||
[b]Note:[/b] For Android, [method vibrate_handheld] requires enabling the [code]VIBRATE[/code] permission in the export preset. Otherwise, [method vibrate_handheld] will have no effect.
|
||||
[b]Note:[/b] For iOS, specifying the duration is only supported in iOS 13 and later.
|
||||
[b]Note:[/b] For Web, the amplitude cannot be changed.
|
||||
[b]Note:[/b] Some web browsers such as Safari and Firefox for Android do not support [method vibrate_handheld].
|
||||
</description>
|
||||
</method>
|
||||
|
|
|
@ -337,3 +337,10 @@ GH-91098
|
|||
Validate extension JSON: Error: Field 'classes/RichTextLabel/methods/remove_paragraph/arguments': size changed value in new API, from 1 to 2.
|
||||
|
||||
Added optional argument. Compatibility method registered.
|
||||
|
||||
|
||||
GH-91143
|
||||
--------
|
||||
Validate extension JSON: Error: Field 'classes/Input/methods/vibrate_handheld/arguments': size changed value in new API, from 1 to 2.
|
||||
|
||||
Added optional argument. Compatibility method registered.
|
|
@ -894,16 +894,25 @@ class Godot(private val context: Context) : SensorEventListener {
|
|||
*/
|
||||
@SuppressLint("MissingPermission")
|
||||
@Keep
|
||||
private fun vibrate(durationMs: Int) {
|
||||
private fun vibrate(durationMs: Int, amplitude: Int) {
|
||||
if (durationMs > 0 && requestPermission("VIBRATE")) {
|
||||
val vibratorService = getActivity()?.getSystemService(Context.VIBRATOR_SERVICE) as Vibrator? ?: return
|
||||
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
|
||||
if (amplitude <= -1) {
|
||||
vibratorService.vibrate(
|
||||
VibrationEffect.createOneShot(
|
||||
durationMs.toLong(),
|
||||
VibrationEffect.DEFAULT_AMPLITUDE
|
||||
)
|
||||
)
|
||||
} else {
|
||||
vibratorService.vibrate(
|
||||
VibrationEffect.createOneShot(
|
||||
durationMs.toLong(),
|
||||
amplitude
|
||||
)
|
||||
)
|
||||
}
|
||||
} else {
|
||||
// deprecated in API 26
|
||||
vibratorService.vibrate(durationMs.toLong())
|
||||
|
|
|
@ -72,7 +72,7 @@ GodotJavaWrapper::GodotJavaWrapper(JNIEnv *p_env, jobject p_activity, jobject p_
|
|||
_get_granted_permissions = p_env->GetMethodID(godot_class, "getGrantedPermissions", "()[Ljava/lang/String;");
|
||||
_get_ca_certificates = p_env->GetMethodID(godot_class, "getCACertificates", "()Ljava/lang/String;");
|
||||
_init_input_devices = p_env->GetMethodID(godot_class, "initInputDevices", "()V");
|
||||
_vibrate = p_env->GetMethodID(godot_class, "vibrate", "(I)V");
|
||||
_vibrate = p_env->GetMethodID(godot_class, "vibrate", "(II)V");
|
||||
_get_input_fallback_mapping = p_env->GetMethodID(godot_class, "getInputFallbackMapping", "()Ljava/lang/String;");
|
||||
_on_godot_setup_completed = p_env->GetMethodID(godot_class, "onGodotSetupCompleted", "()V");
|
||||
_on_godot_main_loop_started = p_env->GetMethodID(godot_class, "onGodotMainLoopStarted", "()V");
|
||||
|
@ -331,11 +331,18 @@ void GodotJavaWrapper::init_input_devices() {
|
|||
}
|
||||
}
|
||||
|
||||
void GodotJavaWrapper::vibrate(int p_duration_ms) {
|
||||
void GodotJavaWrapper::vibrate(int p_duration_ms, float p_amplitude) {
|
||||
if (_vibrate) {
|
||||
JNIEnv *env = get_jni_env();
|
||||
ERR_FAIL_NULL(env);
|
||||
env->CallVoidMethod(godot_instance, _vibrate, p_duration_ms);
|
||||
|
||||
int j_amplitude = -1.0;
|
||||
|
||||
if (p_amplitude != -1.0) {
|
||||
j_amplitude = CLAMP(int(p_amplitude * 255), 1, 255);
|
||||
}
|
||||
|
||||
env->CallVoidMethod(godot_instance, _vibrate, p_duration_ms, j_amplitude);
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
@ -102,7 +102,7 @@ public:
|
|||
Vector<String> get_granted_permissions() const;
|
||||
String get_ca_certificates() const;
|
||||
void init_input_devices();
|
||||
void vibrate(int p_duration_ms);
|
||||
void vibrate(int p_duration_ms, float p_amplitude = -1.0);
|
||||
String get_input_fallback_mapping();
|
||||
int create_new_godot_instance(const List<String> &args);
|
||||
void begin_benchmark_measure(const String &p_context, const String &p_label);
|
||||
|
|
|
@ -746,8 +746,8 @@ ANativeWindow *OS_Android::get_native_window() const {
|
|||
#endif
|
||||
}
|
||||
|
||||
void OS_Android::vibrate_handheld(int p_duration_ms) {
|
||||
godot_java->vibrate(p_duration_ms);
|
||||
void OS_Android::vibrate_handheld(int p_duration_ms, float p_amplitude) {
|
||||
godot_java->vibrate(p_duration_ms, p_amplitude);
|
||||
}
|
||||
|
||||
String OS_Android::get_config_path() const {
|
||||
|
|
|
@ -153,7 +153,7 @@ public:
|
|||
|
||||
virtual Error move_to_trash(const String &p_path) override;
|
||||
|
||||
void vibrate_handheld(int p_duration_ms) override;
|
||||
void vibrate_handheld(int p_duration_ms, float p_amplitude = -1.0) override;
|
||||
|
||||
virtual String get_config_path() const override;
|
||||
|
||||
|
|
|
@ -51,7 +51,7 @@ public:
|
|||
static void alert(const char *p_alert, const char *p_title);
|
||||
|
||||
bool supports_haptic_engine();
|
||||
void vibrate_haptic_engine(float p_duration_seconds);
|
||||
void vibrate_haptic_engine(float p_duration_seconds, float p_amplitude);
|
||||
|
||||
String get_model() const;
|
||||
String get_rate_url(int p_app_id) const;
|
||||
|
|
|
@ -69,21 +69,41 @@ CHHapticEngine *iOS::get_haptic_engine_instance() API_AVAILABLE(ios(13)) {
|
|||
return haptic_engine;
|
||||
}
|
||||
|
||||
void iOS::vibrate_haptic_engine(float p_duration_seconds) API_AVAILABLE(ios(13)) {
|
||||
void iOS::vibrate_haptic_engine(float p_duration_seconds, float p_amplitude) API_AVAILABLE(ios(13)) {
|
||||
if (@available(iOS 13, *)) { // We need the @available check every time to make the compiler happy...
|
||||
if (supports_haptic_engine()) {
|
||||
CHHapticEngine *cur_haptic_engine = get_haptic_engine_instance();
|
||||
if (cur_haptic_engine) {
|
||||
NSDictionary *hapticDict = @{
|
||||
NSDictionary *hapticDict;
|
||||
if (p_amplitude < 0) {
|
||||
hapticDict = @{
|
||||
CHHapticPatternKeyPattern : @[
|
||||
@{CHHapticPatternKeyEvent : @{
|
||||
CHHapticPatternKeyEventType : CHHapticEventTypeHapticContinuous,
|
||||
CHHapticPatternKeyTime : @(CHHapticTimeImmediate),
|
||||
CHHapticPatternKeyEventDuration : @(p_duration_seconds)
|
||||
CHHapticPatternKeyEventDuration : @(p_duration_seconds),
|
||||
},
|
||||
},
|
||||
],
|
||||
};
|
||||
} else {
|
||||
hapticDict = @{
|
||||
CHHapticPatternKeyPattern : @[
|
||||
@{CHHapticPatternKeyEvent : @{
|
||||
CHHapticPatternKeyEventType : CHHapticEventTypeHapticContinuous,
|
||||
CHHapticPatternKeyTime : @(CHHapticTimeImmediate),
|
||||
CHHapticPatternKeyEventDuration : @(p_duration_seconds),
|
||||
CHHapticPatternKeyEventParameters : @[
|
||||
@{
|
||||
CHHapticPatternKeyParameterID : @("HapticIntensity"),
|
||||
CHHapticPatternKeyParameterValue : @(p_amplitude)
|
||||
},
|
||||
],
|
||||
},
|
||||
},
|
||||
],
|
||||
};
|
||||
}
|
||||
|
||||
NSError *error;
|
||||
CHHapticPattern *pattern = [[CHHapticPattern alloc] initWithDictionary:hapticDict error:&error];
|
||||
|
|
|
@ -123,7 +123,7 @@ public:
|
|||
virtual String get_unique_id() const override;
|
||||
virtual String get_processor_name() const override;
|
||||
|
||||
virtual void vibrate_handheld(int p_duration_ms = 500) override;
|
||||
virtual void vibrate_handheld(int p_duration_ms = 500, float p_amplitude = -1.0) override;
|
||||
|
||||
virtual bool _check_internal_feature_support(const String &p_feature) override;
|
||||
|
||||
|
|
|
@ -571,9 +571,13 @@ String OS_IOS::get_system_font_path(const String &p_font_name, int p_weight, int
|
|||
return ret;
|
||||
}
|
||||
|
||||
void OS_IOS::vibrate_handheld(int p_duration_ms) {
|
||||
void OS_IOS::vibrate_handheld(int p_duration_ms, float p_amplitude) {
|
||||
if (ios->supports_haptic_engine()) {
|
||||
ios->vibrate_haptic_engine((float)p_duration_ms / 1000.f);
|
||||
if (p_amplitude > 0.0) {
|
||||
p_amplitude = CLAMP(p_amplitude, 0.0, 1.0);
|
||||
}
|
||||
|
||||
ios->vibrate_haptic_engine((float)p_duration_ms / 1000.f, p_amplitude);
|
||||
} else {
|
||||
// iOS <13 does not support duration for vibration
|
||||
AudioServicesPlaySystemSound(kSystemSoundID_Vibrate);
|
||||
|
|
|
@ -174,7 +174,7 @@ void OS_Web::add_frame_delay(bool p_can_draw) {
|
|||
#endif
|
||||
}
|
||||
|
||||
void OS_Web::vibrate_handheld(int p_duration_ms) {
|
||||
void OS_Web::vibrate_handheld(int p_duration_ms, float p_amplitude) {
|
||||
godot_js_input_vibrate_handheld(p_duration_ms);
|
||||
}
|
||||
|
||||
|
|
|
@ -98,7 +98,7 @@ public:
|
|||
// Implemented in web_main.cpp loop callback instead.
|
||||
void add_frame_delay(bool p_can_draw) override;
|
||||
|
||||
void vibrate_handheld(int p_duration_ms) override;
|
||||
void vibrate_handheld(int p_duration_ms, float p_amplitude) override;
|
||||
|
||||
String get_cache_path() const override;
|
||||
String get_config_path() const override;
|
||||
|
|
Loading…
Reference in a new issue