Skip to content

Rotary Encoder's "Allow Wrap around" prevents saving the configuration if enabled #1662

Description

@MatteoGodzilla
  • I have checked documentation and the rotary encoder page does not say anything about this problem
  • I have checked in the github issues for this problem and it is not already reported
  • Issue is present on version v.0.7.12

Expected Behavior

Enabling the option and clicking save would soft-refresh the page and report "saved successfully" at the bottom

Current Behavior

When configuring Rotary encoders, it is possible to save the configuration of everything except the option for "allow wrap around", because when pressing the save button after enabling this option for at least one of the two encoders the save button does not soft-refresh the page.
When refreshing the configurator and going into the same section, old values are shown.

I personally have not tested with older firmware versions, i have found issue #1503 but it is closed

Context

Version, as reported from the web configurator

Pico (GP2040-CE_0.7.12_Pico.uf2)
Current: v0.7.12
Latest: v0.7.12
Architecture: rp2040
Build Type: Release
Build: 0014e4a

This is not a commercial device, but one controller i'm personally developing.

Rotary encoder is configured like this (even though i don't think it's causing the issue):

  • Pin A: 27
  • Pin B: 26
  • PPR: 600
  • Multiplier: 1
  • Reset after: 0ms
  • Control mode: Left Analog Y

Operating System: Linux
Browser: Firefox

Steps to Reproduce

  • Enable Rotary Encoder add-on
  • Enable Encoder 1
  • Set up just the pins
  • Save -> Should display "saved correctly"
  • Enable "allow wrap around for encoder 1"
  • Save -> Does not work

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugBug in the firmware - Something is not working as intended

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions