Code review comment for lp:~andreas-pokorny/mir/add-option-to-disable-acceleration

Daniel van Vugt (vanvugt) wrote :

Thanks for putting the time into this. I suspect this is slightly the wrong approach though:

+ bool enable_cursor_acceleration{true};

would be better replaced by an enum that matches enum libinput_config_accel_profile:


So we could set it as LIBINPUT_CONFIG_ACCEL_PROFILE_NONE (or FLAT or ADAPTIVE) instead of enable_cursor_acceleration=false.

review: Needs Fixing

« Back to merge proposal